https://bugs.kde.org/show_bug.cgi?id=523281
cwo <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas |ma/powerdevil/-/commit/57f1 |ma/powerdevil/-/commit/97d9 |904599655bb77de7e5089a5894d |a96bc7a9e8fb21ca4c8d8ea6723 |5930bbaf1 |11f463167 --- Comment #11 from cwo <[email protected]> --- Git commit 97d9a96bc7a9e8fb21ca4c8d8ea672311f463167 by Christoph Wolk. Committed on 17/09/2026 at 12:02. Pushed by cwo into branch 'Plasma/6.7'. applets/brightness: fix Slider binding issues 94a03a3c04787388026a02ff40afd2b77fc9d0f6 introduced a workaround for devices that send intermediate updates on brightness changes that interfere with interactively dragging the slider: it binds the value only while the control is not pressed. But this causes two issues: - Keyboard interaction also makes the control pressed, which stops the Binding and causes it to restore the pre-binding value, 0, before applying the change. So e.g. Arrow Left will always reset the slider to 0, rather than one step down, the first time each Plasma run. - Setting the value delegate's property manually breaks the binding we set from outside, so further changes from elsewhere (such as global shortcuts) will no longer be reflected in the slider, it will stay at whatever value it was set to in the widget. (Except for keyboard brightness, which has some extra logic to constantly reestablish the binding) To fix this, we make the binding between the delegate more explicit, setting it to the current value when pressed and a binding while not. We also change how we communicate changes to the outside; rather than having a binding that breaks and has to be established constantly, we keep the delegate's value property always bound and communicate user input by passing the value as an argument to the signal that causes the change to happen anyway. This solves both issues and reduces code complexity as things are better encapsulated (albeit at the cost of having a bit of imperative state handling in the Slider) Manual cherry-pick from 57f190459 M +12 -13 applets/brightness/BrightnessItem.qml M +2 -10 applets/brightness/PopupDialog.qml https://invent.kde.org/plasma/powerdevil/-/commit/97d9a96bc7a9e8fb21ca4c8d8ea672311f463167 -- You are receiving this mail because: You are watching all bug changes.
