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