https://bugs.kde.org/show_bug.cgi?id=482306

--- Comment #13 from Jakob Petsovits <jpe...@petsovits.com> ---
(In reply to Anthony Vital from comment #12)
> Here's a small patch that fixes the issue against the 6.0 branch (I have no
> easy way to try the master branch, but it should be the same).
> So the issue seems to be that after the lid is closed and the brightness set
> to 0, powerdevil receives a signal keyboardBrightnessChanged, and saves this
> value for restoration. This patch ensures we store this value only if the
> lid is open.
> This check existed in the 5.27 branch, I'm not sure why it was removed. 

Thanks for the patch, great find. I tested this on my ThinkPad, latest master
branch, and it indeed makes a difference there as well.

I did a little bit of digging in the commit history, my reading is that the
removal of this condition was probably unintentional. It got removed together
with another condition in the same line that did indeed have to disappear due
to a refactoring. More details and perhaps also a confirmation of this theory
in the merge request that will get linked by the bot in the next comment.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to