D8154: Minor spell check interface fixes

[Christoph Roick]

croick added a comment.


  In https://phabricator.kde.org/D8154#152602, @dhaumann wrote:
  
  > 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?
  
  
  I didn't find one. I just discovered these yesterday and don't think anybody could investigate this with the spell check bar working only once (for whom and since when, I don't know. Bug 359682 dated back to 02-2016).

INLINE COMMENTS

> dhaumann wrote in spellcheckdialog.cpp:198
> Hm, could you explain why removing this line is correct?
> 
> Is it because in line 202 we continue with the replacementStartCursor anyway?

Yes, the m_currentSpellCheckRange is invalidated in any case, so there is no reason to set its range here.

REPOSITORY
  R39 KTextEditor

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

To: croick, #ktexteditor
Cc: dhaumann, #frameworks, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171006/2039c6c2/attachment.html>