WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
181002
overflow:scroll is scrollable when the mouse is over the border, unlike other browsers
https://bugs.webkit.org/show_bug.cgi?id=181002
Summary
overflow:scroll is scrollable when the mouse is over the border, unlike other...
Simon Fraser (smfr)
Reported
2017-12-19 14:39:29 PST
When the mouse is over the border of an overflow:scroll, the content is scrollable. This seems wrong and doesn't match other browsers, or iframes.
Attachments
Testcase
(3.17 KB, text/html)
2017-12-19 14:41 PST
,
Simon Fraser (smfr)
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1
2017-12-19 14:41:00 PST
Created
attachment 329832
[details]
Testcase
Simon Fraser (smfr)
Comment 2
2017-12-19 15:01:44 PST
https://codepen.io/anon/pen/ppyQgY
Frédéric Wang (:fredw)
Comment 3
2018-01-10 02:48:00 PST
This quick hack provides the correct behavior for the test case: diff --git a/Source/WebCore/rendering/RenderBlock.cpp b/Source/WebCore/rendering/RenderBlock.cpp index f2ca0e134c9..781b12343c1 100644 --- a/Source/WebCore/rendering/RenderBlock.cpp +++ b/Source/WebCore/rendering/RenderBlock.cpp @@ -2238,3 +2238,4 @@ bool RenderBlock::nodeAtPoint(const HitTestRequest& request, HitTestResult& resu if (hitTestAction == HitTestBlockBackground || hitTestAction == HitTestChildBlockBackground) { - LayoutRect boundsRect(adjustedLocation, size()); + LayoutRect boundsRect = paddingBoxRect(); + boundsRect.moveBy(adjustedLocation); if (visibleToHitTesting() && locationInContainer.intersects(boundsRect)) { However, I'm not sure that's what we want. Do you have any suggestion?
Simon Fraser (smfr)
Comment 4
2018-01-10 11:10:35 PST
Does that affect click behavior as well?
Frédéric Wang (:fredw)
Comment 5
2018-01-11 00:49:23 PST
(In reply to Simon Fraser (smfr) from
comment #4
)
> Does that affect click behavior as well?
I suspect it does, that's my doubt. My idea was to pass some specific info via the HitAction param in the case of overflow scrolling, so we can distinguish that case. Anyway, I just wanted to check if you already thought about something, I'll try and debug more after I have a patch for review in
bug 173833
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug