Why don't we just get rid of the activated signal and have the applet determine whether to open the URL or not? In other words, have the applet connect to clicked if it's in the panel or KGlobalSettings says to otherwise connect to doubleclick. It seems that activated is rather presumptuous since the Icon doesn't know anything about the context.<br>
Chris<br><br><div class="gmail_quote">On Feb 10, 2008 9:41 AM, Marco Martin <<a href="mailto:notmart@gmail.com">notmart@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>-----------------------------------------------------------<br>This is an automatically generated e-mail. To reply, visit:<br><a href="http://matt.rogers.name/r/100/" target="_blank">http://matt.rogers.name/r/100/</a><br>
-----------------------------------------------------------<br><br>Review request for Plasma.<br><br><br>Summary<br>-------<br><br>Ok, this change is pretty trivial but changes the behaviour, so... :)<br>when an icon applet is is the panel always activate it with the single click regardless of single/double click settings, don't think it makes much sense to activate them with double click, because i have serious problem to consider the panel a file manager view :)<br>
and maybe when/if the desktop folder will be visualized by its own plasmoid i think using always single click even in the desktop would make sense, but for now i would go single or double in the desktop and single in the panel<br>
<br><br>Diffs<br>-----<br><br> /trunk/KDE/kdebase/workspace/plasma/applets/icon/icon.cpp<br><br>Diff: <a href="http://matt.rogers.name/r/100/diff" target="_blank">http://matt.rogers.name/r/100/diff</a><br><br><br>Testing<br>
-------<br><br><br>Thanks,<br><br>Marco<br><br>_______________________________________________<br>Panel-devel mailing list<br><a href="mailto:Panel-devel@kde.org">Panel-devel@kde.org</a><br><a href="https://mail.kde.org/mailman/listinfo/panel-devel" target="_blank">https://mail.kde.org/mailman/listinfo/panel-devel</a><br>
</blockquote></div><br>