| Summary: | Screen lock doesn't ignore power button presses | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Martin Bednar <martin+kde> |
| Component: | Screen locking | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | 4wy78uwh, nate, niccolo |
| Priority: | NOR | ||
| Version First Reported In: | 6.3.90 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/-/commit/f1605df4a461874f39e6aef217397caab4d91e47 | Version Fixed/Implemented In: | 6.4.0 |
| Sentry Crash Report: | |||
|
Description
Martin Bednar
2025-05-20 16:54:50 UTC
Is there a particular reason you're pressing the power button? Do other keys to wake the screen not work? Regardless, can't reproduce on my hardware, though it has a power button that you need to hold down for 2+ seconds to trigger it. (In reply to Nate Graham from comment #1) > Is there a particular reason you're pressing the power button? not particularly: it's also the fingerprint reader, and sometimes my finger is a bit heavy :D > > Regardless, can't reproduce on my hardware, though it has a power button > that you need to hold down for 2+ seconds to trigger it. This might be tied to power management setting "When power button pressed" -> "Show logout screen" > This might be tied to power management setting "When power button pressed"
> -> "Show logout screen"
Yes: can't reproduce without having this set in the profile I'm currently on.
Yeah, I've got that setting turned on too. Would you like a recording? It's really minor though... Can reproduce. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5543 Git commit ba7a8c16b7934fa23d3a44bb38667554e39996b7 by Niccolò Venerandi. Committed on 02/06/2025 at 11:34. Pushed by niccolove into branch 'master'. Do not show logout screen when in lockscreen / screensaver M +12 -5 libkworkspace/sessionmanagement.cpp https://invent.kde.org/plasma/plasma-workspace/-/commit/ba7a8c16b7934fa23d3a44bb38667554e39996b7 Git commit f1605df4a461874f39e6aef217397caab4d91e47 by Niccolò Venerandi. Committed on 02/06/2025 at 11:35. Pushed by niccolove into branch 'Plasma/6.4'. Do not show logout screen when in lockscreen / screensaver (cherry picked from commit ba7a8c16b7934fa23d3a44bb38667554e39996b7) Co-authored-by: Niccolò Venerandi <niccolo@venerandi.com> M +12 -5 libkworkspace/sessionmanagement.cpp https://invent.kde.org/plasma/plasma-workspace/-/commit/f1605df4a461874f39e6aef217397caab4d91e47 Thank you! You guys rock. To confirm, this shan't affect those who have “When power button pressed” → “Power off”? I ask because I need the POWER_SW to always work, for the few times that the OS becomes inaccessible (via HID)s, like https://gitlab.freedesktop.org/drm/amd/-/issues/3462#screenshotsvideo-files. (In reply to Roke Julian Lockhart Beedell from comment #11) > To confirm, this shan't affect those who have “When power button pressed” → > “Power off”? I ask because I need the POWER_SW to always work, for the few > times that the OS becomes inaccessible (via HID)s, like > https://gitlab.freedesktop.org/drm/amd/-/issues/3462#screenshotsvideo-files. Looking at the code, the check only happens after the logout prompt has been requested. So my understanding is that it shouldn't. I just tried setting the power button action to suspend, and using plasma 6.3.91 (which AFAIK contains the change) it did suspend when pressed in the lock screen. |