Merge lp:~themuso/unity-greeter/switch-screen-reader-key into lp:unity-greeter

Proposed by Luke Yelavich
Status: Merged
Approved by: Robert Ancell
Approved revision: 1340
Merged at revision: 1343
Proposed branch: lp:~themuso/unity-greeter/switch-screen-reader-key
Merge into: lp:unity-greeter
Diff against target: 12 lines (+1/-1)
1 file modified
src/menubar.vala (+1/-1)
To merge this branch: bzr merge lp:~themuso/unity-greeter/switch-screen-reader-key
Reviewer Review Type Date Requested Status
Robert Ancell Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+231671@code.launchpad.net

Commit message

Switch the screen reader keyboard shortcut to be the same as the default shortcut used in the desktop session.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Robert Ancell (robert-ancell) :
review: Approve
Revision history for this message
Robert Ancell (robert-ancell) wrote :

On IRC Luke confirmed the session key had changed so this just matches that.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/menubar.vala'
2--- src/menubar.vala 2014-04-08 03:48:16 +0000
3+++ src/menubar.vala 2014-08-21 05:17:56 +0000
4@@ -230,7 +230,7 @@
5 high_contrast_item.set_active (UGSettings.get_boolean (UGSettings.KEY_HIGH_CONTRAST));
6 var item = new Gtk.CheckMenuItem.with_label (_("Screen Reader"));
7 item.toggled.connect (screen_reader_toggled_cb);
8- item.add_accelerator ("activate", accel_group, Gdk.Key.s, Gdk.ModifierType.CONTROL_MASK, Gtk.AccelFlags.VISIBLE);
9+ item.add_accelerator ("activate", accel_group, Gdk.Key.s, Gdk.ModifierType.SUPER_MASK | Gdk.ModifierType.MOD1_MASK, Gtk.AccelFlags.VISIBLE);
10 item.show ();
11 submenu.append (item);
12 item.set_active (UGSettings.get_boolean (UGSettings.KEY_SCREEN_READER));

Subscribers

People subscribed via source and target branches