Closed
Bug 1123292
Opened 10 years ago
Closed 10 years ago
[Ubuntu] Alt + up/down in searchbar doesn't select one-click search engines
Categories
(Firefox :: Search, defect)
Tracking
()
RESOLVED
WONTFIX
Iteration:
39.2 - 23 Mar
People
(Reporter: phorea, Assigned: enndeakin)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
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
Comment 1•10 years ago
|
||
[Tracking Requested - why for this release]: this bug makes the keyboard binding we re-added for 36 in bug 1110678 unusable on Linux.
status-firefox36:
--- → affected
status-firefox37:
--- → affected
status-firefox38:
--- → affected
tracking-firefox36:
--- → ?
tracking-firefox37:
--- → ?
Flags: qe-verify+
Flags: firefox-backlog+
Updated•10 years ago
|
status-firefox35:
--- → unaffected
tracking-firefox38:
--- → +
Updated•10 years ago
|
Points: --- → 2
Comment 2•10 years ago
|
||
Florian, are you planning to work on this during the 36 cycle?
Flags: needinfo?(florian)
Comment 3•10 years ago
|
||
(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.
Flags: needinfo?(florian)
Assignee | ||
Comment 5•10 years ago
|
||
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.
Flags: needinfo?(enndeakin)
Comment 6•10 years ago
|
||
(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.
Assignee | ||
Comment 7•10 years ago
|
||
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.
Assignee | ||
Comment 8•10 years ago
|
||
This following patch will disable the menubar from opening while the search popup is open.
Comment 9•10 years ago
|
||
Neil - I'm assigning this one to you as it looks like you're taking it.
Assignee: nobody → enndeakin
Assignee | ||
Comment 10•10 years ago
|
||
Petruta, can you confirm that comment 7 is what is happening? Does the patch here help?
Comment 11•10 years ago
|
||
Neil, this bug is tracking 36+, can you get the patch ready to land on beta ASAP?
Flags: needinfo?(enndeakin)
Updated•10 years ago
|
Status: NEW → ASSIGNED
Iteration: --- → 38.2 - 9 Feb
Comment 12•10 years ago
|
||
(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.
Assignee | ||
Comment 13•10 years ago
|
||
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.
Flags: needinfo?(enndeakin)
Comment 14•10 years ago
|
||
ok, so, won't fix for 36.
Updated•10 years ago
|
Iteration: 38.2 - 9 Feb → 38.3 - 23 Feb
Updated•10 years ago
|
Iteration: 38.3 - 23 Feb → 39.1 - 9 Mar
Assignee | ||
Comment 15•10 years ago
|
||
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.
Flags: needinfo?(karlt)
Comment 16•10 years ago
|
||
(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.
Flags: needinfo?(karlt)
Updated•10 years ago
|
Iteration: 39.1 - 9 Mar → 39.2 - 23 Mar
Assignee | ||
Comment 17•10 years ago
|
||
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?
Flags: needinfo?(florian)
Comment 18•10 years ago
|
||
(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.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: needinfo?(florian)
Resolution: --- → WONTFIX
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•