<table><tr><td style="">xuetianweng 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/D25339">View Revision</a></tr></table><br /><div><div><p>Actually I was trying to use this approach in the patch because I was afraid that variable line height may need to estimate the whole document height to make scroll work correctly.</p>

<p>But during this patching experience I realized that the ktexteditor first line is always a whole line. So the renderer only need to care about the current page instead of whole document, which makes variable line height might not as hard as thought.</p>

<p>I won't have time recently to look into the variable line height though.</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/D25339">https://phabricator.kde.org/D25339</a></div></div><br /><div><strong>To: </strong>xuetianweng, KTextEditor, cullmann, dhaumann, Frameworks, rjvbb<br /><strong>Cc: </strong>ahmadsamir, brauch, sars, pshinjo, rjvbb, fakefred, anthonyfieroni, kde-frameworks-devel, kwrite-devel, rrosch, LeGast00n, cblack, domson, michaelh, ngraham, bruns, demsking, cullmann, dhaumann<br /></div>