<table><tr><td style="">thomassc 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;"><p>We trade one bug for another. Which one is worse?</p></blockquote>

<p>Without having had a detailed look, it seems to me like the problem reported by <a href="https://phabricator.kde.org/p/rjvbb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rjvbb</a> might be caused by the removal of the workaround with the second drawing passes in this commit (the changes in src/view/kateviewinternal.cpp), which probably results in more issues with oversized characters. This is independent from the other change (in src/render/katerenderer.cpp), which adapts the line height and fixes the bug with the bottom of lines being cut off. So, one might try to revert the removal of the second drawing passes, but keep the change to the line heights.</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>