https://bugs.kde.org/show_bug.cgi?id=304696
Konstantin Kharlamov <hi-an...@yandex.ru> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
|ma/powerdevil/-/commit/1464 |ma/powerdevil/-/commit/dc4a
|7dc6ffd1104c1c0ab393518de8a |286037a802f3c18512e761b8908
|65cb39d70 |2740aa1e9
Version Fixed In|6.0 |5.27.8
--- Comment #22 from Konstantin Kharlamov <hi-an...@yandex.ru> ---
Git commit dc4a286037a802f3c18512e761b89082740aa1e9 by Konstantin Kharlamov.
Committed on 29/08/2023 at 01:42.
Pushed by ngraham into branch 'Plasma/5.27'.
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/dc4a286037a802f3c18512e761b89082740aa1e9
--
You are receiving this mail because:
You are the assignee for the bug.
