Bug 40569 - [Qt] qtwebkit_webframe_scrollRecursively is private
Summary: [Qt] qtwebkit_webframe_scrollRecursively is private
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on: 49737
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-14 07:24 PDT by nokiabugz
Modified: 2013-04-12 11:29 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description nokiabugz 2010-06-14 07:24:23 PDT
This API call needs to be public in order to be able to implement frames or blocks with an overflow in a browser application.
Comment 1 Antonio Gomes 2010-06-30 11:29:56 PDT
fwiw, in bug 39217 I introduced an additional 'starting node' parameter to EventHandler::scrollRecursively and scrollOverflow that can/should be used in the body implementation of this method.
Comment 2 Antonio Gomes 2010-07-16 13:51:04 PDT
It turned out to be useful to the WRT guys as well.
Comment 4 Benjamin Poulain 2010-11-18 09:43:59 PST
Thinking about it, two behaviors are possible regarding to the scroll distance.

Let's say we scroll 100px on a div with overflow, where the scroll span left is 40px:
1) scroll 40px and ignore the 60px left from the call
2) scroll 40px on the div, and then 60px on the parent frame

In my opinion, (1) is the correct for touch inteface with inner scrolling. I can see (2) being useful when scrolling programatically.

I lower the priority of this task since there are more urgent thing to do and Joseph does not seem to follow up on this problem.
Comment 5 Yael 2010-11-18 11:24:41 PST
(In reply to comment #4)
> Thinking about it, two behaviors are possible regarding to the scroll distance.
> 
> Let's say we scroll 100px on a div with overflow, where the scroll span left is 40px:
> 1) scroll 40px and ignore the 60px left from the call
> 2) scroll 40px on the div, and then 60px on the parent frame
> 
> In my opinion, (1) is the correct for touch inteface with inner scrolling. I can see (2) being useful when scrolling programatically.
> 
> I lower the priority of this task since there are more urgent thing to do and Joseph does not seem to follow up on this problem.

From my experience with Spatial Navigation, I agree with you, Bengamin. 
(1) Seem to be a better approach, because with (2), we sometime get premature scrolling of the parent container.

Joe left Nokia, so I doubt that he will continue working on this bug :-)
Comment 6 Antonio Gomes 2013-04-12 11:29:55 PDT
WONTFIX, likely