D11953: [Folder View] Set shortcut on plasmoid.action

[Kai Uwe Broulik]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180405/110cc6b5/attachment.html>