Merge lp:~shockone89/audience/universal-shortcuts into lp:~audience-members/audience/trunk

[Volodymyr Shatsky]

Finally shortcuts work with Russian layout.

[Volodymyr Shatsky (shockone89)]

Cody, I saw your question about this keyboard layouts thing on some mailing list the other day. Maybe this technique will be useful for your Pantheon Terminal.

[Artem Anufrij (artem-anufrij)]

I can't confirm this solution.

On my system (russian layout) CTRL+Q (as example) doesn't work.

review: Needs Fixing (functionality)

[Zisu Andrei (matzipan)]

Rejecting since bug #1235489 has been marked as fixed.

review: Disapprove

Unmerged revisions

251. By Volodymyr Shatsky on 2013-04-07

Shortcuts work with all keyboard layouts

=== modified file 'Audience/Audience.vala'
--- Audience/Audience.vala	2013-04-02 16:46:22 +0000
+++ Audience/Audience.vala	2013-04-07 04:41:18 +0000
@@ -218,7 +218,11 @@
             
             //shortcuts
             this.mainwindow.key_press_event.connect ( (e) => {
-                switch (e.keyval) {
+                //Covert the key to the English layout
+                Gdk.KeymapKey keymap_key = Gdk.KeymapKey () {keycode = e.hardware_keycode};
+                uint keyval = Gdk.Keymap.get_default ().lookup_key (keymap_key);
+
+                switch (keyval) {
                     case Gdk.Key.p:
                     case Gdk.Key.space:
                         video_player.playing = !video_player.playing;