Review Request: Make quickopen filter timeout depend on how many items need to be searched

Sven Brauch svenbrauch at gmx.de
Thu Apr 5 12:42:15 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104486/
-----------------------------------------------------------

(Updated April 5, 2012, 12:42 p.m.)


Review request for KDevelop.


Changes
-------

Remove the applyFilters(), it introduces a bug which prevents opening entries other than the first ;p


Description
-------

I find the recently introduced 150ms-timeout for searching quickopen very annoying, as I loved the instant feedback quickopen had before. It is clear that something like the timer is necessary for large projects, because the items cannot be searched in real-time; however, for smaller projects (and especially for the outline filter!) the additional delay is just plain unnecessary.

This patch adjusts the delay to the amount of items to be searched; it is 0 if the amount of items to be searched is less than 5000, and reaches its maximum of 150ms (the old default value) if there's more than 50.000 items. Those values are just random, and better suggestions are welcome.

An immediately noticeable positive effect of this patch (in every project) is that the outline widget now again always provides instant, smooth feedback.


Diffs (updated)
-----

  plugins/quickopen/quickopenmodel.h 3da9f1c 
  plugins/quickopen/quickopenmodel.cpp 6fd9c89 
  plugins/quickopen/quickopenplugin.cpp 91a9211 

Diff: http://git.reviewboard.kde.org/r/104486/diff/


Testing
-------


Thanks,

Sven Brauch

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120405/4eae8c73/attachment.html>


More information about the KDevelop-devel mailing list