Review Request 125307: Add option for desktoptheme to set preferred icon theme

[Marco Martin]

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


not sure about it, as it would only change the behavior of IconItem but not QIconItem, that is the most used for various reasons, so the two would end up using different themes

- Marco Martin


On Sept. 19, 2015, 8:48 a.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125307/
> -----------------------------------------------------------
> 
> (Updated Sept. 19, 2015, 8:48 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> It is now possible to set preferred icon theme in desktoptheme metadata.desktop.
> 
> As discussed on ML, this fixes using light breeze icon theme with breeze dark desktoptheme.
> 
> 
> Diffs
> -----
> 
>   src/plasma/theme.cpp c49ad4c 
>   src/plasma/private/theme_p.cpp 31a6512 
>   src/plasma/theme.h 3f49719 
>   src/desktoptheme/breeze/metadata.desktop 07bbfc3 
>   src/plasma/private/theme_p.h 5b8f71c 
>   src/declarativeimports/core/iconitem.h 3ef0306 
>   src/declarativeimports/core/iconitem.cpp 692cd8d 
>   src/desktoptheme/breeze-dark/metadata.desktop 77647a4 
> 
> Diff: https://git.reviewboard.kde.org/r/125307/diff/
> 
> 
> Testing
> -------
> 
> Icon theme is picked correctly with PlasmaCore.IconItem. Changing the desktoptheme updates the icons.
> 
> 
> Thanks,
> 
> David Rosca
> 
>

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