Review Request 120550: Use the same strategy as FrameSvgItem and IconItem in QIconItem

Aleix Pol Gonzalez aleixpol at kde.org
Mon Oct 13 01:13:41 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120550/
-----------------------------------------------------------

Review request for KDE Frameworks, David Edmundson and Marco Martin.


Repository: kdeclarative


Description
-------

Moves the caching types used in Plasma Workspace into a QuickAddons submodule.

Adopts them in QIconItem by moving it from a QPainterItem to a QQuickPainter item. Uses the same strategies used in Plasma Framework for caching the textures.


Diffs
-----

  src/qmlcontrols/kquickcontrolsaddons/qiconitem.cpp 0eb46b1 
  src/quickaddons/CMakeLists.txt PRE-CREATION 
  src/quickaddons/imagetexturescache.h PRE-CREATION 
  src/quickaddons/imagetexturescache.cpp PRE-CREATION 
  tests/qiconitem_test.qml PRE-CREATION 
  src/CMakeLists.txt eb0dfd3 
  src/qmlcontrols/kquickcontrolsaddons/CMakeLists.txt 2977812 
  src/qmlcontrols/kquickcontrolsaddons/qiconitem.h 839a33f 

Diff: https://git.reviewboard.kde.org/r/120550/diff/


Testing
-------

I added some manual test (that was impossible to run before the patch). Also tested it in KRunner and Muon Discover, which use the component. Everything still works.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141013/33f7dd48/attachment.html>


More information about the Kde-frameworks-devel mailing list