Review Request 114158: Use Sonnet::SpellCheckDecorator internally in KTextEdit

David Edmundson david at davidedmundson.co.uk
Wed Nov 27 18:02:21 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114158/
-----------------------------------------------------------

(Updated Nov. 27, 2013, 6:02 p.m.)


Review request for KDE Frameworks.


Summary (updated)
-----------------

Use Sonnet::SpellCheckDecorator internally in KTextEdit


Repository: kdelibs


Description
-------

Use Sonnet::Decorator internally in KTextEdit

In order to keep api compatibility we create a custom Sonnet::Decorator and proxy the virtual methods.
The spellcheckdecorator event filter intercepts the context menu for mispelled words our implementation will only be called for all other context menu events.


Diffs
-----

  tier3/ktextwidgets/src/widgets/ktextedit.h 2086607 
  tier3/ktextwidgets/src/widgets/ktextedit.cpp f5d0331 

Diff: http://git.reviewboard.kde.org/r/114158/diff/


Testing
-------

I still have something wrong with my spell checker in KF5: 
Aurélien tested for me, there appear to be some issues, such as not showing a context menu until a language is set, but this seems to be independent from this patch.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131127/5edd26c8/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list