New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 594873 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 558575



Sign in to add a comment

Scroll Anchoring: interaction with CSS positioning

Project Member Reported by kenjibaheux@chromium.org, Mar 15 2016

Issue description

Scroll Anchoring is an intervention whose intent is to mitigate reflows that unnecessarily impact the user experience.

We should confirm if it interacts well with CSS positioning.
 
Including edge cases like: absolute-positioned descendants of static-positioned scrollers.

Comment 2 by skobes@chromium.org, Mar 15 2016

Owner: skobes@chromium.org
Status: Started (was: Unconfirmed)
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/54502dc10d4a4cf3be8d9fb89db3d2927c7bbaf9

commit 54502dc10d4a4cf3be8d9fb89db3d2927c7bbaf9
Author: skobes <skobes@chromium.org>
Date: Fri Mar 25 21:22:23 2016

Handle position:absolute candidates correctly in ScrollAnchor.

An absolute-positioned candidate should be considered only if it would
actually scroll with the scroller (otherwise the anchoring scroll will have no
effect on it).

BUG= 594873 

Review URL: https://codereview.chromium.org/1831993002

Cr-Commit-Position: refs/heads/master@{#383361}

[modify] https://crrev.com/54502dc10d4a4cf3be8d9fb89db3d2927c7bbaf9/third_party/WebKit/Source/core/layout/ScrollAnchor.cpp
[modify] https://crrev.com/54502dc10d4a4cf3be8d9fb89db3d2927c7bbaf9/third_party/WebKit/Source/core/layout/ScrollAnchorTest.cpp

Comment 4 by skobes@chromium.org, Mar 25 2016

Status: Fixed (was: Started)
I tested other combinations of static / relative / absolute / fixed on the candidate and the scroller and I think they all behave correctly.

We don't do anything special with position: sticky, that may be worth investigating after http://crrev.com/1308273010 has landed.

Sign in to add a comment