D4947: [KTextEditor] Expose additional internal View's functionality to the public API

[John Salatas]

jsalatas added a comment.


  In https://phabricator.kde.org/D4947#95893, @cullmann wrote:
  
  > Sorry for late response, thanks for ping!
  
  
  No worries :)
  
  > 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.
  >  If others don't oppose, I would be in favor to merge that then.
  
  Done
  
  > For the folding interface: I actually didn't do any work on that since long (nor did Dominik I guess).
  >  Therefore, if you want to have it exposed, feel free too enhance it (might just be non-virtual function addon to view, too, later).
  >  But I think that should go in a second differential item after this stuff is merged.
  
  OK, I guess I can do it. Will post another differential review when it's done. Hopefully soon :\

REPOSITORY
  R39 KTextEditor

REVISION DETAIL
  https://phabricator.kde.org/D4947

To: jsalatas, #frameworks, tfry, mwolff, #ktexteditor
Cc: cullmann, dhaumann, anthonyfieroni, mwolff, kwrite-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170318/b3f7fd91/attachment.html>