<table><tr><td style="">loh.tar updated this revision to Diff 56444.<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/D20606">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Add docu to header, not full happy with</li>
<li class="remarkup-list-item">Add desired adjustment checks</li>
<li class="remarkup-list-item">Make toggleFoldingInRange more action friendly<ul class="remarkup-list">
<li class="remarkup-list-item">Add first try to unfold range itself into toggleFoldingInRange</li>
<li class="remarkup-list-item">Be a smart ass, minimize if nesting in mouseReleaseEvent</li>
</ul></li>
<li class="remarkup-list-item">Fix to eager toggleFoldingInRange, don't fold else part when written on one line } else {</li>
</ul>

<p>Regading toggleFoldingInRange</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The return value is not needed anymore and will only in rare cases say "false"</li>
<li class="remarkup-list-item">Should now be useable as slot, so remove the return value?</li>
<li class="remarkup-list-item">Better name idea?</li>
</ul></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D20606?vs=56375&id=56444">https://phabricator.kde.org/D20606?vs=56375&id=56444</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20606">https://phabricator.kde.org/D20606</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/view/kateviewhelpers.cpp<br />
src/view/kateviewhelpers.h<br />
src/view/kateviewinternal.cpp</div></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, cullmann, ngraham, dhaumann<br /><strong>Cc: </strong>mwolff, kwrite-devel, kde-frameworks-devel, KTextEditor, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>