Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Propogation of overscroll-behaviour from body to root #29

Closed
jgraham opened this issue Nov 18, 2021 · 10 comments
Closed

Propogation of overscroll-behaviour from body to root #29

jgraham opened this issue Nov 18, 2021 · 10 comments
Labels
compat-bug-proposal Proposals for web compat bugs
Milestone
Interop 2022

Comments

@jgraham
Copy link
Contributor

jgraham commented Nov 18, 2021 •
edited
Loading

Description
Blink propogates the overflow-behavior from the <body> element to the viewport, instead of from the root element to the viewport. See https://docs.google.com/document/d/1-pqljDgzzgRVve2oWUq237JPgtT12wcoctxbtBaHZdU/edit# for more.

Bug reports
CHR-954423

Specification
https://drafts.csswg.org/css-overscroll/Overview.html
w3c/csswg-drafts#6406

Tests
https://wpt.fyi/results/css/css-overflow?label=master&label=experimental&aligned&q=propagation (need to check exactly which tests cover this issue).

This is a bug to include in #9
@jgraham jgraham added the compat-bug-proposal Proposals for web compat bugs label Nov 18, 2021
@gsnedders
Copy link
Member

https://wpt.fyi/results/css/css-overflow?label=master&label=experimental&aligned&q=propagation (need to check exactly which tests cover this issue).

I'm unsure any of these tests actually cover this, looking at where there are differences?

@foolip
Copy link
Member

foolip commented Dec 6, 2021

@jgraham you've mentioned overflow:scroll but linked bugs and issues are about overscroll-behavior. Is that a typo, or is overflow:scroll a part of this too?

Also, are there examples of site compat issues caused by this?

@foolip
Copy link
Member

foolip commented Dec 7, 2021

The test for #30 does involve overflow: scroll so now I suspect there's just been a mixup of adjacent issues. @jgraham can you confirm?

@jgraham
Copy link
Contributor Author

jgraham commented Dec 7, 2021 •
edited
Loading

Yes, sorry this should have been overscroll-behaviour. I believe the bug and google doc linked are correct, and that browser match for scroll-behaviour, although I'm not certain about that. The Google doc has a big list of sites which are affected by the overscroll-behaviour difference.

@jgraham
Copy link
Contributor Author

jgraham commented Dec 7, 2021

@emilio might be able to confirm whether this issue is now correct :)

@emilio
Copy link

emilio commented Dec 7, 2021 •
edited
Loading

Yeah, this should probably just be about overscroll-behavior. Unfortunately there are no WPT tests for this it seems, because we don't quite have the facility to test overscroll in WPT I think, and propagated styles are not exposed programmatically.

@jensimmons
Copy link
Contributor

@smfr believes this is probably be very risky to change.

@jensimmons
Copy link
Contributor

@jgraham Could you rewrite the title & description of this item, so we can all make sure we are talking about the same thing?

@smfr
Copy link

smfr commented Dec 7, 2021

Oh, if this is about overscroll-behaviour then I have no concerns.

@jgraham jgraham changed the title Propogation of scroll behaviour from body to root Propogation of overscroll-behaviour from body to root Dec 8, 2021
@jgraham
Copy link
Contributor Author

jgraham commented Dec 8, 2021 •
edited
Loading

@jgraham Could you rewrite the title & description of this item, so we can all make sure we are talking about the same thing?

Done. Apologies for the muddle on this one.

@foolip foolip mentioned this issue Dec 10, 2021
Closed
@foolip foolip closed this as completed Aug 18, 2022
@gsnedders gsnedders added this to the Interop 2022 milestone Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
compat-bug-proposal Proposals for web compat bugs
Projects
None yet
Milestone
Interop 2022
Development

No branches or pull requests
6 participants
@smfr @jensimmons @gsnedders @jgraham @foolip @emilio