Review Request 122540: Add screen reader to Accessibility KCM

[Frederik Gladhorn]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122540/
-----------------------------------------------------------

(Updated Feb. 12, 2015, 6:12 p.m.)


Review request for Plasma, Jonathan Riddell, Sebastian Sauer, and Jeremy Whiting.


Repository: plasma-desktop


Description
-------

This patch adds a simple check box to the KCM which enables the screen
reader.
It additionally sets up the default shortcut - Meta-Alt-S which is the
same as in Gnome. Using the same shortcut is of course extremely
important, since blind users rely on an easy way to activeate the screen
reader.
    
For now the whole process is hard-coded to Orca since there are no real
alternatives for Linux screen readers at the moment.
    
The gconfig call which disables the screen reader will make Orca exit.
It still needs to be launched manually.

I don't have much time for cleaning up the KCM, so help would be very much appreciated.
There are a lot of low hanging fruits for cleanup there. Also the port to kf5, away from deprecated stuff is by far not complete.


Diffs (updated)
-----

  kcms/access/kaccess.cpp 2419efb 
  kcms/access/kcmaccess.cpp 7e0217d 
  kcms/access/CMakeLists.txt 55990d2 
  kcms/access/accessibility.ui 2e0db83 
  kcms/access/kaccess.h e101de4 

Diff: https://git.reviewboard.kde.org/r/122540/diff/


Testing
-------

This allows me to start and stop Orca using meta-alt-s or the kcm.


Thanks,

Frederik Gladhorn

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150212/4a7975f5/attachment.html>