https://bugs.kde.org/show_bug.cgi?id=356132

David Edmundson <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #2 from David Edmundson <[email protected]> ---
That patch doesn't make a lot of sense.

but the user of BrightnessItem (PopupDialog.qml) *also* sets a value:

        BrightnessItem {
            id: brightnessSlider
            value: batterymonitor.screenBrightness
            ...
       }

so you'd set the value of 1, then this would override it before you've started
doing anything.

Currently we set it to undefined which isn't very good code tbh, but it won't
do anything as we only connect the valueChanged signal on the slider after
everything is loaded. We then call updateBrightness in Logic.js that updates
the slider's initial value.

That initial value comes from powerdevil, I think you want to be looking there
and this slider is a red herring.
I think you're looking very much in the wrong place; the slider won't do

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to