> 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.
> 
> Siteshwar Vashisht wrote:
>     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.
> 
> Lindsay Mathieson wrote:
>     As Siteshwar said, it can only be done in the QtWebKit sources and has 
> been, but is very unlikely to make its way into KDE. See this discussion here:
>     
>       https://lists.webkit.org/pipermail/webkit-qt/2012-February/002429.html

Bah... IMHO, I see what's needed just mixed in this code. Can I please try in 
this weekend to follow my broken mind before considering this?


- Andrea


-----------------------------------------------------------
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

Reply via email to