----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111912/#review37673 -----------------------------------------------------------
Ship it! Ship It! - Kevin Ottens On Aug. 13, 2013, 10:25 a.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. 13, 2013, 10:25 a.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/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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel