Review Request 125568: Revised Breeze Plasma theme for 5.5.

Martin Klapetek martin.klapetek at gmail.com
Fri Oct 9 23:11:44 UTC 2015



> On Oct. 9, 2015, 12:40 p.m., Kai Uwe Broulik wrote:
> > Finally new sticky notes \o/
> > 
> > I'm a bit sad about the loss of shadows around widgets. Also, I liked Plasma's configure icon much better and wish that one was put into the Breeze theme rather than vice-versa as it is now.
> 
> Martin Klapetek wrote:
>     I agree with the icon. I feel like it's changing every release or so.
> 
> Uri Herrera wrote:
>     Yeah, the shadows.. I was intending on having the shadows the same way they're in the actual windows but I find it to be really hard to get it right. The super heavy shadow that was being used was a left-over from the Oxygen and Air themes. It is still using a shadow of course just not as heavy.
>     
>     I added hamburger menu icon to the icon theme a while back.
> 
> Martin Klapetek wrote:
>     > I added hamburger menu icon to the icon theme a while back.
>     
>     The one on the notes applet. In the "before" screenshot, it's the two horizontal lines with circles, the "after" has a window of some sort. Before it was a wrench icon. I do agree with Kai that the horizontal-bars-with-circly-things looks better as a settings icon, the window of some sort is not that much telling.
> 
> Uri Herrera wrote:
>     That icon is the _literal_ representation of the window that it opens.
>     
>     ![](http://i.imgur.com/axfxffD.png)
> 
> Martin Klapetek wrote:
>     Yeah I get that. But I don't want to keep remembering the looks of windows to associate them with an icon/action (not mentioning the config window looks might change and that would change the icon (again) I presume). It's kind of like the "save" button all of a sudden getting a /literal/ looks of the save dialog instead of that floppy that everyone is so used to.
> 
> Uri Herrera wrote:
>     You did do that whenver you started using software that had the floppy as the save icon. You had to remember "this floppy icon means save" and now you're used to it. It's the same.
> 
> Uri Herrera wrote:
>     Also the icon (the one with sliders) is for 1) Start menu categories 2) a somewhat small version of System Settings but not really and 3) a workaround for applications whose toolbar uses a category icon (the actual use) -
> 
> Martin Klapetek wrote:
>     But then again - this icon is being changed for the third time in 5 releases. By the time me and all our users got this association into mine/their minds, there comes the need to reassociate. This time with a window. Which I still think makes no sense. Especially since the config dialog is very likely /not/ going to look like this on Plasma Mobile. Hence the generic/symbolic representation of the action seems better fit rather than a /literal/ representation of the window.
> 
> andreas kainz wrote:
>     But Uri use the config icon instead of the shortcut icon (old one) and about 3 changes in 5 releases. sorry we work heavy on the icons.
> 
> Uri Herrera wrote:
>     That's as simple to fix as Plasma mobile using a different icon and me making it. In fact it wasn't changed, it was missing. The one I made first was the one with the sliders (for the toolbars) and that was the one used when the current Breeze theme was made because it was the only icon for that action (configuration). The version before that was using assets from Oxygen and Air.
>     
>     And I do understand what you mean, but, it's the same whenever we learn anything new. Always, we have to get used to it first. I (and Andreas of course) have changed other icons too over the course of the last year. Haven't seen that as a problem, setback, issue or cause of confusion for people using Breeze.
> 
> andreas kainz wrote:
>     and of corse most icons were changed cause of a bug report

> fix as Plasma mobile using a different icon and me making it

Which is precisely what I would like to avoid in our software. There are just too many "configure" icons around. But oh well, why do I bother.


- Martin


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


On Oct. 9, 2015, 10:18 p.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, 10:18 p.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, radio 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/2ef8e97c/attachment-0001.html>


More information about the Plasma-devel mailing list