Summary: | Pan Scrolling Jumps out of frames if the initial location is in a frame that can't be scrolled | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Weinstein <bweinstein> | ||||
Component: | WebCore Misc. | Assignee: | Brian Weinstein <bweinstein> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aroben, maxime.britto, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Windows XP | ||||||
Attachments: |
|
Description
Brian Weinstein
2009-12-10 16:33:03 PST
Additionally, we don't jump out of frames in the rest of the scrolling process, so it doesn't make sense for us to do it on the initial scroll. The regular wheel scroll doesn't trigger in that case ? The scroll wheel doesn't jump out of iFrames either. The bug here is that: 1) There's an iFrame that can scroll. 2) There's an iFrame that can't scroll. If you start pan scrolling in (1), it will scroll the iFrame, and then NOT jump out to the main document. If you start pan scrolling in (2), it will jump out of the document. That behavior is inconsistent and a bug. It wasn't always like this, but was a regression from r52008. Before r52008, it would think it could pan scroll, and show the icon even if there was nothing for it to scroll inside the iFrame. (In reply to comment #3) > The scroll wheel doesn't jump out of iFrames either. Do you mean "pan scroll" instead of "scroll wheel"? (In reply to comment #4) > (In reply to comment #3) > > The scroll wheel doesn't jump out of iFrames either. > > Do you mean "pan scroll" instead of "scroll wheel"? No, Maxime asked about the scroll wheel, so I was saying that the scroll wheel doesn't jump out of frames/iFrames either. Created attachment 44698 [details]
[PATCH] Fix
style-queue ran check-webkit-style on attachment 44698 [details] without any errors.
(In reply to comment #5) > (In reply to comment #4) > > (In reply to comment #3) > > > The scroll wheel doesn't jump out of iFrames either. > > > > Do you mean "pan scroll" instead of "scroll wheel"? > > No, Maxime asked about the scroll wheel, so I was saying that the scroll wheel > doesn't jump out of frames/iFrames either. This does not match the behavior I see in ToT Safari. Scrolling the mouse wheel within an iframe scrolls that iframe until it can't be scrolled anymore, then scrolls the iframe's parent frame until that frame can't be scrolled, and so on. Oh, I was using a bad test case that called preventDefault on mousewheel. Sorry. Comment on attachment 44698 [details]
[PATCH] Fix
This looks OK and probably fixes this bug. But the existing logic doesn’t seem exactly right to me. A renderer can be an element with fixed positioning, in which case it doesn't make sense to advance to its parent when deciding what to scroll.
r=me
Landed in r52012. |