| Summary: | Turn off keyboard backlight the same way as done for the display when using escape key | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Porkepix <porkepix> |
| Component: | Screen locking | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | CONFIRMED --- | ||
| Severity: | wishlist | CC: | ilikefoss, jpetso, nate |
| Priority: | NOR | ||
| Version First Reported In: | 6.2.4 | ||
| Target Milestone: | 1.0 | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| See Also: |
https://bugs.kde.org/show_bug.cgi?id=466344 https://bugs.kde.org/show_bug.cgi?id=466023 |
||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
Porkepix
2023-02-24 11:15:40 UTC
Seems like a sensible plan to me. Since the idea is to save power and reduce unnecessary light, the keyboard backlight should be included. Still an issue on Plasma 6.0.4. The main challenge is that the code for turning off the display on "Esc" doesn't go through PowerDevil (which generally handles all of the keyboard backlight stuff) and instead goes to X11 or KWin/Wayland directly, via KScreen::Dpms. So in order for PowerDevil to react to this, we need PowerDevil's own KScreen::Dpms instance to observe the current DPMS state of the (primary?) display and adjust keyboard brightness based on that. I don't know if KWin's DPMS protocol (https://wayland.app/protocols/kde-dpms) will update other processes when the DPMS state changes. Would be worth exploring though, and we could use the opportunity to further decouple display vs. keyboard backlight code. *** Bug 501463 has been marked as a duplicate of this bug. *** Still an issue on Plasma 6.5.0! |