davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY ::activated depends on the mouse click behaviour - as double click starts a rename this led to the global shortcuts KCM being unusable with double click set. This patch follows the selection chaging - which is more semantically correct as we're not really activating an option in the left hand sidebar, merely selecting it to show the correct sub-options. It fixes keys up/down changing selection but also allows double click mouse settings to work. TEST PLAN Ran. clicked once, got the correct sub-options. REPOSITORY rPLASMADESKTOP Plasma Desktop BRANCH master REVISION DETAIL https://phabricator.kde.org/D2865 AFFECTED FILES kcms/keys/kglobalshortcutseditor.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, #plasma Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas