[rfc][icedtea-web] Launching PolicyEditor from CertWarningPane

Andrew Azores 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.

Done.

>
>
> ty!
>
>  J.

Thanks,

-- 
Andrew A



More information about the distro-pkg-dev mailing list