> On April 19, 2012, 3:58 p.m., Andrea Diamantini wrote: > > I don't understand these changes. Why aren't we doing inline spell check on > > strings? I think that it is the unique way to let people use spellcheck. > > Just check on keyPressEvent if you are on an INPUT/TEXTAREA element on > > content editable text and if so, check words before spaces.
I guess we had already discussed this issue. Support for inline spell check will require us to modify QtWebkit sources. I just followed the way KWebKitPart does it. I am not sure how helpful your current suggestion would be in providing full fledged inline spell check. - Siteshwar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104647/#review12658 ----------------------------------------------------------- On April 17, 2012, 4:59 p.m., Siteshwar Vashisht wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104647/ > ----------------------------------------------------------- > > (Updated April 17, 2012, 4:59 p.m.) > > > Review request for rekonq. > > > Description > ------- > > Adding support for spellcheck in Rekonq using Sonnet. Right now spellcheck is > not inline and you need to right click text box and select "Spellcheck" to > perform checks. > > > Diffs > ----- > > src/webview.h 3ae9b7e > src/webview.cpp d2e351c > > Diff: http://git.reviewboard.kde.org/r/104647/diff/ > > > Testing > ------- > > Yes. > > > Thanks, > > Siteshwar Vashisht > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
