Review Request 128721: monochrome kicker icons

Marco Martin notmart at gmail.com
Sat Aug 20 20:14:13 UTC 2016


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



personally, i like it.

as for the new name for the leave section, is this intended for plsma 5.8 as well?

- Marco Martin


On Aug. 20, 2016, 12:49 p.m., Andreas Kainz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128721/
> -----------------------------------------------------------
> 
> (Updated Aug. 20, 2016, 12:49 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> the kicker icons are the only colored icon in the plasma breeze theme all other icons are monochrome.
> 
> As the system icons are now again only monochrome the leave icon in kicker would be monochrome so there are two posibilities:
> 
> 1. make the other kicker icons also monochrome (see this request)
> 2. add a new icon name for the leave section in kicker (new name -> problem with other themes from kaos, manjaro, ...)
> 
> 
> Diffs
> -----
> 
>   src/desktoptheme/breeze/icons/applications.svgz 190f195 
>   src/desktoptheme/breeze/icons/bookmarks.svgz 6b52e8a 
>   src/desktoptheme/breeze/icons/computer.svgz fcbe605 
>   src/desktoptheme/breeze/icons/view.svgz fc0272f 
> 
> Diff: https://git.reviewboard.kde.org/r/128721/diff/
> 
> 
> Testing
> -------
> 
> with breeze and breeze dark theme
> 
> monochrome icons:
> https://dl.dropboxusercontent.com/u/1642456/vdg/Cantor/kickerMonochrome.png
> 
> 
> Thanks,
> 
> Andreas Kainz
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160820/160e4994/attachment-0001.html>


More information about the Plasma-devel mailing list