D22276: Add an action to insert a non-indented newline

Christoph Cullmann noreply at phabricator.kde.org
Fri Jul 5 21:47:15 BST 2019


cullmann added inline comments.

INLINE COMMENTS

> dhaumann wrote in kateview.cpp:2881
> Do we really need this call? Also Don't we miss setting the cursor position? Incliding a begin/endEdit?

I assume this is just copied from the normal

void KTextEditor::ViewPrivate::keyReturn()
{

  doc()->newLine(this);
  m_viewInternal->iconBorder()->updateForCursorLineChange();
  m_viewInternal->updateView();

}

;=) In doubt I would just keep this as is and one can think separately if that is needed in both places.

REPOSITORY
  R39 KTextEditor

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

To: ahmadsamir, #ktexteditor, cullmann, dhaumann
Cc: bruns, mickaelbo, kde-frameworks-devel, kwrite-devel, LeGast00n, domson, michaelh, ngraham, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190705/c7b92a21/attachment.html>


More information about the KWrite-Devel mailing list