dhaumann added a comment.
Looks already quite good. Btw, are there any bug reports in the Kate or frameworks-ktexteditor product on bugs.kde.org that you fix here? INLINE COMMENTS > spellcheckdialog.cpp:198 > > - > m_currentSpellCheckRange.setRange(KTextEditor::Range(replacementStartCursor, > m_currentSpellCheckRange.end())); > // we have to be careful here: due to static word wrapping the text > might change in addition to simply Hm, could you explain why removing this line is correct? Is it because in line 202 we continue with the replacementStartCursor anyway? REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D8154 To: croick, #ktexteditor Cc: dhaumann, #frameworks, cullmann, sars
