[Notifications] Split draggable thumbnail area into its own component
ClosedPublic

Authored by broulik on Dec 4 2019, 9:21 PM.

Details

Summary

I'm planning to use it in job notifications, too.
This should also make it easier to migrate away from a custom drag implementation to using Qt's if we were to do that.

Test Plan
  • Took a screenshot:
  • clicking still opened the screenshot
  • right click still brought up the file menu
  • dragging the screenshot somewhere still worked
  • while the drag was active the timeout was halted
  • context menu button still worked

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.Dec 4 2019, 9:21 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 4 2019, 9:21 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Dec 4 2019, 9:21 PM
davidedmundson accepted this revision.Dec 5 2019, 4:19 PM
This revision is now accepted and ready to land.Dec 5 2019, 4:19 PM
This revision was automatically updated to reflect the committed changes.