37992 - WIN32: Shift+Alt+Esc when submenu item highlighted doesn't destroy submenu

Status: VERIFIED FIXED

(Core :: XUL, defect, P3)

Description

[timeless]

Pressing Shift+Alt+Esc to switch tasks when a submenu is open and an item
highlighted doesn't destory the submenu nor its parent.  Using Alt+Esc
results in the menu being destroyed.

To duplicate: Select the Debug menu, open the Verification submenu, then
highlight Microsoft.  Then press Shift+Alt+Esc to switch to the 
previous-topmost task.
The Debug menu and the Verification submenu remain visible and on top.

Platform: Windows2000
Build ID: 2000050215

This bug made with 95% recycled thoughts.

Comment 1

[timeless]

This is related to the Alt-Esc bug which was fixed.  [The description was also 
copied from there; altered slightly to fit]

Comment 2

[David Hyatt]

not high priority.

Comment 3

[Mike Pinkerton (not reading bugmail)]

i see it as destroying the submenu, but it doesn't switch tasks. hyatt usually 
has these kind of bugs, why is it on my plate?

Comment 4

[Peter Trudelle]

Mass-moving all M20-M30 XPToolkit bugs to Future

Comment 5

[:deanis74]

I think what happens is this... When you press Alt+Esc the Alt cancels the open 
sub-menus.  When you press Shift+Alt+Esc (hold down Shift then Alt then press 
Esc), we aren't recognizing this.  But we have to treat this differently because 
we can conceivably have Shift+Alt shortcuts, so we can't dismiss the menu chain 
as soon as Alt is pressed.  We have to wait for Esc or lose focus or something.

Comment 6

[:deanis74]

Also seeing this when pressing Shift+Alt+Tab.

Comment 7

[:deanis74]

The problem is here:

http://lxr.mozilla.org/seamonkey/source/layout/xul/base/src/nsMenuListener.cpp#1
14

The Alt keypress dismisses the menu only if there is no other modifier key 
already down.  From the comment, this is for AltGR support.  Compare this to 
normal Windows behavior, though, where I can have none, any, or all modifier 
keys pressed and as soon as I press Alt down, the open menu is dismissed.

How do we handle both proper Windows behavior as well as AltGR support?

Comment 8

[:deanis74]

My patch for bug 45108 fixes this as well.

Comment 9

[Mike Pinkerton (not reading bugmail)]

fixed.

Comment 10

[:deanis74]

v