Merge lp:~mterry/unity-greeter/screen-reader into lp:unity-greeter

Proposed by Michael Terry
Status: Merged
Merged at revision: 243
Proposed branch: lp:~mterry/unity-greeter/screen-reader
Merge into: lp:unity-greeter
Diff against target: 40 lines (+19/-4)
1 file modified
src/user-list.vala (+19/-4)
To merge this branch: bzr merge lp:~mterry/unity-greeter/screen-reader
Reviewer Review Type Date Requested Status
Unity Greeter Development Team Pending
Review via email: mp+87911@code.launchpad.net

Description of the change

Launch orca when users asks for it (without any windows popping up)

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Committed with brace and comment style fixes

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/user-list.vala'
--- src/user-list.vala 2011-12-09 18:06:18 +0000
+++ src/user-list.vala 2012-01-09 10:14:23 +0000
@@ -569,6 +569,12 @@
569569
570 private void keyboard_toggled_cb (Gtk.CheckMenuItem item)570 private void keyboard_toggled_cb (Gtk.CheckMenuItem item)
571 {571 {
572 // FIXME: The below would be sufficient if gnome-session were running
573 // to notice and run a screen keyboard in /etc/xdg/autostart... But
574 // since we're not running gnome-session, we hardcode onboard here.
575 //var settings = new Settings ("org.gnome.desktop.a11y.applications");
576 //settings.set_boolean ("screen-keyboard-enabled", item.active);
577
572 if (keyboard_window == null)578 if (keyboard_window == null)
573 {579 {
574 int id;580 int id;
@@ -646,10 +652,19 @@
646652
647 private void screen_reader_toggled_cb (Gtk.CheckMenuItem item)653 private void screen_reader_toggled_cb (Gtk.CheckMenuItem item)
648 {654 {
649 var settings = new Settings ("org.gnome.desktop.a11y.applications");655 // FIXME: The below would be sufficient if gnome-session were running
650 settings.set_boolean ("screen-reader-enabled", item.active);656 // to notice and run a screen reader in /etc/xdg/autostart... But
651 settings = new Settings ("org.gnome.desktop.interface");657 // since we're not running gnome-session, we hardcode orca here.
652 settings.set_boolean ("toolkit-accessibility", item.active);658 //var settings = new Settings ("org.gnome.desktop.a11y.applications");
659 //settings.set_boolean ("screen-reader-enabled", item.active);
660
661 // Hardcoded orca:
662 if (item.active) {
663 Process.spawn_command_line_async ("orca --replace --no-setup --disable=splash-window --disable=main-window");
664 }
665 else {
666 Process.spawn_command_line_async ("orca --quit");
667 }
653 }668 }
654669
655 public void show_message (string text, bool error = false)670 public void show_message (string text, bool error = false)

Subscribers

People subscribed via source and target branches