Review Request 119163: Add basic accessibility properties
Aleix Pol Gonzalez
aleixpol at kde.org
Mon Jul 7 23:26:59 UTC 2014
> On July 7, 2014, 6:27 p.m., David Edmundson wrote:
> > src/declarativeimports/plasmacomponents/qml/ProgressBar.qml, line 140
> > <https://git.reviewboard.kde.org/r/119163/diff/1/?file=288432#file288432line140>
> >
> > This probably needs
> >
> > Accessible.name: value
> >
> > otherwise a blind person can see it's a progress bar but no idea what the progress is.
>
> Frederik Gladhorn wrote:
> Actually for a progress bar the screen reader already knows the type and we have code in QAccessibleQuickItem (qtdeclarative) that automatically checks for value, minimumValue, maximumValue properties. So I assume this already works by magic (I should probably add more documentation in Qt, but we were initially not 100% sure if we wanted to stick with this approach).
Well, I'd say this is find for the component. If the application developer wants to provide further information, she can.
- Aleix
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119163/#review61844
-----------------------------------------------------------
On July 7, 2014, 6:51 p.m., Frederik Gladhorn wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119163/
> -----------------------------------------------------------
>
> (Updated July 7, 2014, 6:51 p.m.)
>
>
> Review request for Plasma, Sebastian Sauer and Jeremy Whiting.
>
>
> Repository: plasma-framework
>
>
> Description
> -------
>
> Adds some accessibility properties to the plasma-framework controls. More work is needed, but these make many elements show up in accessibility helper tools.
>
>
> Diffs
> -----
>
> src/declarativeimports/plasmacomponents/qml/TabButton.qml 21302ef
> src/declarativeimports/plasmacomponents/qml/TextArea.qml 0ca6fe5
> src/declarativeimports/plasmacomponents/qml/TextField.qml d67dda3
> src/declarativeimports/plasmacomponents/qml/ToolBar.qml 91b8f77
> src/declarativeimports/plasmacomponents/qml/ToolButton.qml 21afc21
> src/declarativeimports/plasmacomponents/qml/private/DualStateButton.qml 8c6919e
> src/declarativeimports/plasmacomponents/qml/Button.qml 186a39a
> src/declarativeimports/plasmacomponents/qml/CommonDialog.qml 75f0362
> src/declarativeimports/plasmacomponents/qml/Label.qml 033c41c
> src/declarativeimports/plasmacomponents/qml/ListItem.qml 4e6d5e7
> src/declarativeimports/plasmacomponents/qml/ModelContextMenu.qml 74d81d1
> src/declarativeimports/plasmacomponents/qml/ProgressBar.qml b85c100
> src/declarativeimports/plasmacomponents/qml/RadioButton.qml 769682b
> src/declarativeimports/plasmacomponents/qml/ScrollBar.qml 6b6f9dd
> src/declarativeimports/plasmacomponents/qml/SectionScroller.qml 9c6cc2e
> src/declarativeimports/plasmacomponents/qml/Slider.qml 8441eb5
> src/declarativeimports/plasmacomponents/qml/Switch.qml 60f179d
> src/declarativeimports/plasmacomponents/qml/TabBar.qml 809318a
>
> Diff: https://git.reviewboard.kde.org/r/119163/diff/
>
>
> Testing
> -------
>
> http://blogs.fsfe.org/gladhorn/2014/07/07/plasma-next-accessibility/
>
>
> Thanks,
>
> Frederik Gladhorn
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140707/4ea62579/attachment.html>
More information about the Plasma-devel
mailing list