<div dir="ltr"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 25, 2015 at 10:24 AM, kainz.a <span dir="ltr"><<a href="mailto:kainz.a@gmail.com" target="_blank">kainz.a@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Ok I would prefere to have the icons at one place because Uri and I do a lot of work that the right icon in the right SIZE is available for the application in the screenshot you see toolbar icons (22px) for the widgets (64px) and this lack.<br><br></div></div></blockquote><div>if something is using the "wrong" icon, please open a bug /email pointing to the misuse and the name of what icon should be used instead, and that case will be fixed within hours.<br><br></div><div>From our code we just write iconName = "activities" or something, and there's no indication of whether it's meant to be used for a toolbar or what without a visual check.  There is a folder structure in the icon set, but when we write this we can't see it. Not helped that it seems to have changed over the last 5 years.<br></div><div><br><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><a href="https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/PlasmaDark_BreezeLight.png" target="_blank">https://dl.dropboxusercontent.com/u/1642456/VDG/KF5/PlasmaDark_BreezeLight.png</a><br><br></div><div>In addition there is one problem. Cause you can use breeze-light icons in breeze-dark plasma you can't see the icons because the light breeze icons are not realy visible on the dark plasma theme. I don't have an solution for that maybe the see also section in the system settings (future) can help.<br><br></div></div></blockquote><br></div><div class="gmail_quote">So in the current state:<br></div><div class="gmail_quote"> - Plasma icons get recoloured<br></div><div class="gmail_quote"> - normal icons do not </div><br></div><div class="gmail_extra">Our icons come from both places<br><br></div><div class="gmail_extra">If a user has a light applications theme and a dark plasma theme, they get screwed.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Our options are:<br><br></div>We make the the icon theme a property of the Plasma theme?<br><div class="gmail_extra"> - Arguably within the colors file, so it can change in multiple colorscopes<br><br></div><div>We make you guys put all icons ever possibly used by Plasma inside the plasma repo which does the colour hinting<br></div><div> (and fix all apples to actually use Plasma.IconItem in all cases - never QIconItem, which some places currently do)<br></div><div><br></div><div> Put the colour hinting into the Breeze icon set in breeze; and have a custom QIconLoader used by the Plasma case to do the hinting whilst loading it from the same directory as everything else.<br></div></div><br><div><div class="gmail_extra"><br></div><div class="gmail_extra">David<br></div></div></div>