Reproduced under Ubuntu 12.04 LTS 32-bit (all branches), the other platforms are not affected. Steps to reproduce: Go to search bar and start typing Use Alt + up/down arrows to select the one-click search engines Expected results: The command should go through the one-click search engines list Actual results: Alt + up/down command closes and opens the dropdown without any search engines to be selected from the list
[Tracking Requested - why for this release]: this bug makes the keyboard binding we re-added for 36 in bug 1110678 unusable on Linux.
Florian, are you planning to work on this during the 36 cycle?
(In reply to Sylvestre Ledru [:sylvestre] from comment #2) > Florian, are you planning to work on this during the 36 cycle? I would prefer if someone else could do it. From a quick experiment, it seems that pressing only alt, or alt+any key closes the panel on keydown, which causes this.popupOpen to always be false at (http://hg.mozilla.org/mozilla-central/annotate/0467412f8413/browser/components/search/content/search.xml#l966), which is why we reopen it and it keeps blinking. I don't know which code is responsible for closing the panel when alt is pressed. I initially thought it was http://hg.mozilla.org/mozilla-central/annotate/6bfc0e1c4b29/toolkit/content/widgets/autocomplete.xml#l489 but no, that's not what's closing the panel here.
Neil, might you be able to look into this?
When Alt is pressed on Ubuntu, it normally focuses the menubar, which is at the top of the screen, and closes any popups. That's what is happening on my system. Is this the behaviour you're seeing here? If so, I recall that affected another popup somewhere, but don't remember the details. I'll look to see if there is a way to disable this.
(In reply to Neil Deakin from comment #5) > If so, I recall that affected another popup somewhere, but don't remember > the details. I'll look to see if there is a way to disable this. It affects the popup of the awesomebar too.
After testing this some more, it's actually the HUD that is opened with alt. When I disable the shortcut for this (System Settings -> Keyboard -> Shortcuts), the popup does not close when I press alt.
Created attachment 8559255 [details] [diff] [review] noaltsearch This following patch will disable the menubar from opening while the search popup is open.
Neil - I'm assigning this one to you as it looks like you're taking it.
Petruta, can you confirm that comment 7 is what is happening? Does the patch here help?
Neil, this bug is tracking 36+, can you get the patch ready to land on beta ASAP?
(In reply to Neil Deakin from comment #7) > After testing this some more, it's actually the HUD that is opened with alt. > When I disable the shortcut for this (System Settings -> Keyboard -> > Shortcuts), the popup does not close when I press alt. Indeed, disabling this system keyboard shortcut makes the bug disappear on the search panel (interestingly removing the system shortcut doesn't fix the problem on the awesomebar, so there may be an additional issue there). Unfortunately, attachment 8559255 [details] [diff] [review] doesn't seem to help.
OK, so if it's HUD that is taking the focus, then I'm not sure if there is a way around it in certain cases. Even other applications don't disable this.
ok, so, won't fix for 36.
I'm not sure what can be used to prevent this. I asked mconley and he wasn't sure, but perhaps Karl knows. The issue here is that pressing Alt on Ubuntu brings up the HUD. When the search panel is open, it would be desirable to allow the Alt+Arrow shortcuts to work. If we can't, alternatively we can either use another shortcut on Linux or not have a shortcut.
(In reply to Neil Deakin from comment #13) > Even other applications don't disable this. If the system is taking this key sequence before it gets to the application, and it sounds like it is, then we are limited in what we can do. Even if we could grab the keyboard focus, I'm not sure that would be the right thing to do. This is not a dropdown that the user explicitly opened, but merely one that happened to open as the user typed. Grabbing the keyboard focus would prevent other system shortcuts, such as volume keys from having effect. If Alt+Up/Down is already used on Unity, then I would suggest using a different shortcut. The functionality is already available with plain Up/Down, it seems.
OK, so I think we should either: 1. Do nothing and just close this bug, allowing those who don't have the system shortcut assigned to use this alt+cursor to navigate the search list. 2. Disable the shortcut on Linux. 3. Use a different shortcut on Linux. Florian, do you have a preference here?
(In reply to Neil Deakin from comment #17) > OK, so I think we should either: > > 1. Do nothing and just close this bug, allowing those who don't have the > system shortcut assigned to use this alt+cursor to navigate the search list. > 2. Disable the shortcut on Linux. > 3. Use a different shortcut on Linux. > > Florian, do you have a preference here? I think I would just go with 1; it's less work for us and avoids introducing inconsistencies in our code for specific OSes.