Comment 49 for bug 597825

Revision history for this message
In , Ehsan-mozilla (ehsan-mozilla) wrote :

(In reply to comment #48)
> I tried the build, but when i press alt and then use the mouse somewhere else,
> the accelerators are still there.

Hmm, I think I was mistaken on how the Alt pressing should work. Here's what should happen IINM:

If Alt is pressed on a Window with menus, the first menu should get focus and accelerators should appear. In this case, pressing Alt again should turn off the accelerators again and the menu should lose focus. Clicking mouse somewhere other than on the menus should also dismiss the focus and turn off accelerators.

For windows without menus, pressing Alt should turn on accelerators, and pressing Alt again or using the mouse will not turn off the access keys.

So, I guess the problem you observed was with the menus, right? The current patch mistakenly turns off accelerators when Alt is pressed for the second time on any window.