Review Request: Make KFilterProxySearchLine look more like KDEs searchlines

Christoph Feck christoph at maxiom.de
Thu Sep 17 19:22:59 BST 2009


You are right, you cannot remove it.

I had the impression the widget was already part of a vertical layout 
(lineedit + listview)...

Regarding the "Search..." vs. "Search", we should ask HIG/usability?

Sorry for the confusion.
Christoph

On Thursday 17 September 2009 20:02:31 Alessandro Diaferia wrote:
> 2009/9/17 Alessandro Diaferia <alediaferia at gmail.com>
> 
> > 2009/9/17 Christoph Feck <christoph at maxiom.de>
> >
> >> -----------------------------------------------------------
> >> This is an automatically generated e-mail. To reply, visit:
> >> http://reviewboard.kde.org/r/1622/#review2392
> >> -----------------------------------------------------------
> >>
> >>
> >> The HBox can also be removed, as it only contains the LineEdit now.
> >>
> >> About the "Search..." string, do we need the dots here? At least,
> >> systemsettings does not have them.
> >>
> >> - Christoph
> >>
> >> On 2009-09-17 09:41:42, Alessandro Diaferia wrote:
> >> > -----------------------------------------------------------
> >> > This is an automatically generated e-mail. To reply, visit:
> >> > http://reviewboard.kde.org/r/1622/
> >> > -----------------------------------------------------------
> >> >
> >> > (Updated 2009-09-17 09:41:42)
> >> >
> >> >
> >> > Review request for kdelibs.
> >> >
> >> >
> >> > Summary
> >> > -------
> >> >
> >> > 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.
> >>
> >> > Suggestions/insults are welcome :)
> >> >
> >> > Cheers
> >> >
> >> >
> >> > Diffs
> >> > -----
> >> >
> >> >   trunk/KDE/kdelibs/kdeui/itemviews/kfilterproxysearchline.cpp 1023670
> >> >
> >> > Diff: http://reviewboard.kde.org/r/1622/diff
> >> >
> >> >
> >> > Testing
> >> > -------
> >> >
> >> >
> >> > Thanks,
> >> >
> >> > Alessandro
> >
> > 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 :)
> >
> > --
> > Alessandro Diaferia
> > KDE Developer
> 
> Ok I misunderstood. How can I remove the QHBoxLayout keeping the lineedit
> still full size?
> 




More information about the kde-core-devel mailing list