[Kde-pim] Review Request 120394: Removed the minimum characters requirement for quick search

Wyatt Childers darkarc at skelril.com
Sat Sep 27 20:25:18 BST 2014


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

(Updated Sept. 27, 2014, 7:25 p.m.)


Status
------

This change has been discarded.


Review request for KDEPIM.


Repository: ktp-kde


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
-----


Diff: https://git.reviewboard.kde.org/r/120394/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