Bug 189907 - Make Element.scrollIntoView properly work with nested scroll boxes and smooth behavior
Summary: Make Element.scrollIntoView properly work with nested scroll boxes and smooth...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: UI Events (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: https://drafts.csswg.org/cssom-view/#...
Keywords: InRadar
Depends on: 188043
  Show dependency treegraph
Reported: 2018-09-23 21:40 PDT by Frédéric Wang (:fredw)
Modified: 2021-12-16 07:40 PST (History)
5 users (show)

See Also:

Testcase (1.72 KB, text/html)
2021-12-15 16:46 PST, Simon Fraser (smfr)
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang (:fredw) 2018-09-23 21:40:30 PDT
Follow-up of bug 188043. This might need a bit more work so I'm moving that part out of the main bug.

Basically, the patch in bug 188043 naively follows https://drafts.csswg.org/cssom-view/#scroll-an-element-into-view but does not wait for smooth scroll of step 12 to complete, possibly resulting in wrong final positions. It's not clear we actually want to align on the current spec anyway, Gecko and Chromium use different algorithm. For details, see https://github.com/w3c/csswg-drafts/issues/3127

The WPT test for this is http://w3c-test.org/css/cssom-view/scroll-behavior-scrollintoview-nested.html
Comment 1 Radar WebKit Bug Importer 2021-08-24 16:12:30 PDT
Comment 2 Simon Fraser (smfr) 2021-12-15 16:46:38 PST
Created attachment 447299 [details]

Seems to work OK looking at the attached testcase
Comment 3 Rob Buis 2021-12-16 07:40:13 PST
Possibly I fixed this, anyway maybe Fred can double-check.