<table><tr><td style="">loh.tar updated this revision to Diff 53281.<br />loh.tar retitled this revision from "[RFC]SearchBar: Don't block GUI when enter incremental pattern" to "SearchBar: Don't block GUI when enter incremental pattern".<br />loh.tar edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-guqyejiqufek2sv/">(Show Details)</a><br />loh.tar added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19367">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Consider multi line pattern</li>
<li class="remarkup-list-item">Remove notes from Test Plan</li>
</ul>

<p>TODOs, but how?</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Fix failing autotest</li>
<li class="remarkup-list-item">Add autotest for feature?</li>
</ul></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">(No)Issues;<br />
- There may a range added where fromPos = toPos. Happens when start from first <br />
  or last position of the doc. For my taste is that better than to check every <br />
  time that very rare case<br />
<br />
Potential TODOs:<br />
- See code<br />
- Merge selectRange2 into indicateMatch<br />
- There is a status (KSqueezedTextLabel) set in indicateMatch, but I have never seen this hint. Tinker the UI file make it visible. Not so bad. Do anyone knows why this is still there? May that be better to use instead of the fancy in-view-hint which is sometimes annoying because it hides the text? Then Ctrl-H would only use fancy in-view-hint when no bar is visible. PowerUi would need such label too.<br />
</span></div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D19367?vs=52728&id=53281">https://phabricator.kde.org/D19367?vs=52728&id=53281</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19367">https://phabricator.kde.org/D19367</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/search/katesearchbar.cpp<br />
src/search/katesearchbar.h</div></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, dhaumann, cullmann<br /><strong>Cc: </strong>cullmann, kwrite-devel, kde-frameworks-devel, KTextEditor, gennad, domson, michaelh, ngraham, bruns, demsking, sars, dhaumann<br /></div>