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