Review Request 120902: [KRatingPainter] Support monochrome icon themes (such as breeze)

David Edmundson david at davidedmundson.co.uk
Thu Oct 30 14:04:05 UTC 2014


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



src/kratingpainter.cpp
<https://git.reviewboard.kde.org/r/120902/#comment48667>

    this was previously only turned into an image to do the toGrayScale conversion.
    
    I don't think we want to turn it back.



src/kratingpainter.cpp
<https://git.reviewboard.kde.org/r/120902/#comment48669>

    This part seems like it needs fixing too, as it's still just taking the full potentially grey icon and greyscaling it.


- David Edmundson


On Oct. 29, 2014, 11:30 p.m., Christoph Feck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120902/
> -----------------------------------------------------------
> 
> (Updated Oct. 29, 2014, 11:30 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Bugs: 339863
>     https://bugs.kde.org/show_bug.cgi?id=339863
> 
> 
> Repository: kwidgetsaddons
> 
> 
> Description
> -------
> 
> The old code converted the yellow star icon to grayscale for the unrated part. With icon themes that have no color, this means the rating cannot be read, because all stars look identical.
> 
> This code adds support for a "rating-unrated" icon that theme creators can use for the unrated part. If the icon theme does not support this icon, it falls back to desaturating.
> 
> 
> Diffs
> -----
> 
>   src/kratingpainter.cpp 8e8a6a3 
> 
> Diff: https://git.reviewboard.kde.org/r/120902/diff/
> 
> 
> Testing
> -------
> 
> I couldn't actually test with breeze theme (Qt issue?), but I locally added a dedicated "rating-unrated.png" file to oxygen and tested both cases (present and not).
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>

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


More information about the Kde-frameworks-devel mailing list