D10557: [Clipboard plasmoid] Split delegate into separate files

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, graesslin.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION 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

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D10557

AFFECTED FILES
  applets/clipboard/contents/ui/ClipboardItemDelegate.qml
  applets/clipboard/contents/ui/DelegateToolButtons.qml
  applets/clipboard/contents/ui/ImageItemDelegate.qml
  applets/clipboard/contents/ui/TextItemDelegate.qml
  applets/clipboard/contents/ui/UrlItemDelegate.qml

To: broulik, #plasma, graesslin
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180215/3a7ebdbd/attachment.html>