<div dir="ltr">Thanks for the replay<br><div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-28 14:31 GMT+02:00 David Edmundson <span dir="ltr"><<a href="mailto:david@davidedmundson.co.uk" target="_blank">david@davidedmundson.co.uk</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);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></span><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></div></div></div></blockquote><div><br><span class="">I will make bug reports and I will learn to fix it myself and push it to reviewboard. Sometimes we have to make new icons in the right category with a new name so it could happen that you have to update kde:breeze where the icons are to get the new icons. but before an release everything would be fine.</span><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><div></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><span class=""><div><br></div></span></div></div></div></div></blockquote><div><br></div><div>I was yesterday the first time in the source and saw that you don't get an information about the icon "Type". The VDG discuss the plasmoid icons yesterday and the result was to make colored app icons (64px) for all plasmoids so an plasmoid is from our point of view the same than an application. What we would suggest is that the plasmoid icons get an prefix like plasmoid-xxx like the preferences-xxx icons for system settings.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);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></span></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.<span class=""><font color="#888888"><br></font></span></div></div><span class=""><font color="#888888"><br></font></span></div></blockquote><div><br></div><div>most of the problems with breeze is that the icons are monochrome and the user get in trouble when the contrast is bad. It would be really nice to find an soulution that not only plasma can recolour the icons also the application could. This would be a BIG visual benefit for the user. So if it would be possible tell me what I have to do.<br><br><a href="https://forum.kde.org/viewtopic.php?f=285&t=127663&start=60#p340602">https://forum.kde.org/viewtopic.php?f=285&t=127663&start=60#p340602</a><br><a href="http://wstaw.org/m/2015/08/28/colorize-icons.png">http://wstaw.org/m/2015/08/28/colorize-icons.png</a><br></div><div> </div>Thanks<br></div><div class="gmail_quote">Andreas<br></div><br></div></div></div>