Quoting jpwhiting, <jpwhiting> heh kratingpainter takes the rating icon from the theme and changes the image pixels to grayscale manually... <jpwhiting> strange <jpwhiting> works I guess, though I would think two opposite icons would be better for accessibility <jpwhiting> so if gray is the foreground color as it seems in that bug
On Sat, Oct 11, 2014 at 12:23 PM, Andrew Lake <jamboar...@gmail.com> wrote: > https://bugs.kde.org/show_bug.cgi?id=339863 > > Andrew Lake <jamboar...@gmail.com> changed: > > What |Removed |Added > ---------------------------------------------------------------------------- > Status|UNCONFIRMED |CONFIRMED > Ever confirmed|0 |1 > CC| |jamboar...@gmail.com > > --- Comment #1 from Andrew Lake <jamboar...@gmail.com> --- > It appears the effect used to show an "empty" star is to desaturate the rating > (star) icon. The effect makes sense for hi color icon themes like oxygen. > However, monochrome or greyscale icons are by definition desaturated so the > effect doesn't result in any visible difference. We could add color to the > rating icon in the Breeze icon theme but that would break the style guidelines > for the icon theme, and might fairly be considered a band-aid for the > ineffectiveness of the desaturation effect to correctly produce the desired > "empty" star for monochrome and greyscale rating icons. One solution to the > underlying problem might be to decrease the opacity as well. > > I'll confirm that this a problem, but I think the underlying problem is in the > components where ratings are displayed. I'm not entirely sure which components > those are. > > -- > You are receiving this mail because: > You are the assignee for the bug. > _______________________________________________ > Plasma-devel mailing list > Plasma-devel@kde.org > https://mail.kde.org/mailman/listinfo/plasma-devel -- Bhushan Shah http://bhush9.github.io IRC Nick : bshah on Freenode _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel