<br><br><div class="gmail_quote">2009/9/17 Christoph Feck <span dir="ltr"><<a href="mailto:christoph@maxiom.de">christoph@maxiom.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
-----------------------------------------------------------<br>
This is an automatically generated e-mail. To reply, visit:<br>
</div><a href="http://reviewboard.kde.org/r/1622/#review2392" target="_blank">http://reviewboard.kde.org/r/1622/#review2392</a><br>
-----------------------------------------------------------<br>
<br>
<br>
The HBox can also be removed, as it only contains the LineEdit now.<br>
<br>
About the "Search..." string, do we need the dots here? At least, systemsettings does not have them.<br>
<font color="#888888"><br>
- Christoph<br>
</font><div class="im"><br>
<br>
On 2009-09-17 09:41:42, Alessandro Diaferia wrote:<br>
><br>
> -----------------------------------------------------------<br>
</div><div class="im">> This is an automatically generated e-mail. To reply, visit:<br>
> <a href="http://reviewboard.kde.org/r/1622/" target="_blank">http://reviewboard.kde.org/r/1622/</a><br>
> -----------------------------------------------------------<br>
><br>
</div><div class="im">> (Updated 2009-09-17 09:41:42)<br>
><br>
><br>
</div><div><div></div><div class="h5">> Review request for kdelibs.<br>
><br>
><br>
> Summary<br>
> -------<br>
><br>
> Imho using KLineEdit::setClickMessage is nicer and more common into KDE instead of an additional QLabel. I left the inheritance from QWidget as is since future additions would be made more easily but the widget could directly inherit from KLineEdit now.<br>

><br>
> Suggestions/insults are welcome :)<br>
><br>
> Cheers<br>
><br>
><br>
> Diffs<br>
> -----<br>
><br>
>   trunk/KDE/kdelibs/kdeui/itemviews/kfilterproxysearchline.cpp 1023670<br>
><br>
> Diff: <a href="http://reviewboard.kde.org/r/1622/diff" target="_blank">http://reviewboard.kde.org/r/1622/diff</a><br>
><br>
><br>
> Testing<br>
> -------<br>
><br>
><br>
> Thanks,<br>
><br>
> Alessandro<br>
><br>
><br>
<br>
</div></div></blockquote></div><br>I just used the dots looking at the nepomuk search line used in dolphin.. Wil remove them along with the KHBox include as well in the case they are not needed :)<br clear="all"><br>-- <br>
Alessandro Diaferia<br>KDE Developer<br>