<table><tr><td style="">cullmann 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/D4947" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Sorry for late response, thanks for ping!</p>

<p>First about the interface you added: I think that it is ok the way, beside for the enum values I would go for CamelCase like all other things and something like VisibleLine or RealLine, that is more KF/Qt style I would say.<br />
If others don't oppose, I would be in favor to merge that then.</p>

<p>For the folding interface: I actually didn't do any work on that since long (nor did Dominik I guess).<br />
Therefore, if you want to have it exposed, feel free too enhance it (might just be non-virtual function addon to view, too, later).<br />
But I think that should go in a second differential item after this stuff is merged.</p>

<p>Actually looking forward to use a new diff tool for testing :P</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/D4947" rel="noreferrer">https://phabricator.kde.org/D4947</a></div></div><br /><div><strong>To: </strong>jsalatas, Frameworks, tfry, mwolff, KTextEditor<br /><strong>Cc: </strong>cullmann, dhaumann, anthonyfieroni, mwolff, kwrite-devel<br /></div>