https://bugs.kde.org/show_bug.cgi?id=392197

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/pla
                   |                            |sma-workspace/b8ffa755ed2df
                   |                            |6bf6a47b9268e7da93355a4d856
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED

--- Comment #3 from Nate Graham <n...@kde.org> ---
Git commit b8ffa755ed2df6bf6a47b9268e7da93355a4d856 by Nathaniel Graham.
Committed on 23/03/2018 at 14:19.
Pushed by ngraham into branch 'Plasma/5.12'.

Move to KRunner's second results item with a single keypress

Summary:
Currently, to navigate the list of results after typing a search term, you need
to press the down arrow twice: first to focus the result list, and a second
time to actually change the selection. This patch fixes that by changing the
selection during the focus oprtation.

Test Plan:
Deployed change, killed, restarted, and activated KRunner, typed a search term:
- The down arrow key now moves to the second item
- Clicked on the text field; up arrow now moves to the previous item in the
results list

Activating the {nav Open Containing Folder} button for the first result via the
keyboard changes slightly with this patch:
- Currently, it's {nav down arrow > tab key}
- With this patch, it's {nav tab key three times} or {nav down arrow > shift +
tab}

Seems like an acceptable change to me to better support the common case of just
wanting to easily navigate to the results list items.

Reviewers: #plasma, broulik, mart

Reviewed By: #plasma, broulik, mart

Subscribers: plasma-devel

Tags: #plasma

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

M  +4    -2    lookandfeel/contents/runcommand/RunCommand.qml

https://commits.kde.org/plasma-workspace/b8ffa755ed2df6bf6a47b9268e7da93355a4d856

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to