<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-09-28 23:24 GMT+02:00 Sebastian Kügler <span dir="ltr"><<a href="mailto:sebas@kde.org" target="_blank">sebas@kde.org</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"><span class="">On Monday, September 28, 2015 20:33:33 Marco Martin wrote:<br>
> On Monday 28 September 2015, andreas kainz wrote:<br>
> > > On Sept. 28, 2015, 2:29 p.m., Eike Hein wrote:<br>
> > > > applets/showActivityManager/package/metadata.desktop, line 114<br>
> > > > <<a href="https://git.reviewboard.kde.org/r/125437/diff/1/?file=408607#file4086" rel="noreferrer" target="_blank">https://git.reviewboard.kde.org/r/125437/diff/1/?file=408607#file4086</a><br>
> > > > 0<br>
> > > > 7line114><br>
> > > ><br>
> > > >     The applet icons aren't really part of the icon theme, but the<br>
> > > >     icon<br>
> > > >     spec does say "As a further restriction, all icon names may only<br>
> > > >     contain lowercase letters, numbers, underscore, dash, or period<br>
> > > >     characters." - maybe we should keep the lowercase restriction for<br>
> > > >     our icon assets? It's not a big deal though.<br>
> ><br>
> > should I remove the org.kde.? I would prefere icon names like<br>
> ><br>
> > plasma-showactivitymanager<br>
> ><br>
> > according to our wiki<br>
> > <a href="https://github.com/NitruxSA/breeze-icon-theme/wiki/Icon-naming-and-categor" rel="noreferrer" target="_blank">https://github.com/NitruxSA/breeze-icon-theme/wiki/Icon-naming-and-categor</a><br>
> > ization-guidelines-in-Breeze<br>
><br>
> fine with me, in that case is perhaps better to have<br>
> plasma-applet-showactivitymanager<br>
> to be really sure all names won't clash with something else some day<br>
<br>
</span>I'd prefer having the plugin name (so keeping the org.kde prefix) also used in<br>
the icon name, it's way easier to find the icon programmatically this way<br>
(even if we define it in the .desktop file).<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div><br>I'd prefere the naming of app specific icons according to our guidelines. hope it would work for you too Sebastian. I understand that it is difficult to find the right widget name in the plasma source code, cause the names are quite different, but this is no icon name problem and of course widgets should have the same icon type than all other apps.<br><br>plasma-applet-showactivitymanager <br></div></div><br></div></div>