Review Request 125622: Use better guard against recursion when changing volume with sliders

David Rosca nowrep at gmail.com
Fri Oct 30 11:54:39 UTC 2015


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

(Updated Oct. 30, 2015, 5:54 a.m.)


Status
------

This change has been marked as submitted.


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/20151030/aa650075/attachment.html>


More information about the Plasma-devel mailing list