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 636383 - tracker-status-icon grabs control+alt+S keybinding unconditionally which causes issues for emacs
tracker-status-icon grabs control+alt+S keybinding unconditionally which caus...
Status: RESOLVED FIXED
Product: tracker
Classification: Core
Component: Applet / Search Bar
git master
Other Linux
: Normal normal
: ---
Assigned To: tracker-applet
Jamie McCracken
Depends on:
Blocks:
 
 
Reported: 2010-12-03 16:13 UTC by James Hunt
Modified: 2010-12-06 12:29 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
possible git patch (1.23 KB, patch)
2010-12-03 16:13 UTC, James Hunt 		committed Details | Review

Description James Hunt 2010-12-03 16:13:44 UTC 
Created attachment 175791 [details] [review]
possible git patch

As reported in Ubuntu bug 672614...

  https://bugs.launchpad.net/ubuntu/+source/tracker/+bug/672614

... tracker-status-icon grabs the control+alt+S shortcut unconditionally. This causes problems for emacs users since this keybinding stops you running isearch-forward-regexp within emacs (a not uncommon requirement) and instead pops up tracker-search-tool.

The simplest fix would seem to be as proposed by Andrei Dziahel on the bug above - simply remove the call to set_global_keybinding(). This way, the user or gnome packager is free to configure an appropriate keybinding to start tracker-search-tool from gnome-keybinding-properties. The default of course could be made to be the same as the existing hard-coded keybinding.

(Attached patch ignores fact that removing this call makes src/tracker-status-icon/tomboykeybinder.c largely redundant).
Comment 1 Martyn Russell 2010-12-06 12:29:03 UTC 
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.
Comment 2 Martyn Russell 2010-12-06 12:29:32 UTC 
Comment on attachment 175791 [details] [review]
possible git patch

I didn't remove it, but #if 0 it out, we can put it back in later if we ever manage to make this configurable