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 636371 - alt-f1 opens but does not close the overview
alt-f1 opens but does not close the overview
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
2.91.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2010-12-03 14:01 UTC by Adam Williamson
Modified: 2011-01-10 20:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
main: allow Alt-F1 to exit the overview (1.51 KB, patch)
2011-01-06 20:34 UTC, Dan Winship
committed Details | Review

Description Adam Williamson 2010-12-03 14:01:29 UTC
alt-f1 works to open the overview, but not to close it. Alessandro Crismani notes that Super key works both to open and close it, so I'm pretty sure this is a bug. (Obviously, the use case for this is a keyboard with no Super key, like my 1991 model :>)
Comment 1 Dan Winship 2011-01-06 20:34:48 UTC
Created attachment 177699 [details] [review]
main: allow Alt-F1 to exit the overview

Also, change _globalKeyPressHandler to handle KEY_PRESS, not
KEY_RELEASE, for consistency with other code (and so that the
combination of an Alt-F1 press and release doesn't first enter the
overview and then immediately exit it).
Comment 2 Owen Taylor 2011-01-07 21:13:42 UTC
Review of attachment 177699 [details] [review]:

Hmm, this makes Super inconsistent for going into the overview and leaving the overview - it's done on release when going into the overview to allow for future handling of Super+<key> bindings. Guess that's OK, can revisit if we want Super+<key> bindings to be active in the overview.
Comment 3 Colin Walters 2011-01-07 21:33:30 UTC
Review of attachment 177699 [details] [review]:

Looks good to me; minor nit: would have been nice to explicitly mention you'd looked at the other key press handlers and checked they wouldn't break with the keydown->keyup change (I assume you did, and I just looked too).
Comment 4 Dan Winship 2011-01-10 20:28:06 UTC
Attachment 177699 [details] pushed as ed83b54 - main: allow Alt-F1 to exit the overview