<table><tr><td style="">dhaumann 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/D12271">View Revision</a></tr></table><br /><div><div><p>The behavior that is proposed here was like Kate behaved before, and as result we got bug reports that not all trailing spaces were removed. So we removed this.</p>

<p>I can see that when saving very often, this may be annoying. On the other hand, the current behavior is very deterministic: Remove trailing spaces definitely should remove ALL trailing spaces without any exceptions.</p>

<p>As such: -2 for this patch - sorry, but we are turning in circles otherwise, changing behavior forth and back and forth and back.</p>

<p>A config option would work, but I am against it, since in my opinion this option belongs to a category that just bloat up our already complex config dialogs.</p>

<p>A by far better fix would be to allow the cursor to be placed behind the last text column:</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);"> do_work(); |
<---------></pre></div>

<p>Note, the length of the line does not include the space after saving, still the cursor keeps its position.</p>

<p>But here comes the challenge: Currently, Kate does not allow the cursor to be positioned behind the last valid cursor position in normal editing mode. It does work in block-selection mode, though.<br />
So the real / true fix would be much better, but will also be much harder.</p>

<p>But with that background in mind, I do not think we should add poor workarounds that do not fix the real problem, sorry. So I'd reject this patch, and vote with -1 against an option, and with +1 to a proper fix that would allow the cursor position in this case to be behind the last column.</p></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/D12271">https://phabricator.kde.org/D12271</a></div></div><br /><div><strong>To: </strong>sraizada, KTextEditor<br /><strong>Cc: </strong>dhaumann, anthonyfieroni, KTextEditor, Frameworks, michaelh, kevinapavew, ngraham, bruns, demsking, head7, cullmann, kfunk, sars<br /></div>