[Folder View] Set shortcut on plasmoid.action
ClosedPublic

Authored by broulik on Apr 5 2018, 11:53 AM.

Details

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 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

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Apr 5 2018, 11:53 AM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 5 2018, 11:53 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Apr 5 2018, 11:53 AM
hein accepted this revision.Apr 5 2018, 12:00 PM
This revision is now accepted and ready to land.Apr 5 2018, 12:00 PM
This revision was automatically updated to reflect the committed changes.