After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 786402 - Combo types list doesn't close on esc
Combo types list doesn't close on esc
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: File Search Interface
3.24.x
Other Linux
: Normal normal
: 3.24
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-08-17 08:34 UTC by Sebastien Bacher
Modified: 2017-08-17 13:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
search-popover: make type list box keyboard-navigable (2.67 KB, patch)
2017-08-17 09:32 UTC, Ernestas Kulik
committed Details | Review

Description Sebastien Bacher 2017-08-17 08:34:30 UTC
Using nautilus 3.24.2.1
- click on the search icon
- click on the down arrow from the combo
- open the "what" combo list
- hit "esc" on the keyboard

the combo list should close but it doesn't
Comment 1 Carlos Soriano 2017-08-17 08:36:18 UTC
The problem is that the focus is not in the popover.
Comment 2 Ernestas Kulik 2017-08-17 08:38:18 UTC
(In reply to Carlos Soriano from comment #1)
> The problem is that the focus is not in the popover.

Judging from the description, the problem is rather that the revealer does not hide the child when ESC is pressed, not that the popover doesn’t go away.

If the focus was on the popover, the popover itself would go away, no?
Comment 3 Ernestas Kulik 2017-08-17 08:39:24 UTC
(In reply to Ernestas Kulik from comment #2)
> (In reply to Carlos Soriano from comment #1)
> > The problem is that the focus is not in the popover.
> 
> Judging from the description, the problem is rather that the revealer does
> not hide the child when ESC is pressed, not that the popover doesn’t go away.
> 
> If the focus was on the popover, the popover itself would go away, no?

Well, it’s not just a revealer, but you get it.
Comment 4 Carlos Soriano 2017-08-17 08:48:31 UTC
Ah! I see.
But one starting problem is that the focus is not there, so it's not keyboard navigable at all, and that's more serious and could be the root problem.
Comment 5 Ernestas Kulik 2017-08-17 09:32:29 UTC
Created attachment 357783 [details] [review]
search-popover: make type list box keyboard-navigable

Currently, the type list box in the search popover is not
keyboard-navigable, which, as a consequence, results in the user’s
inability to also easily close the popover.
Comment 6 Ernestas Kulik 2017-08-17 09:33:29 UTC
With the patch, the whole popover goes away on ESC being pressed, is that good enough, Sebastien?
Comment 7 Carlos Soriano 2017-08-17 10:16:16 UTC
Review of attachment 357783 [details] [review]:

Excellent, thanks!
Comment 8 Carlos Soriano 2017-08-17 13:57:32 UTC
Attachment 357783 [details] pushed as 6e2379e - search-popover: make type list box keyboard-navigable