D5695: Applet: Automatically raise maximum volume when over defined maximum volume

David Rosca noreply at phabricator.kde.org
Sat May 6 13:27:42 UTC 2017


drosca added inline comments.

INLINE COMMENTS

> sebas wrote in ListItemBase.qml:282
> Should be "volume", since that's the property. Or perhaps we don't need "volume" or "Volume".

"Volume" is a role from the model, so assigning to the "volume" property instead only makes it go through one indirection level. The "volume" property should be made readonly as it is there only for property changes.

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  auto-raise-volume (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D5695

To: drosca, #plasma, sebas
Cc: sebas, Zren, plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170506/007458e2/attachment.html>


More information about the Plasma-devel mailing list