Accessibility in lock screen UI
ClosedPublic

Authored by davidedmundson on Sep 12 2017, 8:44 PM.

Details

Summary

Add more explicit text to labels (available via "flat review" keys)
Add names to media controls

Test Plan

Ran lockscreen with orca running

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Sep 12 2017, 8:44 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 12 2017, 8:44 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik added inline comments.
lookandfeel/contents/lockscreen/MediaControls.qml
148

Text depending on state? ie. Play media or Pause media?

davidedmundson added inline comments.Sep 12 2017, 8:51 PM
lookandfeel/contents/lockscreen/MediaControls.qml
148

It should, but there's a Qt (I think) bug which means dynamic updates don't get propagated to the screen reader correctly and it will just read out the initial version forever.

Working on that next.

broulik accepted this revision.Sep 12 2017, 8:51 PM
This revision is now accepted and ready to land.Sep 12 2017, 8:51 PM
This revision was automatically updated to reflect the committed changes.