Spin-off from bug 364620. The effect of not having the same styling is that if you have an icon in a menulist menu, the icon is too far at the right compared with the surrounding menuitem icons. Pinstripe doesn't have the issue, but I somehow forget to add it to classic.
JFTR: Pinstripe styles: toolkit/themes/pinstripe/global/menu.css#180 etc
Created attachment 249718 [details] [diff] [review] Fix menus in menulists This will fix the issue with menus in menulists. If you have a menu in a menulist with an icon, the icon does not line up with the label text anymore (same behaviour as menuitems). Not sure if it's too late for 1.1...
Any chance of a before/after screenshot?
Created attachment 249782 [details] Before/after screenshot Here's a screenshot. Note that I wouldn't mind having the icons left margin align with the left margin of the menuitem labels. But (iirc) that would affect the checkmarks as well - and they should not be aligned with the labels. So, this was a compromise when I did the native styling. What I did was that I forgot the menus, though. Note also that I have never seen any icon in a "real" mac menulist. Ideally, I would like one widget for menuitems with checkmarks and one other for menuitems with icons - now they use the same menu-iconic one.
Comment on attachment 249718 [details] [diff] [review] Fix menus in menulists Removing obsolete request...
Karsten, can you check this in on trunk, please? Some baking is always good.
Comment on attachment 249718 [details] [diff] [review] Fix menus in menulists first-a=me, need one more
Landed on trunk.
Comment on attachment 249718 [details] [diff] [review] Fix menus in menulists a=me for 1.1.1
Landed on MOZILLA_1_8_BRANCH.