[DesktopView] Move cancelling showing desktop into keyPressEvent
ClosedPublic

Authored by broulik on Aug 31 2017, 2:01 PM.

Details

Summary

This will route the event through the containment first which can then intercept it, e.g. to prevent Escape from exiting showing desktop while renaming where it should only cancel that.

CCBUG: 352988

Test Plan

I can now event.accept in FolderView's escape handlers to prevent exiting showing desktop when renaming and instead only cancel the rename.

I can still type to launch KRunner. I can still type a new file name just fine. I can still navigate with my keys.

Diff Detail

Repository
R120 Plasma Workspace
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.Aug 31 2017, 2:01 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 31 2017, 2:01 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Aug 31 2017, 2:10 PM
This revision is now accepted and ready to land.Aug 31 2017, 2:10 PM
This revision was automatically updated to reflect the committed changes.