> 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 > ------- > >  >  > >  >  > > > Thanks, > > Uri Herrera > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel