Review Request 125568: Revised Breeze Plasma theme for 5.5.
Uri Herrera
uri_herrera at nitrux.in
Fri Oct 9 19:45:11 UTC 2015
> On Oct. 9, 2015, 8:50 a.m., David Edmundson wrote:
> > In general all looks fine.
> >
> > Two things I want to check up on:
> >
> > - looks like you've changed the colours of the notes. The name of these colours are hardcoded in the applet, so that needs checking they match
> >
> > - Have you changed the K icon for the launcher into something else?
>
> Marco Martin wrote:
> seems so, and yeah, that logo definitely *cannot* be changed.
* I did change the color of the sticky notes to a more modern soothing and Breeze colored version. Also, matched it with the style (almost flat but not completely flat) of the icons. I made sure to add the same number of colors that were in the widget. Works fine when I change them.
* Yes, to the Plasma logo. Since it's the Plasma desktop it seems more appropriate to me, branding wise, to use the Plasma logo. But well I can change it...
- Uri
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125568/#review86528
-----------------------------------------------------------
On Oct. 9, 2015, 8:28 a.m., Uri Herrera wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125568/
> -----------------------------------------------------------
>
> (Updated Oct. 9, 2015, 8:28 a.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> SVG elements done from scratch, reused only a small portion of the existing files. Colors now match Breeze colorscheme. Buttons now match widget style. Icons in the system tray are taken from the icon theme. Icons in Kickoff are not monochromatic as only 16px and 22px icons are monochrome in Breeze. Better padding overall for elements. New visual for both task manager widgets that matches the states of the buttons. Checkboxes, raio buttons, sliders and scrollbars now have the colors used in the Breeze colorscheme.
>
>
> Diffs
> -----
>
>
> Diff: https://git.reviewboard.kde.org/r/125568/diff/
>
>
> Testing
> -------
>
> ![Before](http://i.imgur.com/Cz2JNee.png)
> ![After](http://i.imgur.com/hiZlyrb.png)
>
> ![Elements after](http://i.imgur.com/uMFsMzp.png)
> ![Elements before](http://i.imgur.com/pIs1CZ2.png)
>
>
> Thanks,
>
> Uri Herrera
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151009/acc9017f/attachment-0001.html>
More information about the Plasma-devel
mailing list