Review Request 118842: Add reset function to QImageItem image property
David Edmundson
david at davidedmundson.co.uk
Fri Jun 20 10:41:33 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118842/
-----------------------------------------------------------
(Updated June 20, 2014, 10:41 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and Plasma.
Repository: kdeclarative
Description
-------
Add reset function to QImageItem
This allows
QImageItem {
image: undefined
}
without warnings.
This is useful as otherwise it's impossible to unset a set QImageItem as QImage is uncreatable from within QML.
The immediate usage is to fix a bug in notifications where sometimes an image from a previous notification "leaks" into our current notification.
Diffs
-----
src/qmlcontrols/kquickcontrolsaddons/qimageitem.h eef59bc
src/qmlcontrols/kquickcontrolsaddons/qimageitem.cpp ed20605
src/qmlcontrols/kquickcontrolsaddons/qpixmapitem.h eee8189
src/qmlcontrols/kquickcontrolsaddons/qpixmapitem.cpp fcc66b5
Diff: https://git.reviewboard.kde.org/r/118842/diff/
Testing
-------
Wrote small qml file, code had warning before, now it doesn't.
Images are still displayed if set.
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140620/d054139c/attachment.html>
More information about the Plasma-devel
mailing list