User Agent: Mozilla/5.0 (Windows NT 5.1; rv:25.0) Gecko/20100101 Firefox/25.0 (Beta/Release) Build ID: 20131025151332 Steps to reproduce: Either by pressing F11 to go into Fullscreen mode, or applying the "autohide" attribute to the menubar in SeaMonkey, correctly hides the bar, however pressing Alt does not unhide the bar. Pressing Alt and down (or the appropriate access key) will display a menu, but the menubar itself remains hidden. Actual results: When the autohide attribute is used on a menu toolbar, it loads up this binding: chrome://global/content/bindings/toolbar.xml#toolbar-menubar-autohide The binding in turn applies the inactive=true attribute to the menu toolbar. In Firefox, pressing the Alt button removes the inactive=true attribute which temporarily shows the menu toolbar. Any activity outside of this should reapply inactive=true and rehide the menu toolbar

This was originally applied to SeaMonkey in Bug 540629

So this is a regression (somewhere) between 2.21 release & 2.22 release. > In Firefox, pressing the Alt button removes the inactive=true attribute which > temporarily shows the menu toolbar I do not see that at all? From what I can see, FF never displays the menu toolbar at all when in Full Screen. IMO, our method is much better :-). Similar FF Bug 648504 - Alt-key access to pull-down Menu Bar menus does not work in full-screen mode

In Firefox, this method is used for when the menubar is hidden in normal mode.

Maybe I can summarize better: Firefox 3.6 introduced the ability to autohide the menu toolbar. This was created by Bug 477256, and works like this: Unchecking the "Menu Bar" item in the toolbar context menu applies the "autohide=true" attribute to the menu toolbar. This in turn activates the toolbar-menubar-autohide binding which applies the "inactive=true" attribute which tells some CSS to hide the toolbar. Pressing the Alt button causes the binding to remove the "inactive=true" attribute which temporarily shows the menu toolbar until some other action is taken, which reinstate the "inactive=true" attribute. SeaMonkey devs did not implement this into SeaMonkey for normal mode. SeaMonkey Bug 540629 adds the autohide attribute to the menu toolbar when in Fullscreen mode, which activates the toolbar-menubar-autohide binding, etc. This effectively fixes Bug 648504 and Bug 575755 for Firefox in SeaMonkey.

Maybe I can summarize better: Firefox 3.6 introduced the ability to autohide the menu toolbar. This was created by Bug 477256, and works like this: Unchecking the "Menu Bar" item in the toolbar context menu applies the "autohide=true" attribute to the menu toolbar. This in turn activates the toolbar-menubar-autohide binding which applies the "inactive=true" attribute which tells some CSS to hide the toolbar. Pressing the Alt button causes the binding to remove the "inactive=true" attribute which temporarily shows the menu toolbar until some other action is taken, which reinstate the "inactive=true" attribute. SeaMonkey devs did not implement this into SeaMonkey for normal mode. SeaMonkey Bug 540629 adds the autohide attribute to the menu toolbar when in Fullscreen mode, which activates the toolbar-menubar-autohide binding, etc. This effectively fixes Bug 648504 and Bug 575755 for Firefox in SeaMonkey. However, this is currently broken as the "inactive=true" attribute is applied, but is not correctly removed after pressing Alt.

Bug 653881 broke event handling in this edge case. I filed bug 943804 to have this fixed in the back end. It's possible to work around the bug in UI code and we might have to do this in beta or aurora, so I'm leaving this bug open for that.

Ironically bug 653881 made the workaround more minimal than it would have been, as the <children> is now an actual element and therefore I don't need to insert an explicit dummy element.

Comment on attachment 8339182 [details] [diff] [review] Branch workaround This works. r=me for branches. If Core Bug 943804 lands on branches then this patch isn't needed, right?

Right, this patch is only needed if 943804 doesn't uplift.

Comment on attachment 8339182 [details] [diff] [review] Branch workaround Bug 943804 will miss mozilla-beta so we'll need this for comm-beta a=me comm-beta

https://hg.mozilla.org/releases/comm-beta/rev/80f16a3e89db 2.24/2.25 aren't affected because bug 943084 landed in Gecko 27.