[rfc][icedtea-web] Launching PolicyEditor from CertWarningPane
aazores at redhat.com
Mon Mar 24 13:39:19 UTC 2014
On 03/24/2014 08:16 AM, Jiri Vanek wrote:
> Looks ok to go.
> Few feature works
> - with only one item the submenu donot have much sense, but it will
> greow (? :) ) so ok if finished in 1.5.1
I'm thinking maybe the Filter option can actually be the second item
under the View menu. It seems to be a fitting place to me, since the
filtering is probably not important enough to be always displayed, and
View is definitely the right menu for it to go in if it does go in a menu.
> - the modality and filter needs to be fixed until 1.5
> - when there are already stored items in policies, the first one is
> selected, and user would like actually edit the *newly* added.
> Probably not only filter, but be sure that the newly
> add/currnelty-should-be-edited item is really selected.
Well, the policy file is parse asynchronously when the editor opens, so
if this doesn't complete first, then when the dialogs add a new
codebase, it may not be the last one added. Since adding a new codebase
automatically selects it, this could make it so that the current
applet's codebase isn't actually selected. I'll start a new discussion
thread for this, and another one for modality as well.
> Please push the tweeks for attributevalidator as separate changeset
> and add trusted-only to news during commit please.
More information about the distro-pkg-dev