[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [plasmashell] [Bug 356132] Starting KDE causes with power widget causes screen brightness to go to 0
From:       David Edmundson via KDE Bugzilla <bugzilla_noreply () kde ! org>
Date:       2015-12-04 0:36:10
Message-ID: bug-356132-17878-E91n6nSEK8 () http ! bugs ! kde ! org/
[Download RAW message or body]

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

David Edmundson <kde@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kde@davidedmundson.co.uk

--- Comment #2 from David Edmundson <kde@davidedmundson.co.uk> ---
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.
[prev in list] [next in list] [prev in thread] [next in thread]