Review Request 129835: [IconDialog] Set proper icons group

Anthony Fieroni bvbfan at abv.bg
Thu Feb 2 04:41:17 UTC 2017


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

(Updated Feb. 2, 2017, 4:41 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma, Christoph Feck and Marco Martin.


Changes
-------

Submitted with commit ba21d4e15be4f83e734101ec989677eda9b07160 by Anthony Fieroni to branch master.


Repository: kdeclarative


Description
-------

Follow the comment in IconThemes: https://api.kde.org/frameworks/kiconthemes/html/kicondialog_8cpp_source.html#l00500
iconSize == 0 && group == KIconLoader::NoGroup


Diffs
-----

  src/qmlcontrols/kquickcontrolsaddons/icondialog.cpp be54174 

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


Testing
-------

Change group to Desktop, explicit size will override it.


Thanks,

Anthony Fieroni

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170202/f51fb301/attachment.html>


More information about the Plasma-devel mailing list