applet: Support RTL layouts in volume slider
Extracted code into VolumeSlider component of its own, and synced it from a recent PC3.Slider rewrite which implemented better RTL support. Removed double-property hack with ignoreValueChange backing flag, because in modern Qt/QML Slider::moved signal should be used to react to user-driven changes while preserving the binding on `value` property. Such binding still needs to be reset after user interactions, so there's a simple rebinding on release in onPressedChanged handler. Timer is no longer needed, and slider still works just fine without it. See also: https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/585
-
Developer
Very good - separating out the volume slider definitely makes the applet item an easier read, and is what the KCM does already.
-
Author Developer
There's a separate implementation in KCM? Wait, OH SH~
-
Developer
Don't get me started! Because it's the QQC slider, you can't use stepSize without getting ugly tickmarks, so there's no scrolling on it and it doesn't snap to 1% values properly (worked around in master/5.26)
Also it doesn't show the current audio level like the applet...
-
Developer
There's a separate implementation in KCM? Wait, OH SH~
No, because it uses desktop style it't not as easy implementable as in the applet
Please register or sign in to comment