Review Request 111912: Introduce Sonnet::TextEditInstaller: easily add spellcheck support to a QTextEdit

[Kevin Ottens]

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



tier1/sonnet/src/ui/highlighter.cpp
<http://git.reviewboard.kde.org/r/111912/#comment27681>

    I still don't see what the changes in this file have to do with the rest of the patch. Could you explain it to me?
    
    Beside it looks odd that it turns a singleShot timer into potentially a recurring one (it's not created single shot and the calls to setSingleShot are scattered everywhere, there's a non-null probability of things going sour here).


- Kevin Ottens


On Aug. 8, 2013, 10:03 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111912/
> -----------------------------------------------------------
> 
> (Updated Aug. 8, 2013, 10:03 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Description
> -------
> 
> This patch introduces a new class: Sonnet::TextEditInstaller. It makes it easy to add spellcheck support to a QTextEdit.
> 
> Spellcheck support means two things:
> 1. Install Sonnet::Highlighter to highlight spelling error.
> 2. Intercept context menu to replace it with a list of suggestions when user right-clicks on a misspelled word.
> 
> Minimal usage is simple: create a new TextEditInstaller, passing it the QTextEdit as argument. The patch adds a test_textedit executable which demonstrates the class.
> 
> I am posting it early to get feedback on the API and the class name, I am not completely happy with either.
> 
> PS: This patch includes my plugin fixes [1], since it is useless without them.
> [1]: https://git.reviewboard.kde.org/r/111895/
> 
> 
> Diffs
> -----
> 
>   tier1/sonnet/src/ui/CMakeLists.txt 723d8f3 
>   tier1/sonnet/src/ui/highlighter.h c303db1 
>   tier1/sonnet/src/ui/highlighter.cpp 5c6a590 
>   tier1/sonnet/src/ui/spellcheckdecorator.h PRE-CREATION 
>   tier1/sonnet/src/ui/spellcheckdecorator.cpp PRE-CREATION 
>   tier1/sonnet/tests/CMakeLists.txt 6e0e450 
>   tier1/sonnet/tests/test_textedit.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111912/diff/
> 
> 
> Testing
> -------
> 
> Tested with test_textedit.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130809/beaab8f9/attachment.html>