<table><tr><td style="">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/D17128">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I am not sure if the normal join lines should try to be that clever.</p></blockquote>
<p>But consider the "auto comment" feature after you hit "Return", it's also somehow "special clever" but very useful. This tries to add the corresponding functionality. Well, the true opposite would be when you hit "Del" at the on of a line. Yeah, sounds good, will look at that too but it is (almost, see next note) independent from this aim.</p>
<p>I'm currently digging into the wrap functionally which is broken since a decade <a href="https://bugs.kde.org/show_bug.cgi?id=135737" class="remarkup-link" target="_blank" rel="noreferrer">BUG135737</a>. Sometimes it works already pretty well (with normal text) but sometimes it drives nuts (without selection in a code file). Very strange. Why I mention this? There I have follow a suggestion posted in that bug report, just o trigger a "Return" input at the right place. Now, while writing this text I came up with this "Del" idea. So, it may a good idea to re-design this patch (?)</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>It would make more sense to have an extra action for such a "smart" variant in my eyes.</p></blockquote>
<p>An extra action looks to me at first sight a little overdone, but an option how join behaves may appropriate. Let me know where to place that in the Edit Options page if you (and other) like that idea</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/D17128">https://phabricator.kde.org/D17128</a></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor<br /><strong>Cc: </strong>cullmann, kwrite-devel, kde-frameworks-devel, KTextEditor, michaelh, ngraham, bruns, demsking, head7, kfunk, sars, dhaumann<br /></div>