> On Aug. 3, 2011, 11:54 a.m., Pierre Rossi wrote: > > src/webview.cpp, line 484 > > <http://git.reviewboard.kde.org/r/102192/diff/1/?file=30658#file30658line484> > > > > You probably want to make sure that you're not modifying elements not > > editable for the user. See: > > > > http://doc.qt.nokia.com/latest/qwebhittestresult.html#isContentEditable > > > > Also, if you're gonna use QWebElement, then you probably could just use > > setInnerXml or appendInside that sounds more robust and secure than > > evaluating JavaScript that could contain potentially anything (even evil > > stuff).
I already check it. (else if (isEditable) part.) Also, I couldn't get setInnerXml or appendInside work. That's why chose evaluating JavaScript. :) - Furkan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102192/#review5342 ----------------------------------------------------------- On Aug. 3, 2011, 10:39 a.m., Furkan Üzümcü wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102192/ > ----------------------------------------------------------- > > (Updated Aug. 3, 2011, 10:39 a.m.) > > > Review request for rekonq. > > > Summary > ------- > > You couldn't drop text into a text field on a web page. Now you can. > > > Diffs > ----- > > src/webview.cpp f31d234 > > Diff: http://git.reviewboard.kde.org/r/102192/diff > > > Testing > ------- > > Compiles and works. > > > Thanks, > > Furkan > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
