D11611: Move to KRunner's second results item with a single keypress
ngraham closed this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D11611 To: ngraham, #plasma, broulik, mart Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D11611: Move to KRunner's second results item with a single keypress
ngraham updated this revision to Diff 30319. ngraham added a comment. Also don't lose focus when there are no search results REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11611?vs=30311=30319 BRANCH fix-krunner-double-down-key-issue (branched from master) REVISION DETAIL https://phabricator.kde.org/D11611 AFFECTED FILES lookandfeel/contents/runcommand/RunCommand.qml To: ngraham, #plasma, broulik, mart Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D11611: Move to KRunner's second results item with a single keypress
broulik accepted this revision. broulik added a comment. I guess this could go into 5.12 also REPOSITORY R120 Plasma Workspace BRANCH fix-krunner-double-down-key-issue (branched from master) REVISION DETAIL https://phabricator.kde.org/D11611 To: ngraham, #plasma, broulik, mart Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D11611: Move to KRunner's second results item with a single keypress
ngraham created this revision. ngraham added reviewers: Plasma, broulik. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. ngraham requested review of this revision. REVISION 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. BUG: 392197 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 moved to the previous item in the results list Activating the Open Containing Folder button for the first result via the keyboard changes slightly with this patch: - Currently, it's down arrow > tab key - With this patch, it's tab key three times or down arrow > shift + tab Seems like an acceptable change to me to better support the common case of just wanting to navigate the list items REPOSITORY R120 Plasma Workspace BRANCH fix-krunner-double-down-key-issue (branched from master) REVISION DETAIL https://phabricator.kde.org/D11611 AFFECTED FILES lookandfeel/contents/runcommand/RunCommand.qml To: ngraham, #plasma, broulik Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart