476533 2023-11-04 02:39:37 +0000 [kdesrc-build Plasma 6] Pressing Esc no longer turns off the screen when locked 2024-01-24 20:19:30 +0000 1 1 10 Unmaintained kscreenlocker general git-master Compiled Sources Linux RESOLVED FIXED qt6, regression NOR normal --- 1 putr4.s plasma-bugs-null bharadwaj.raju777 nate https://invent.kde.org/plasma/kscreenlocker/-/commit/0b889fee8cf5080ea21fa5e90d32379b26d58de6 6.0 0 oldest_to_newest 2263432 0 putr4.s 2023-11-04 02:39:37 +0000 SUMMARY With Plasma 6 set up using kdesrc-build and after getting around bug 476508 I found that pressing Esc does not turn off the screen when locked, and if the password field is on screen pressing Esc makes it act like I just pressed Enter, it says "Unlocking failed", shakes the password field and then hides said field. This is different from bug 466344 in that I don't get the issue in the distro installed Plasma 5.27.9, and in that other bug pressing Esc doesn't attempt to unlock the system like in my case. STEPS TO REPRODUCE 1. Install Plasma 6 via kdesrc-build 2. Login 3. Lock the screen 4. Press Esc OBSERVED RESULT The screen does not turn off. EXPECTED RESULT The screen turns off. SOFTWARE/OS VERSIONS Operating System: Arch Linux KDE Plasma Version: 5.27.80 KDE Frameworks Version: 5.240.0 Qt Version: 6.6.0 Kernel Version: 6.5.9-arch2-1 (64-bit) Graphics Platform: Wayland Processors: 16 × AMD Ryzen 7 6800H with Radeon Graphics Memory: 30.6 GiB of RAM Graphics Processor: AMD Radeon Graphics ADDITIONAL INFORMATION kscreenlocker is at commit 6777145016231427cccb4721de1e469a1512715c when I compiled it. Also, when I tried adding some QWarning calls at and around [1] and recompiling I see that the greeter did recognise that the Esc key was pressed, but it never gets to the `dpms->switchMode(KScreen::Dpms::Off)` call. I don't think it's an issue with DPMS not being supported since the the turn off screen keyboard shortcut works just fine, even when the screen is locked. [1]: https://invent.kde.org/plasma/kscreenlocker/-/blob/6777145016231427cccb4721de1e469a1512715c/greeter/greeterapp.cpp#L619 2263780 1 nate 2023-11-06 14:41:08 +0000 Can reproduce. Fallout from the recent KScreenlocker authenticator changes. 2264025 2 bharadwaj.raju777 2023-11-07 17:41:18 +0000 If I just add an unconditional `dpms->switchMode(KScreen::Dpms::Off);` before the connect call, it seems to work. 2264541 3 nate 2023-11-09 21:39:18 +0000 Wanna submit it and see what happens? 2264573 4 putr4.s 2023-11-10 00:47:47 +0000 I filed https://invent.kde.org/plasma/kscreenlocker/-/merge_requests/176 to get rid of the evidently broken connect call and just turn off the screen directly. No idea what would happen on a system without DPMS support though. 2279913 5 nate 2024-01-12 17:12:22 +0000 Bug 338796 got re-opened, so let's track it there since there's an open MR to fix it again. *** This bug has been marked as a duplicate of bug 338796 *** 2282740 6 jpetso 2024-01-23 22:27:07 +0000 Git commit 9aa3bd142370d8fa09f1c42b852ad54846f54260 by Jakob Petsovits, on behalf of Prajna Sariputra. Committed on 23/01/2024 at 23:18. Pushed by jpetso into branch 'master'. greeter: Fix pressing Esc to turn off the display The updated KScreen::Dpms documentation guarantees isSupported() to work right after construction, so use it without async instead of the supportedChanged signal which doesn't get emitted anymore during initialization. M +3 -5 greeter/greeterapp.cpp https://invent.kde.org/plasma/kscreenlocker/-/commit/9aa3bd142370d8fa09f1c42b852ad54846f54260 2282987 7 jpetso 2024-01-24 20:19:30 +0000 Git commit 0b889fee8cf5080ea21fa5e90d32379b26d58de6 by Jakob Petsovits, on behalf of Prajna Sariputra. Committed on 24/01/2024 at 21:16. Pushed by jpetso into branch 'Plasma/6.0'. greeter: Fix pressing Esc to turn off the display The updated KScreen::Dpms documentation guarantees isSupported() to work right after construction, so use it without async instead of the supportedChanged signal which doesn't get emitted anymore during initialization. (cherry picked from commit 9aa3bd142370d8fa09f1c42b852ad54846f54260) M +3 -5 greeter/greeterapp.cpp https://invent.kde.org/plasma/kscreenlocker/-/commit/0b889fee8cf5080ea21fa5e90d32379b26d58de6