Review Request 125568: Revised Breeze Plasma theme for 5.5.

[Alex L.]

In data venerdì 9 ottobre 2015 23:19:51, Uri Herrera ha scritto:
> But that's exactly what you should be doing, use icons depending on the
> context, just because an icon "works" doesn't mean, like at all, that it
> should be used everywhere, regardless of whether is an
> action/status/category, that's exactly why I had (and still have)
> workaround icons in the theme. Because icons were not used properly and
> were used just becasue "they worked". And come on now there's only 2
> configure icons... that's not too many.

I have to disagree, in this case the context is different but the concept is 
the same: that icon on note plasmoid is intended to open settings of that 
plasmoids. The same icon on apps launcher is intended to open system settings. 
In my opinion, this is the right way of using the same icon in different 
contexts: let context to be what explain will happen clicking on that icon.

The hamburger menu icon is a totally different thing: it open a menu, not 
system/apps/plasmoids settings. Hamburger menu generally contains actions 
intended to be use while performing the application's main tasks: i.e. 
swithing between list view and icons view in a file manager. Instead the 
settings icons (one with slides) is intended to open a settings that user will 
rarely use, i.e. to customize the behavior of the app to fit his needs, but do 
it one time and then come back to do tasks that app is designed for.

Anyway, thanks for your work Uri, Breeze Plasma theme looks very better for 
me!

Ciao,
-- 
Alex Long | KDE Visual Design Group
email/xmpp: alexl at opmbx.org
IM: telegram.me/alxlg