Review Request 125622: Use better guard against recursion when changing volume with sliders
David Rosca
nowrep at gmail.com
Tue Oct 13 15:21:27 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125622/
-----------------------------------------------------------
Review request for Plasma.
Repository: plasma-pa
Description
-------
As noted in https://git.reviewboard.kde.org/r/125088/, this should fix acessibility on volume sliders.
This also fixes changing the volume with keyboard (when slider has focus).
Diffs
-----
applet/contents/ui/ListItemBase.qml 54f6eb0
src/kcm/package/contents/ui/VolumeSlider.qml da77315
Diff: https://git.reviewboard.kde.org/r/125622/diff/
Testing
-------
Changing volume in applet still works + it fixes changing volume in kcm (i broke it with last commit - there is no setVolume() in kcm VolumeSlider).
Thanks,
David Rosca
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151013/5237bee7/attachment.html>
More information about the Plasma-devel
mailing list