shortcuts for type ahead find and find previous + find for find previous
David Faure
faure at kde.org
Tue Apr 5 16:54:08 BST 2005
On Tuesday 05 April 2005 14:53, Lubos Lunak wrote:
>
> Hello,
>
> I'd like to commit the attached patches that make the shortcuts for type
> ahead find configurable (because / and ' are not very usable e.g. in
> lineedits) and add action for 'Find Previous' (#87130). Additionally the find
> previous action needs a couple of fixes for the search code. Ok to commit?
> The Edit menu gets even a bit bigger with 3 more items, but I think it's
> still ok.
- bool findTextNext();
+ bool findTextNext( bool reverse = false );
This looks like a BIC to me - the method is public.
+ int tmp2 = d->m_findPosEnd;
+ d->m_findPosEnd = d->m_findPosStart;
+ d->m_findPosStart = tmp2;
qSwap( d->m_findPosEnd, d->m_findPosStart ) ?
Thanks for the fixes for "find previous" - tricky stuff indeed.
--
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
More information about the kfm-devel
mailing list