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 663623 - media-keys: Take over some shortcuts from metacity
media-keys: Take over some shortcuts from metacity
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: media-keys
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks: 663626
 
 
Reported: 2011-11-08 14:01 UTC by Florian Müllner
Modified: 2011-11-08 14:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
common: Allow to grab 'Print' without modifiers (1.03 KB, patch)
2011-11-08 14:01 UTC, Florian Müllner
committed Details | Review
media-keys: Move some metacity shortcuts into g-s-d (4.18 KB, patch)
2011-11-08 14:01 UTC, Florian Müllner
reviewed Details | Review
media-keys: Move some metacity shortcuts into g-s-d (3.27 KB, patch)
2011-11-08 14:21 UTC, Florian Müllner
committed Details | Review

Description Florian Müllner 2011-11-08 14:01:02 UTC
It's quite bizarre that "launch calculator" lives in g-s-d while "launch terminal" is handled by the window manager ...
Comment 1 Florian Müllner 2011-11-08 14:01:05 UTC
Created attachment 200983 [details] [review]
common: Allow to grab 'Print' without modifiers
Comment 2 Florian Müllner 2011-11-08 14:01:09 UTC
Created attachment 200984 [details] [review]
media-keys: Move some metacity shortcuts into g-s-d

The shortcuts for launching a terminal and taking screenshots have
been provided by the wm historically, but it makes more sense to
handle them like other global shortcuts in the media-keys plugin.
Comment 3 Bastien Nocera 2011-11-08 14:04:52 UTC
Review of attachment 200983 [details] [review]:

Looks correct (gnome-3-2 and master please)
Comment 4 Bastien Nocera 2011-11-08 14:06:38 UTC
Review of attachment 200984 [details] [review]:

Rest looks fine.

::: data/org.gnome.settings-daemon.plugins.media-keys.gschema.xml.in.in
@@ +106,3 @@
+      <_description>Binding to take a screenshot of a window.</_description>
+    </key>
+    </key>

We have the ability to launch arbitrary commands, do we really need a specific shortcut for launching a terminal? I'd say no.
Comment 5 Florian Müllner 2011-11-08 14:20:44 UTC
Comment on attachment 200983 [details] [review]
common: Allow to grab 'Print' without modifiers

Attachment 200983 [details] pushed as 5c4d989 - common: Allow to grab 'Print' without modifiers
Comment 6 Florian Müllner 2011-11-08 14:21:36 UTC
Created attachment 200989 [details] [review]
media-keys: Move some metacity shortcuts into g-s-d

(In reply to comment #4)
> We have the ability to launch arbitrary commands, do we really need a specific
> shortcut for launching a terminal? I'd say no.

Yeah, makes sense.
Comment 7 Florian Müllner 2011-11-08 14:55:29 UTC
Attachment 200989 [details] pushed as f32ae44 - media-keys: Move some metacity shortcuts into g-s-d

Patch OK'ed on IRC, pushing.