[Clipboard plasmoid] Split delegate into separate files
ClosedPublic

Authored by broulik on Feb 15 2018, 9:20 PM.

Details

Summary

The delegate is quite heavy with lots of seldom used items and opening the clipboard plasmoid takes a significant amount of time.

Test Plan
  • Plasmoid opens almost instantly now
  • Verified that single line and multi line text still shows properly
  • Verified that delegate height is correct in all cases (initially, multi line text, with tool buttons shown, image, list)
  • Unchecked "ignore images", copied image, delegate was shown correctly
  • Copied a couple of files, they showed up correctly, both when there's enough room and when it overflows "+n files"
  • Verified that clicking a delegate item copies it to clipboard and moves it to the top
  • Removing items from history still works
  • Generating barcode still works
  • Configured an action, copied some text with it in, clicked "run action" button, popup to choose action showed up and action could be triggered
  • Verified that tool buttons are created only on hover and only once
  • Verified that the fancy gradient is rendered correctly in both LTR and RTL mode

Diff Detail

Repository
R120 Plasma Workspace
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Feb 15 2018, 9:20 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 15 2018, 9:20 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Feb 15 2018, 9:20 PM
broulik updated this revision to Diff 27647.Feb 20 2018, 10:47 PM

Remove debug

mart accepted this revision.Feb 21 2018, 11:51 AM
This revision is now accepted and ready to land.Feb 21 2018, 11:51 AM
This revision was automatically updated to reflect the committed changes.