SUMMARY In Plasma 5.21 beta, I cannot navigate through a list of elements in any application by pressing the arrow keys without releasing my finger. For example, opening a text file in Kate and keeping down pressed would normally make the cursor go all the way down to the end of the file ; here, it goes down one line and stops, as if I pressed down only and released immediately. Same thing for left, right, up. Happens in QT (Dolphin, Konsole, Kate, etc.) and non-Qt applications (Firefox, Eclipse, etc.). The bug appears on Wayland, I have not tested on X11. STEPS TO REPRODUCE 1. Open any application. 2. Push an arrow key without releasing it in order to try to navigate. 3. OBSERVED RESULT It goes in the direction only by one step. EXPECTED RESULT It should keep moving. SOFTWARE/OS VERSIONS Windows: macOS: Linux/KDE Plasma: (available in About System) KDE Plasma Version: 5.20.90 KDE Frameworks Version: 5.78 Qt Version: 5.12.2 ADDITIONAL INFORMATION Archlinux Kde-unstable repo packages.
Ugh, this must be a regression from the key-held stuff. Arrow keys were supposed to be excluded. Regardless, the feature will be reverted for Plasma 5.21, but we need to fix this for 5.22. Jan, can you please take care of this?
*** Bug 431938 has been marked as a duplicate of this bug. ***
The commit has been reverted so this is no longer an issue for Plasma 5.21. We'll make sure it doesn't regress in the future.
Hmm, the bug still happens in Plasma 5.21 (not beta) on Archlinux.
I think I may understand what happened. We had an update script that changed people's settings to use a new feature which was stripped out right before the beta, so the update script updated people's settings incorrectly and caused this problem. Then later we fixed the update script to update people's settings correctly to account for the feature being removed and ensure no regressions. However the automatic system for running update scripts only runs them once. Since we changed the same update script between the beta and the final release, people who were running the beta don't get the update script run again with its correct data when they upgrade from the beta to the final release! So the good news is that this won't affect anyone upgrading from Plasma 5.20 to 5.21 (i.e. most people). The bad news is that it will probably affect everyone who tested the beta. Sorry. :( Jan, any ideas for how we can migrate beta users to the final configuration without asking all of them to manually fix this?
> So the good news is that this won't affect anyone upgrading from Plasma 5.20 to 5.21 (i.e. most people). That's not correct. I never tried the beta and it broke my settings.
The migration is wrong https://invent.kde.org/plasma/plasma-desktop/-/blob/master/kcms/keyboard/kcminputrc_migrate_repeat_value.py KeyboardRepeating was a tri-state in which 0 meant STATE_ON. It's being migrated to "nothing".
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/353
Yikes. Thanks for handling this, folks!
Git commit 5550af1fd10dccda80be4586f19e3aa0995be2bc by Jan Blackquill. Committed on 17/02/2021 at 18:06. Pushed by cblack into branch 'master'. kcms/keyboard: fix migration TriState::STATE_ON is 0, while TriState::STATE_OFF is 1. We're currently migrating 0 -> disabled, when it should be 1 -> disabled, * -> enabled. FIXED-IN: 5.21.1 M +1 -1 kcms/keyboard/kcminputrc_migrate_repeat_value.py https://invent.kde.org/plasma/plasma-desktop/commit/5550af1fd10dccda80be4586f19e3aa0995be2bc
Git commit 64546c81d43f8b10487d9c6c5bb0fab4de457d13 by Jan Blackquill. Committed on 17/02/2021 at 18:07. Pushed by cblack into branch 'Plasma/5.21'. kcms/keyboard: fix migration TriState::STATE_ON is 0, while TriState::STATE_OFF is 1. We're currently migrating 0 -> disabled, when it should be 1 -> disabled, * -> enabled. FIXED-IN: 5.21.1 (cherry picked from commit 5550af1fd10dccda80be4586f19e3aa0995be2bc) M +1 -1 kcms/keyboard/kcminputrc_migrate_repeat_value.py https://invent.kde.org/plasma/plasma-desktop/commit/64546c81d43f8b10487d9c6c5bb0fab4de457d13
*** Bug 433147 has been marked as a duplicate of this bug. ***
*** Bug 433335 has been marked as a duplicate of this bug. ***
*** Bug 433255 has been marked as a duplicate of this bug. ***
Coming from 433335 (which was marked as duplicate of this bug), the comment by Jan suggests that it was fixed in 5.21.1. Arch repos have the updated packages, and after installing it, the issue still persists. (Just to add: I have never installed the beta packages).
I can confirm here, the bug is not yet fixed.
I'm not sure whether this is related or not, but also since the update to Plasma 5.21, navigating through texts with ctrl+arrow which should move by words has stopped working. In particular, I liked to use that in Konsole. My input method is set to ibus. Should I open a new issue for that?
Yeah, I'm so sorry. :( We are hoping to finally finally finally have this fixed in Plasma 5.21.2 with https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/355
Git commit b4f3a470e7c477815b1f95f1a8e40bdd1dcf5b83 by Jan Blackquill. Committed on 24/02/2021 at 18:19. Pushed by cblack into branch 'master'. kcms/keyboard: handle botched 5.21.0 migration Since we shipped a botched migration script for 5.21.0, we need to ship a new migration script that un-botches the effects of the 5.21.0 script. We also change the ID of the plain kcminput_repeat migration so that we can distinguish between the botched migration and the non-botched migration, and only reverse migrations applied by the botched migration. FIXED-IN: 5.21.2 M +4 -0 kcms/keyboard/CMakeLists.txt A +60 -0 kcms/keyboard/kcminputrc_fix_botched_5_21_0.py A +35 -0 kcms/keyboard/kcminputrc_fix_botched_5_21_0.upd M +1 -1 kcms/keyboard/kcminputrc_repeat.upd https://invent.kde.org/plasma/plasma-desktop/commit/b4f3a470e7c477815b1f95f1a8e40bdd1dcf5b83
Git commit c03bba6eb40bf12f497f05417ef7ab3d40f7edae by Jan Blackquill. Committed on 24/02/2021 at 18:21. Pushed by cblack into branch 'Plasma/5.21'. kcms/keyboard: handle botched 5.21.0 migration Since we shipped a botched migration script for 5.21.0, we need to ship a new migration script that un-botches the effects of the 5.21.0 script. We also change the ID of the plain kcminput_repeat migration so that we can distinguish between the botched migration and the non-botched migration, and only reverse migrations applied by the botched migration. FIXED-IN: 5.21.2 (cherry picked from commit b4f3a470e7c477815b1f95f1a8e40bdd1dcf5b83) M +4 -0 kcms/keyboard/CMakeLists.txt A +60 -0 kcms/keyboard/kcminputrc_fix_botched_5_21_0.py A +35 -0 kcms/keyboard/kcminputrc_fix_botched_5_21_0.upd M +1 -1 kcms/keyboard/kcminputrc_repeat.upd https://invent.kde.org/plasma/plasma-desktop/commit/c03bba6eb40bf12f497f05417ef7ab3d40f7edae