On Monday, 2013-08-05, Aurélien Gâteau wrote: > I pushed a first version of the patch here: > > https://codereview.qt-project.org/#change,62119 > > But people are not really sold to the idea so far. > > I am afraid adding this default syntax highlighter is not going to be > enough to get proper spell checking anyway: we need to extend the context > menu to provide suggestions for mistyped words, something which cannot be > provided by the syntax highlighter.
One problem is that the syntax highlighter is only one part of spell checking. For me the idea of a "default highlighter" makes no sense as it is unclear what it would highlight, i.e. which syntax rules it would use, etc. If we want spell check integration depending on platform then that is what we should push for. > Not sure what is the best course of action there, except making > spell-checking a feature provided by frameworks, which developers would > have to explicitly enable on their QTextEdit. This would also address one > of the comments I got from the review which asked how to disable the > default highlighter. Hmm. Using the sonnet framework is a contious choice anyway, so it shouldn't be a problem to let the app developer use it. Making it easy to hook it up with text input widgets would be a good feature in any case. Cheers, Kevin -- Kevin Krammer, KDE developer, xdg-utils developer KDE user support, developer mentoring
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
