| Summary: | After focus is automatically moved to search results list, deleting a character in the search field requires pressing the backspace key twice | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Martin Bednar <martin+kde> |
| Component: | Application Menu (Kicker) widget | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | funny0facer, gwarser, hendrikloewe3, kdebugs, nate, notmart, tamas |
| Priority: | NOR | Keywords: | accessibility, regression, usability |
| Version First Reported In: | 6.0.1 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/87e8e0a28a4b73916e73d92a599562028bec6f62 | Version Fixed In: | 6.3.0 |
| Sentry Crash Report: | |||
| Attachments: | Screencast of the bug including visible keystrokes | ||
|
Description
Martin Bednar
2024-03-11 15:49:40 UTC
Can reproduce. Created attachment 167758 [details]
Screencast of the bug including visible keystrokes
I added a screencast and can confirm the behavior also on 6.0.2
KRunner works fine
Application Launcher works fine
Application Menu does not behave as the other 2
Operating System: Arch Linux
KDE Plasma Version: 6.0.2
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2
Kernel Version: 6.8.1-arch1-1 (64-bit)
Graphics Platform: Wayland
After investigating a bit, I think the focus change is intentional so that you can navigate the search results list, but requiring two backspace keystrokes is not; this part is clearly a bug. is this still happening? can't reproduce anymore Can still reproduce. OS: openSUSE Leap 15.6 Plasma Version: 6.2.90 Frameworks Version: 6.10.0 Qt Version: 6.8.1 Kernel: 6.4.0-150600.23.33-default (64-bit) Graphics: Wayland on discrete GPU If something is found, the focus moves to the first entry. Pressing backspace once moves it back. Pressing twice removes the last char. I can still reproduce it trivially with Kicker (not kickoff) on git master: 1. Open menu 2. Type "Hello" 3. press backspace The search field is focused but "o" is not removed. If I press backspace again, "o" is removed but the search field loses focus. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2757 Git commit 5d9a5ce1c7319f05c5859321c18ec37b97ab58d9 by Marco Martin. Committed on 28/01/2025 at 15:50. Pushed by mart into branch 'master'. applets/kicker: fix backspace when searching use a proper focus handling when switching back and forth from navigating items in the menu and searching by writing text. * ItemListView knows about the search field and always forwards events to it * when the listview is focused, if a non navigation key is pressed, just moves the active focus to the search field, the letter (or backspace, or right/left arrow) will already have been managed by the search field * if the search field is focused, pressing a navigation key still transfers the focus to the listview M +2 -0 applets/kicker/package/contents/ui/ItemListDialog.qml M +16 -20 applets/kicker/package/contents/ui/ItemListView.qml M +4 -4 applets/kicker/package/contents/ui/MenuRepresentation.qml M +1 -0 applets/kicker/package/contents/ui/RunnerResultsList.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/5d9a5ce1c7319f05c5859321c18ec37b97ab58d9 Git commit 87e8e0a28a4b73916e73d92a599562028bec6f62 by Marco Martin. Committed on 28/01/2025 at 15:50. Pushed by mart into branch 'Plasma/6.3'. applets/kicker: fix backspace when searching use a proper focus handling when switching back and forth from navigating items in the menu and searching by writing text. * ItemListView knows about the search field and always forwards events to it * when the listview is focused, if a non navigation key is pressed, just moves the active focus to the search field, the letter (or backspace, or right/left arrow) will already have been managed by the search field * if the search field is focused, pressing a navigation key still transfers the focus to the listview (cherry picked from commit 5d9a5ce1c7319f05c5859321c18ec37b97ab58d9) 63a4478c applets/kicker: fix backspace when searching 565ddf4d remove unnecessary change Co-authored-by: Marco Martin <notmart@gmail.com> M +2 -0 applets/kicker/package/contents/ui/ItemListDialog.qml M +16 -20 applets/kicker/package/contents/ui/ItemListView.qml M +4 -4 applets/kicker/package/contents/ui/MenuRepresentation.qml M +1 -0 applets/kicker/package/contents/ui/RunnerResultsList.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/87e8e0a28a4b73916e73d92a599562028bec6f62 *** Bug 499315 has been marked as a duplicate of this bug. *** |