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.

Reply via email to