Ticket #454 (closed defect: duplicate)

Opened 12 months ago

Last modified 5 weeks ago

Shortcut with Ctrl doesn't work on vanilla 12.04 Ubuntu

Reported by: mkorenkov Owned by: somebody
Priority: Fix available Milestone:
Component: guake Version: 0.4
Keywords: shortcut, ubuntu Cc:

Description

I have just installed Guake on a new Ubuntu 12.04. I have modified settings through Guake preferences app and changed shortcut to use Ctrl + <KEY>.

Unfortunately, Ctrl + <KEY> shortkey doesn't work. Guake triggers on <KEY> instead of expected Ctrl + <KEY>.

These links below say that not only I have noticed this problem

*  http://askubuntu.com/questions/142034/ctrl-not-working-with-guake *  http://translate.google.com/translate?sl=auto&tl=en&js=n&prev=_t&hl=en&ie=UTF-8&layout=2&eotf=1&u=http%3A%2F%2Fforum.ubuntu.ru%2Findex.php%3Ftopic%3D192409.0&act=url

Attachments

27.png Download (9.7 KB) - added by mkorenkov 12 months ago.
fix_ctrl_global.patch Download (1.5 KB) - added by devoidfury 11 months ago.
globalhotkeys.so Download (89.7 KB) - added by Fra 10 months ago.
globalhotkeys.32.so Download (74.5 KB) - added by rinsukaze 7 months ago.
Library for 32 bit

Change History

Changed 12 months ago by mkorenkov

  Changed 11 months ago by bibstha

Ctrl does not work for "Toggle Guake Visibility" but works for "Toggle Fullscreen". I tried using Ctrl+ for Visibility. However it works without Ctrl, just with the .

This also appeared in Archlinux.

follow-up: ↓ 4   Changed 11 months ago by devoidfury

GTK+ version 3.4.3 changed the string representation of Ctrl to <Primary>.

It's in the release notes, see here.  http://hpux.connect.org.uk/hppd/hpux/Gtk/Development/gtk+3-3.4.3/readme.html

I've written a fix (globalhotkeys/eggaccelerators.c), but I'm not familiar with the process of submitting it. I'll see if I can figure it out.

Changed 11 months ago by devoidfury

  Changed 11 months ago by devoidfury

  • priority changed from major to Fix available

in reply to: ↑ 2 ; follow-up: ↓ 5   Changed 10 months ago by Fra

Replying to devoidfury:

GTK+ version 3.4.3 changed the string representation of Ctrl to <Primary>. It's in the release notes, see here.  http://hpux.connect.org.uk/hppd/hpux/Gtk/Development/gtk+3-3.4.3/readme.html I've written a fix (globalhotkeys/eggaccelerators.c), but I'm not familiar with the process of submitting it. I'll see if I can figure it out.

Nice to see there is a fix. How can I apply it ? and how long will take this fix to be included in next release ?

in reply to: ↑ 4   Changed 10 months ago by Fra

Replying to Fra:

Replying to devoidfury:

GTK+ version 3.4.3 changed the string representation of Ctrl to <Primary>. It's in the release notes, see here.  http://hpux.connect.org.uk/hppd/hpux/Gtk/Development/gtk+3-3.4.3/readme.html I've written a fix (globalhotkeys/eggaccelerators.c), but I'm not familiar with the process of submitting it. I'll see if I can figure it out.

Nice to see there is a fix. How can I apply it ? and how long will take this fix to be included in next release ?

I have download 0.4.4 code and compiled (bug is still there) So I have applied patch provided by @devoidfury and now CTRL bug is gone.

Changed 10 months ago by Fra

  Changed 10 months ago by Fra

I have attached /globalhotkeys.so file compiled on ubuntu 12.04-64BIT.

In case you are bored to wait for official fix that implement above patch but are lazy to compile it ...

I have managed that if you override this file with the one provided with ubuntu it fix the problem ( /usr/lib/guake/globalhotkeys.so )

Changed 7 months ago by rinsukaze

Library for 32 bit

  Changed 7 months ago by rinsukaze

Attached globalhotkeys.32.so compiled on ubuntu 12.04 32 bit. Just change the name to globalhotkeys.so and copy it to /usr/lib/guake/globalhotkeys.so

  Changed 5 weeks ago by Dokan

  • status changed from new to closed
  • resolution set to duplicate
Note: See TracTickets for help on using tickets.