<table><tr><td style="">loh.tar updated this revision to Diff 52158.<br />loh.tar edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-suxiictelxy3uvq/">(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/D19131">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Do not wrap in some cases with block selection</li>
<li class="remarkup-list-item">Auto wrap once wrapped line again also when static wrap is disabled</li>
</ul>

<p>Autotest looks BTW not so bad</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">The following tests FAILED:
62 - vimode_modes (Failed)
65 - vimode_keys (Failed)</pre></div></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>- The new wrapLine function is very close to old code but some variables are renamed<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Two main i</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">I</span>ssues which needs to be fixed<br />
- Based on TextLine::isAutoWrapped. This info gets lost when you undo some change<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
- There may cases where the user wants to remove the isAutoWrapped flag</span><br />
- ~~The cursor is sometimes misplaced, see pic where the x was DELeted and should be now in line 1 in the middle of the bbbb~~ Hint: It was not DEL key but BACKSPACE<div style="padding: 8px 0;">...</div></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/D19131?vs=52072&id=52158">https://phabricator.kde.org/D19131?vs=52072&id=52158</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19131">https://phabricator.kde.org/D19131</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/document/katedocument.cpp<br />
src/document/katedocument.h</div></div></div><br /><div><strong>To: </strong>loh.tar, cullmann, KTextEditor<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, KTextEditor, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>