<table><tr><td style="">loh.tar updated this revision to Diff 50104.<br />loh.tar edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-vktwil2c4fjxljf/">(Show Details)</a><br />loh.tar set the repository for this revision to R39 KTextEditor.<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/D17949">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Update 'What's This' hint to reflect new behaviour and to be more precise</li>
<li class="remarkup-list-item">Update 'What's This' hint of dyn wrap too to be more precise</li>
<li class="remarkup-list-item">Update/clear Summary</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">This patch address not all aspects but should be a step in the right direction.<br />
<br />
</span>- Don't wrap entire document when nothing is selected but wrap current line<br />
- Join paragraph before wrap to avoid odd result<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
<br />
TODO Update "What this" hint</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">This patch may conflict with D17128, but because that need anyway some more love</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Update 'What's This' hint to reflect new behaviour</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">I like to offer this one now, to fetch some feedback.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Update 'What's This' hint of dyn wrap too to be more precise</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">As always fresh and poor tested. But I think it is a handy improvement and seems</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Old Hints</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">to work nicely. As you know I have often mention not so lovely things about the</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F6563805}</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">wrap feature and there are a couple of bug reports.<br />
<br />
This patch address not all aspects but should be a step in the right direction.<br />
</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">New Hints<br />
{F6563807}</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D17949?vs=50082&id=50104">https://phabricator.kde.org/D17949?vs=50082&id=50104</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17949">https://phabricator.kde.org/D17949</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/view/kateview.cpp<br />
src/view/kateview.h</div></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, mwolff<br /><strong>Cc: </strong>dhaumann, cullmann, mwolff, kwrite-devel, kde-frameworks-devel, KTextEditor, hase, michaelh, ngraham, bruns, demsking, sars<br /></div>