[webkit-reviews] review denied: [Bug 26934] [Qt] It would be useful to have methods for scrolling QWebElement : [Attachment 32197] Proposed patch, for adding scrolling to QWebElement

[bugzilla-daemon at webkit.org]

Adam Treat <treat at kde.org> has denied Joseph Ligman <joseph.ligman at nokia.com>'s
request for review:
Bug 26934: [Qt] It would be useful to have methods for scrolling QWebElement
https://bugs.webkit.org/show_bug.cgi?id=26934

Attachment 32197: Proposed patch, for adding scrolling to QWebElement
https://bugs.webkit.org/attachment.cgi?id=32197&action=review

------- Additional Comments from Adam Treat <treat at kde.org>
And how would a client use this exactly?  I'm not convinced that it is
necessary.  At Torch we have a client that uses QtWebKit that has automatic
panning on a touch screen.  We didn't need this API to accomplish it.

More than happy to have this if a real use-case can be demonstrated.  r- to get
it out of review queue until such use case is understood and because of the
concerns with scrollByPixel API from other comments.