Review Request 124580: Use the better implementation of IconDialog

David Edmundson david at davidedmundson.co.uk
Sat Aug 1 18:27:32 UTC 2015


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

Review request for Plasma.


Repository: plasma-desktop


Description
-------

For some reason we have two IconDialog's exported to QML.

One in KDeclarative, one in Plasma-Frameworks/platformcomponents.

The one in plasmacomponents performs an exec() in the main thread from
QML which is a very bad idea. the one in kdeclarative is good.

BUG: 350864


Diffs
-----

  desktoppackage/contents/activitymanager/ActivityCreationDialog.qml 102926bf57f21f33e2c45ff160ddfc68dd028dde 

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


Testing
-------

Changed my activity icon.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150801/058ea6b2/attachment-0001.html>


More information about the Plasma-devel mailing list