https://bugs.kde.org/show_bug.cgi?id=304696
Nate Graham <n...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
Version Fixed In| |5.27.8
Latest Commit| |https://invent.kde.org/plas
| |ma/powerdevil/-/commit/1464
| |7dc6ffd1104c1c0ab393518de8a
| |65cb39d70
--- Comment #21 from Nate Graham <n...@kde.org> ---
Git commit 14647dc6ffd1104c1c0ab393518de8a65cb39d70 by Nate Graham, on behalf
of Konstantin Kharlamov.
Committed on 25/08/2023 at 16:46.
Pushed by ngraham into branch 'master'.
dimdisplay: only dim the screen at configured dim time
The current logic is completely broken:
1. It disables the screen at `m_dimOnIdleTime` even though disabling
the screen is handled elsewhere with a separate timeout, and the logic
here has prefix "Dim*" so has nothing to do with disabling it.
2. It does not follow the timeout configured by a user (that is the
`m_dimOnIdleTime`) and instead dims the screen at completely
arbitrary 50% and 75% periods of the configured time.
Let's get rid of all that and only do exactly what the user asked: dim
the screen at `m_dimOnIdleTime`.
FIXED-IN: 5.27.8
M +10 -14 daemon/actions/bundled/dimdisplay.cpp
https://invent.kde.org/plasma/powerdevil/-/commit/14647dc6ffd1104c1c0ab393518de8a65cb39d70
--
You are receiving this mail because:
You are the assignee for the bug.
