[Kde-pim] Review Request 120395: Removed the minimum string length from Kontact quick search

Wyatt Childers wchilders at nearce.com
Fri Dec 12 07:47:31 GMT 2014


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

(Updated Dec. 12, 2014, 7:47 a.m.)


Status
------

This change has been discarded.


Review request for KDEPIM.


Repository: kdepim


Description
-------

This fixes an issue where short searches left the application feeling unresponsive. One such example is in the case where you want to search for an abreviation such as "ASL", KMail formerly wouldn't respond. This also fixes a problem where you would search for "ASL " and then remove one character leaving "ASL", the search would then be stuck on "ASL" even if you then after removed the characters, unless you either added the space back then held down backspace, or used the clear button.


Diffs
-----

  messagelist/core/quicksearchline.cpp 596b933eba00b595033b0c0cd57848b2e59ebfd1 

Diff: https://git.reviewboard.kde.org/r/120395/diff/


Testing
-------

I've tested KMail's quick search with a variety of character lengths, everything functions as it did before, but simply fires with smaller searches.


Thanks,

Wyatt Childers

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list