Merge lp:~themuso/ubiquity/a11y-improvements into lp:ubiquity

Proposed by Luke Yelavich
Status: Merged
Merge reported by: Luke Yelavich
Merged at revision: not available
Proposed branch: lp:~themuso/ubiquity/a11y-improvements
Merge into: lp:ubiquity
Diff against target: 67 lines (+22/-2)
3 files modified
debian/changelog (+13/-0)
gui/gtk/stepPartAdvanced.ui (+3/-0)
ubiquity/frontend/gtk_ui.py (+6/-2)
To merge this branch: bzr merge lp:~themuso/ubiquity/a11y-improvements
Reviewer Review Type Date Requested Status
Ubuntu Installer Team Pending
Review via email: mp+210119@code.launchpad.net

Description of the change

Some small accessibility related improvements.

To post a comment you must log in.
6133. By Luke Yelavich

Merge with trunk.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2014-03-12 16:53:09 +0000
3+++ debian/changelog 2014-03-12 22:22:58 +0000
4@@ -1,3 +1,16 @@
5+ubiquity (2.17.10) UNRELEASED; urgency=low
6+
7+ * Set up atk label association for grub boot loader location combo box in
8+ advanced partition editor.
9+ * Add Super + Alt + S as an activation keystroke for the screen reader
10+ profile. This mirrors the same keystroke that can be used to activate the
11+ screen reader when in a Unity or GNOME desktop. The Control + S keystroke
12+ is still present for transitional purposes, so new users don't need to
13+ hunt for new information when they attempt to install trusty. It will be
14+ removed in trusty+1.
15+
16+ -- Luke Yelavich <themuso@ubuntu.com> Thu, 13 Mar 2014 09:21:43 +1100
17+
18 ubiquity (2.17.9) trusty; urgency=medium
19
20 [ Matthew Paul Thomas ]
21
22=== modified file 'gui/gtk/stepPartAdvanced.ui'
23--- gui/gtk/stepPartAdvanced.ui 2013-06-24 10:08:31 +0000
24+++ gui/gtk/stepPartAdvanced.ui 2014-03-12 22:22:58 +0000
25@@ -931,6 +931,9 @@
26 <property name="xalign">0</property>
27 <property name="label" translatable="yes">Help for GRUB device selection goes here.</property>
28 <property name="wrap">True</property>
29+ <accessibility>
30+ <relation type="label-for" target="grub_device_entry"/>
31+ </accessibility>
32 </object>
33 <packing>
34 <property name="left_attach">0</property>
35
36=== modified file 'ubiquity/frontend/gtk_ui.py'
37--- ubiquity/frontend/gtk_ui.py 2014-03-06 06:42:00 +0000
38+++ ubiquity/frontend/gtk_ui.py 2014-03-12 22:22:58 +0000
39@@ -361,8 +361,6 @@
40 AppIndicator.IndicatorStatus.ACTIVE)
41 self.indicator.set_menu(
42 self.builder.get_object('a11y_indicator_menu'))
43- self.live_installer.connect(
44- 'key-press-event', self.a11y_profile_keys)
45 if osextras.find_on_path('canberra-gtk-play'):
46 subprocess.Popen(
47 ['canberra-gtk-play', '--id=system-ready'],
48@@ -370,6 +368,8 @@
49 except:
50 print("Unable to set up accessibility profile support",
51 file=sys.stderr)
52+ self.live_installer.connect(
53+ 'key-press-event', self.a11y_profile_keys)
54
55 def all_children(self, parent):
56 if isinstance(parent, Gtk.Container):
57@@ -668,6 +668,10 @@
58 elif (event.state & Gdk.ModifierType.CONTROL_MASK and
59 event.keyval == Gdk.keyval_from_name('s')):
60 self.a11y_profile_screen_reader_activate()
61+ elif (event.state & Gdk.ModifierType.SUPER_MASK and
62+ event.state & Gdk.ModifierType.MOD1_MASK and
63+ event.keyval == Gdk.keyval_from_name('s')):
64+ self.a11y_profile_screen_reader_activate()
65
66 def a11y_profile_high_contrast_activate(self, widget=None):
67 subprocess.call(['log-output', '-t', 'ubiquity',

Subscribers

People subscribed via source and target branches

to status/vote changes: