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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.25
             Status|ASSIGNED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/powerdevil/commit/6220db
                   |                            |1b8a5038ed5e1359b1afdf70c06
                   |                            |25f0895

--- Comment #10 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 6220db1b8a5038ed5e1359b1afdf70c0625f0895 by David Edmundson, on
behalf of Louis Moureaux.
Committed on 01/09/2022 at 18:52.
Pushed by ngraham into branch 'master'.

Fix profile switching in the brightness actions

The current code assumes that the action is notified of every profile change,
which is only the case when the action is configured to run in all profiles.
The code that prevents brightness from being raised when switching to a more
conservative profile gets confused if the action didn't run in the "previous"
profile. Make it more robust by always querying the core for the previous
profile, which it luckily hasn't updated yet when the action gets executed.
FIXED-IN: 5.25

M  +8    -7    daemon/actions/bundled/brightnesscontrol.cpp
M  +0    -1    daemon/actions/bundled/brightnesscontrol.h
M  +9    -7    daemon/actions/bundled/keyboardbrightnesscontrol.cpp
M  +0    -1    daemon/actions/bundled/keyboardbrightnesscontrol.h

https://invent.kde.org/plasma/powerdevil/commit/6220db1b8a5038ed5e1359b1afdf70c0625f0895

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

Reply via email to