Review Request 128580: If we pass a QIcon as an argument to IconItem::Source, use it
Kai Uwe Broulik
kde at privat.broulik.de
Wed Aug 3 10:53:01 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128580/#review98044
-----------------------------------------------------------
Can we have an autotest for this custom iconloader case?
- Kai Uwe Broulik
On Aug. 2, 2016, 9:11 nachm., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128580/
> -----------------------------------------------------------
>
> (Updated Aug. 2, 2016, 9:11 nachm.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Repository: plasma-framework
>
>
> Description
> -------
>
> Currently the code gets the icon name from the QIcon and tries to do
> some Plasma theming with it.
> However if that fails it then loads the QIcon::fromTheme again.
>
> This is pointless in most cases and will break any icons that have a
> custom loader (all SNIs)
>
>
> Diffs
> -----
>
> src/declarativeimports/core/iconitem.cpp 29c7f05b5df060df7b362b331f7edc412df12307
>
> Diff: https://git.reviewboard.kde.org/r/128580/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160803/2caa3722/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list