Bug 467910 - Keyboard brightness controls turn off backlight completely when reaching 0%
Summary: Keyboard brightness controls turn off backlight completely when reaching 0%
Status: RESOLVED DUPLICATE of bug 430439
Alias: None
Product: Powerdevil
Classification: Plasma
Component: general (show other bugs)
Version: 5.26.5
Platform: Manjaro Linux
: NOR minor
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-03-29 01:50 UTC by Yeshi
Modified: 2023-08-13 19:10 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yeshi 2023-03-29 01:50:37 UTC
Changing the backlight to 0% turns the backlight off entirely, rather than setting a low brightness amount.
However, changing the brightness to 0% through the battery widget does not have the same problem.
I would suggest keeping this bugged behavior as a setting somewhere, as it might be nice to have in some cases.

SOFTWARE/OS VERSIONS
Kernel: 6.1.19-1-MANJARO
KDE Plasma Version: 5.26.5
KDE Frameworks Version: 5.103.0
Qt Version: 5.15.8
Comment 1 Nate Graham 2023-04-03 22:25:06 UTC
This is an Intel GPU driver bug which is fixed in the latest version of Mesa and also worked around in Plasma 5.27.
Comment 2 Nate Graham 2023-08-13 17:35:07 UTC
Git commit e1e74f37ea6ba4d3450ae79ede25a7cac1dd9a84 by Nate Graham.
Committed on 13/08/2023 at 19:29.
Pushed by ngraham into branch 'master'.

Make minimum screen brightness not always be 0

With some drivers, a value of 0 disables the screen backlight
completely, which is almost certainly not what the user wanted. The
Battery & Brightness widget has logic to handle this and instead make
1 the minimum brightness for screens. But Powerdevil does not, and as a
result, the behaviors differ between adjusting the brightness using the
widget and using the keyboard's "brightness down" key. This is
inconsistent and weird.

This commit unifies the behaviors by adding logic to Powerdevil to match
what the widget does.

Arguably this is working around a driver bug, but insisting that all
graphics driver bugs be fixed is not practical due to their diversity
and frequent regressions. And the Battery & Brightness widget already
has such a workaround to prevent users whose graphics drivers don't
work as they should from experiencing this issue, so not doing the same
here is inconsistent.
FIXED-IN: 6.0

M  +4    -4    daemon/powerdevilbrightnesslogic.cpp
M  +14   -0    daemon/powerdevilbrightnesslogic.h
M  +6    -0    daemon/powerdevilscreenbrightnesslogic.cpp
M  +1    -0    daemon/powerdevilscreenbrightnesslogic.h

https://invent.kde.org/plasma/powerdevil/-/commit/e1e74f37ea6ba4d3450ae79ede25a7cac1dd9a84
Comment 3 Nate Graham 2023-08-13 19:10:11 UTC

*** This bug has been marked as a duplicate of bug 430439 ***