<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);">I would rather keep the current behavior and let people use sane fonts if the want unicode.</pre></div></blockquote>

<p>That's not current acceptable IMHO, not with the cumbersome way of selecting the font.</p>

<p>If proper document display depends on "user picking a sane font", they should have a drop-down list where they can pick one quickly, and with a clear and easy way to make this a temporary or a document-specific choice.</p>

<p>Also, while it's clear that you cannot expect everything to work properly with any font, accepting that the user just has to pick the right font to avoid lineheight issues is a slippery slope that I'd fear could easily lead to hardwiring a limited selection of typefaces just to avoid bug reports. That would be fine (AFAIAC) for Kate, but not for the framework that provides the editor component for more complex applications like KDevelop.</p>

<p>Rather, an effort should be made to implement support for variable lineheights (GSoC? junior job?)</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>how should the user set it to the "right value" if it can differ per line?</p></blockquote>

<p>The same way they can pick a "sane font"? FWIW, I'm using Ubuntu Mono, that's supposed to be pretty complete and should be sane too.</p>

<p>The "right value" here would be a value that works for all lines = the tallest lineheight that gives proper line separation everywhere. My guess would be that you will already catch most situations with a choice of 1.5x and 2x .</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>And I don't see a benefit in supporting that at all</p></blockquote>

<p>There are situations where I would welcome the possibility to increase line spacing on the fly, for improved readability - I'm certain I'm not the only one.</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>