----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111912/ -----------------------------------------------------------
(Updated Aug. 7, 2013, 1:05 p.m.) Review request for KDE Frameworks. Changes ------- Drop static install() method and let QTextEdit handle the normal context menu, as suggested by Kevin. 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 (updated) ----- tier1/sonnet/src/core/loader.cpp 035cf64 tier1/sonnet/src/plugins/aspell/CMakeLists.txt 70f06d1 tier1/sonnet/src/ui/CMakeLists.txt 723d8f3 tier1/sonnet/src/ui/highlighter.h c303db1 tier1/sonnet/src/ui/highlighter.cpp e8ba703 tier1/sonnet/src/ui/texteditinstaller.h PRE-CREATION tier1/sonnet/src/ui/texteditinstaller.cpp PRE-CREATION tier1/sonnet/tests/CMakeLists.txt 90b3ab0 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