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