<br><br><div class="gmail_quote">On Sun, Jan 18, 2009 at 12:36 AM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On 17.01.09 17:30:41, Aleix Pol Gonzalez wrote:<br>
> SVN commit 912545 by apol:<br>
><br>
> put some icons to some actions so that menus can be easily understood<br>
> @@ -215,6 +218,7 @@<br>
>          if ( !closeProjectsAdded && prjitem && prjitem->isProjectRoot() )<br>
>          {<br>
>              KAction* close = new KAction( i18n( "Close Project(s)" ), this );<br>
> +            close->setIcon(KIcon("dialog-close"));<br>
<br>
Thats surely wrong, icon names are according to what the icon represents.<br>
And Close Project is surely not for closing a dialog. Putting in icons is<br>
good, but only as long as we're using the right icons. If there's no<br>
proper icon please request one from the artist team via the techbase page.</blockquote><div>Sure, you are right.<br><br>In projectcontroller.cpp we are setting "window-close" for the "Close All Projects" action, do you think we should use this one or remove them both?<br>
<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
>              KAction* action = new KAction( i18n( "Remove Folder" ), this );<br>
> +            action->setIcon(KIcon("user-trash"));<br>
<br>
Not sure about that one either, what does dolphin use for remove<br>
folder/file?</blockquote><div>This one :)<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
>          }<br>
> -        else if(!hasTargets && item->executable())<br>
> +<br>
> +        if(!hasTargets && item->executable())<br>
<br>
Uhm, that seems unrelated, are you sure you wanted to remove the else?</blockquote><div>Ugh yes I was, it is correct (an item will be a target _and_ an executable). Sorry for not including that to the commit message.<br>
<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
Just because the message may never be received does not mean it is<br>
not worth sending.<br>
<br>
_______________________________________________<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</font></blockquote></div><br>