<table><tr><td style="">loh.tar created this revision.<br />loh.tar added a reviewer: KTextEditor.<br />Herald added projects: Kate, Frameworks.<br />Herald added subscribers: kde-frameworks-devel, kwrite-devel.<br />loh.tar requested review of this revision.
</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><strong>REVISION SUMMARY</strong><div><p>When the document is very big and there is no early match while you enter the <br />
search pattern the GUI hangs as long the search is running after each key <br />
stroke. This patch break the document in smaller chunks to search to return fast <br />
enough to the event loop to update user input.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Disable next/prev buttons when not match</li>
</ul>

<p>BUG:339337</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>(No)Issues;</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">There may a range added where fromPos = toPos. Happens when start from first or last position of the doc. For my taste is that better than to check every time that very rare case</li>
</ul>

<p>Potential TODOs:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">See code</li>
<li class="remarkup-list-item">Merge selectRange2 into indicateMatch</li>
<li class="remarkup-list-item">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.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></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<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, KTextEditor, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>