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 634103 - Quit from app menu doesn't work
Quit from app menu doesn't work
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2010-11-05 17:34 UTC by William Jon McCann
Modified: 2010-11-05 19:27 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
app-menu: Fix quit menu item (1022 bytes, patch)
2010-11-05 17:57 UTC, Florian Müllner 		committed Details | Review

Description William Jon McCann 2010-11-05 17:34:58 UTC 
In _onQuit()

The following condition is hit:
if (this._focusedApp == null)
            return;

Perhaps the menu itself is stealing focus (keyboard nav?)?
Comment 1 Florian Müllner 2010-11-05 17:57:06 UTC 
Created attachment 173899 [details] [review]
app-menu: Fix quit menu item

As popup menus now take focus for keyboard navigation, no application
is focused when activating the menu. Use the target application instead,
which keeps track of the application currently associated with the menu.
Comment 2 Dan Winship 2010-11-05 18:14:22 UTC 
Comment on attachment 173899 [details] [review]
app-menu: Fix quit menu item

yeah, that should work
Comment 3 Florian Müllner 2010-11-05 19:27:30 UTC 
Attachment 173899 [details] pushed as 34aa46a - app-menu: Fix quit menu item