broulik created this revision. broulik added reviewers: Plasma, hein. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. broulik requested review of this revision.
REVISION SUMMARY Ensures the shortcuts are shown in the menu. We need to accept the event if we handled it or else the containment will trigger the action leading to double activation. BUG: 392730 FIXED-IN: 5.12.5 TEST PLAN - Context menu shows shortcuts for Ctrl+V and Ctrl+Z now (F5 <https://phabricator.kde.org/F5> in follow-up patch) - Verified that Ctrl+V fires only once and that it pastes in the right folder (ie. it is actually triggering the action of the focussed FV) - Verified that Ctrl+C and Ctrl+X work correctly - Verified that Ctrl+Z works - Verified that Ctrl+click selects file as before, same for Shift (range) and Ctrl rubberband - Verified that Home/End jump correctly, also with Shift to select - Verified that Ctrl+A works correctly REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D11953 AFFECTED FILES containments/desktop/package/contents/ui/FolderView.qml containments/desktop/package/contents/ui/FolderViewLayer.qml To: broulik, #plasma, hein Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
