<table><tr><td style="">rjvbb 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/D5139" rel="noreferrer">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>Note that Kate uses underlines for indicating spelling errors, kdev-python also uses underlines (yellow ones) to indicate style/formatting issues. So we're in line.</p></blockquote>

<p>And I still disagree that this is a place where you can use underlining to indicate overflow. I'm not going to repeat myself why.</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>Please, René, it takes lot of energy to do these kind of reviews, it would be appreciated if you'd sometimes just accept the recommendations from core developers who've been working on KDevelop for so many years already.</p></blockquote>

<p>I do and wish I could do so more often. Already I don't both anymore with most changes that I feel are justified or needed because I know my efforts will just be driven into the ground. The diff-context-lines patch is another good example. I've listened to all objections and found a way to accommodate the GUI addition without adding visible widgets but apparently I should have concluded that all further resistance was futile.</p>

<p>What I don't do is taking orders to implement things I cannot stand behind.</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>This patch could've been pushed (in a revised form) weeks ago</p></blockquote>

<p>Yes, we could have found a middle ground that doesn't involve using font or text attributes at all (like adding 1 or 2 margin lines).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5139" rel="noreferrer">https://phabricator.kde.org/D5139</a></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, mwolff, kfunk<br /><strong>Cc: </strong>kfunk, flherne, mwolff, kdevelop-devel, KDevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>