Bug 397779

Summary: Navigating search result for apps require double-pressing of arrow buttons
Product: [Plasma] plasmashell Reporter: Øystein Steffensen-Alværvik <epost.kde>
Component: Application Menu (Kicker)Assignee: Eike Hein <hein>
Status: RESOLVED FIXED    
Severity: minor CC: nate, plasma-bugs, rikmills
Priority: NOR Keywords: accessibility, usability
Version: 5.13.3   
Target Milestone: 1.0   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In: 5.12.7
Sentry Crash Report:

Description Øystein Steffensen-Alværvik 2018-08-23 10:27:58 UTC
When navigating a search result with the arrow keys, Kicker's results for "Applications" require pressing the Down arrow on the keyboard twice to move down the list. Also, pressing the "→" button will not move to the other lists (like "Desktop search") until you press another arrow key.
Comment 1 Nate Graham 2018-08-23 23:07:07 UTC
Can confirm. Likely needs the same fix we applied to KRunner to fix this problem there (https://cgit.kde.org/plasma-workspace.git/commit/?id=b8ffa755ed2df6bf6a47b9268e7da93355a4d856)
Comment 2 Eike Hein 2018-09-04 09:45:25 UTC
Weird, I recall fixing this before. I guess it regressed again. Will look into it.
Comment 3 Eike Hein 2018-09-05 09:31:10 UTC
I can't reproduce this. Nate, could you give me exact steps?
Comment 4 Eike Hein 2018-09-05 09:36:01 UTC
Nvm, I reproduce a problem with arrow-down handling and also the column issue. Looking into it atm.
Comment 5 Eike Hein 2018-09-06 09:44:01 UTC
Git commit 1692ae244bc5229df78df2d5ba2e76418362cb50 by Eike Hein.
Committed on 06/09/2018 at 09:43.
Pushed by hein into branch 'Plasma/5.12'.

Improve arrow key navigation of Kicker search results

Summary:
Kicker already makes sure the first search result is highlighted
and actionable while searching. This patch makes arrow-down after
typing move to the second search result instead of merely moving
focus from the search field to the list, so the user doesn't have
to press arrow-down twice anymore.

It also allows using arrow-left/right to jump to the other result
columns, when the cursor is at the start or end of the text field,
respectively.

Reviewers: ngraham

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D15286

M  +19   -1    applets/kicker/package/contents/ui/MenuRepresentation.qml
M  +1    -0    applets/kicker/package/contents/ui/RunnerResultsList.qml

https://commits.kde.org/plasma-desktop/1692ae244bc5229df78df2d5ba2e76418362cb50
Comment 6 Eike Hein 2018-09-06 09:46:36 UTC
These fixes will be in the next 5.12.x, 5.13.x and the 5.14 releases.
Comment 7 Øystein Steffensen-Alværvik 2018-09-06 16:22:23 UTC
Cheers!
Comment 8 Rik Mills 2018-09-06 20:26:18 UTC
(In reply to Eike Hein from comment #6)
> These fixes will be in the next 5.12.x, 5.13.x and the 5.14 releases.

5.13 has had its final point release, but maybe this is worth cherry picking?