<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/D19283">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;"><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);">Just load the XML file from bug 404713.
Before this changes here, you did overpaint the next line randomly with the "oversized one", now you "cut" the oversized line.</pre></div></blockquote>

<p>I don't know where to look specifically in that huge file. I do notice that with the partially reverted commit some lines are very close to one another, possibly even certain glyphs being superimposed, and on certain of those lines the cursor gets decorated by what looks to be a '>'.</p>

<p>What I do NOT see is lines being painted over by whitespace, which is worse in terms of readability.</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>Bug 328837 - Add configurable line height to katepart</p></blockquote>

<p>Apparently,</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>There's a problem, though: selection gets inconsistent.</p></blockquote>

<p>I haven't checked if that's the case (nor if the patch still applies).</p>

<p>The idea is good though, and I'm a bit surpised that there could be problems with it given that KTE already determines the lineheight to use (and changing the way that's hardcoded doesn't lead to selection problems).</p>

<p>You could even make that lineheight scale factor a per-document setting (I could easily imagine someone wanting lines to be more separated in code than in prose, for instance).</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/D19283">https://phabricator.kde.org/D19283</a></div></div><br /><div><strong>To: </strong>cullmann, dhaumann<br /><strong>Cc: </strong>rjvbb, loh.tar, thomassc, kwrite-devel, kde-frameworks-devel, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>