<command id="c1" hidden="true"> <broadcaster id="b1" hidden="true"> <toolbarbutton command="c1"> <menuitem1 observes="b1"> <menuitem2 command="c1"> toolbarbutton is hidden. menuitem1 is hidden. menuitem2 is not hidden.
Is this Mac-specific?
I'm seeing it on Windows as well.
Neil: any ideas here?
Created attachment 657318 [details] [diff] [review] Patch Well, one idea is to implement this for menuitems.
Comment on attachment 657318 [details] [diff] [review] Patch Seems reasonable to me, although I don't have a Mac to test that change.
Comment on attachment 657318 [details] [diff] [review] Patch Shane, can you test this out on Mac? It would be good to have so we can clean up http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser-social.js#96
I won't be able to get to this until (maybe) sometime next week. So I'd be happy if Shane can test and review.
I'll take a look at it soon, busy building...
Comment on attachment 657318 [details] [diff] [review] Patch patch looks fine, tests pass, and a quick patch on socialapi shows that this allows us to resolve the issue jaws mentions in comment 6.
Comment on attachment 657318 [details] [diff] [review] Patch This looks fine to me. I saw no problems testing it on OS X 10.7.5. I right-clicked in a bunch of different contexts, and always got an appropriate result. I also ran the included test manually a bunch of times, and saw no failures.
I'm having some trouble with this which I think is related to the fact that having a hidden attribute with *any* value acts as though hidden="true". So the only way to have the command work as though it is not hidden is to remove the attribute from the command. But if that is done, the patch doesn't update the hidden command on the menu item. ie, there seems no way to remove the hidden attribute from the menuitem using just commands.
hrm - please ignore the previous comment - XUL elements do want explicitly hidden="true". I'm not sure what problems I'm seeing but they don't seem to be related to this afaict.