I have respun the webrev, http://cr.opensolaris.org/~padraig/ips-11211-v2/.
The problem was that the property 'always-show-image' property was set
on the ImageMenuItems and this overrode the setting you were changing.
Padraig
On 09/21/09 08:57, Padraig O'Briain wrote:
Thanks for pointing this out.
I will try to determine what is going wrong.
Padraig
On 09/18/09 15:47, Michal Pryc wrote:
Padraig,
This looks fine, with one minor nit:
- if I change to not show the icons in the menu in the appearance
preferences dialog, nothing happens. Other applications such as gedit
does not show menu icons at that stage.
To reproduce:
- start gedit (text editor)
- start Package Manager
- In the System -> Preferences -> Appearance
- Go to the tab Interface
- Uncheck "Show icons in menus"
Expected result:
- icons in both gedit and Package Manager should dissappear from menus.
Actual result
- the icons in Package Manager are still there. Restarting PM doesn't
help.
best
Michal
Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-11211-v1/, fixes
11211 Add stock images to PM top level menus
As part of adding icons to the requested menu items, the icons for
install_update, reload, remove, update_all were move from the glade
file directory to the directory for the default theme.
The icons arrow.png, new_star24x.png were removed as they are no
longer used.
The icons packagemanager.png used as the application icon is now
storeed in the mercurial repository instead of being copied from
another icon.
Padraig
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss