https://bugs.kde.org/show_bug.cgi?id=399646
Rover Bugfinder <rover.bugfin...@outlook.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |rover.bugfin...@outlook.com
--- Comment #4 from Rover Bugfinder <rover.bugfin...@outlook.com> ---
Hi.
Bug still present in KDE 5.78.0 / Powerdevil 5.20.5.
I have a laptop with just one video card, but with 2
"backends"/providers/drivers found by the system:
/sys/class/backlight/dell_backlight and /sys/class/backlight/intel_backlight
(yes, it's a Dell laptop with an Intel video card :) ).
As Bastian says, KDE picks whatever backend is returned first by scanning the
/sys/class/backlight directory. For me, it always seems to be the wrong one :).
I also found (although in some other place) the same piece of code that does
this.
I partially agree that KDE should check which backend is enabled and use that
one (as Dan says). Perhaps this will be enough. Unless there really are 2
graphic cards installed, both enabled, and the user would like to adjust the
brightness on all screens at once. Or maybe just some of them (e.g. because it
works on one, but breaks on the other).
That's why I agree more to a configurable setting - letting the user choose is,
in my opinion, always best.
Should it be a single-select or multi-select setting, should it be in ~/.config
and only there or also on the KDE's systemsettings GUI is another question.
I'd opt for a multi-select setting and a list of check-boxes in systemsettings
with all the found backends (found by listing the directory, just like it's
done now), letting the user select which ones to use and then make Powerdevil
iterate through the selected ones when the brightness action is called (button
press, power level change, whatever). If none selected, change none. Or just
the default. Or maybe introduce another setting - "Use default". If selected
(this would be the selected by default on fresh installations), use all the
found and enabled backends. When deselected (meaning: on purpose), use just the
ones the user selected (including "none"). Something like:
Choose the provider/backend/driver/graphic card for screen brightness:
(o) Use default (all enabled providers)
( ) Choose the provider:
[ ] dell
[x] intel
(2 radiobuttons, 2 check-boxes enabled only when the second radiobutton is
selected).
My C++ knowledge is too poor to fix this myself, and my Qt knowledge is zero,
not to mention making a decision which path to take, doing it and then
compiling half of KDE to do testing :). So, I'm just "keeping this bug alive"
to show that it's still relevant (and help make KDE better than Windows ;)
where the keyboard buttons to change the brightness do actually work on the
same laptop...).
--
You are receiving this mail because:
You are watching all bug changes.
