Review Request 127878: use the "selected" icon mode in file open dialog sidebar

[Emmanuel Pescosta]

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




src/filewidgets/kfileplacesview.cpp (line 157)
<https://git.reviewboard.kde.org/r/127878/#comment64662>

    option.state & (QStyle::State_Selected | QStyle::State_Active) maybe?



src/widgets/kfileitemdelegate.cpp (line 1220)
<https://git.reviewboard.kde.org/r/127878/#comment64661>

    option.state & (QStyle::State_Selected | QStyle::State_Active) maybe?


- Emmanuel Pescosta


On May 10, 2016, 1:57 p.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127878/
> -----------------------------------------------------------
> 
> (Updated May 10, 2016, 1:57 p.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> since now kiconloader can color the icons in the sidebar, by using the "selected" icon mode, when the breeze icon theme is used the icon of the current sidebar item will be of the same color of selected text (white-ish by default), is more in line with breeze style and the monochrome icons become more readable
> 
> 
> Diffs
> -----
> 
>   src/filewidgets/kfileplacesview.cpp 03074e0 
>   src/widgets/kfileitemdelegate.cpp 1516b9e 
> 
> Diff: https://git.reviewboard.kde.org/r/127878/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> menu3.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/05/09/41f52d86-6fc5-4799-bd9e-18d17dc5a65a__menu3.png
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160510/c81b1f27/attachment.html>