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.

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

IntersectionObserver spec needs to be clearer about "overflow clip" / "overflow clipping" terminology #351

Open
dholbert opened this issue Mar 7, 2019 · 5 comments

Comments

@dholbert
Copy link
Member

dholbert commented Mar 7, 2019

The IntersectionObserver spec uses the term "overflow clip" twice:
Section 2.2 says:

If the intersection root has an overflow clip

Section 3.2.4 says:

If container has overflow clipping

This terminology isn't precise enough.

Does it mean literally overflow:clip?

Or does it also mean overflow:hidden (which isn't named "clip" but whose spec text says the "content is clipped")

Or does it mean overflow: [non-visible] (since e.g. scroll also says "the content is clipped")? Or something else? Please clarify the terminology and ideally link to a definition if possible -- thanks!

@a2sheppy
Copy link

a2sheppy commented Mar 7, 2019

Watching this so I can update documentation, as I'm trying to fill in a gap in the docs on MDN where we don't really cover how clipping is handled by IO.

@dholbert
Copy link
Member Author

dholbert commented Mar 7, 2019

It looks like no browser currently supports overflow:clip (per footnote 2 on https://caniuse.com/#search=overflow ), so that's probably not what the spec means here.

So I'm guessing the spec is using "overflow clip" to mean overflow: [non-visible]. (Though it's quite easy to misread "overflow clip[ping]" as overflow:clip, so definitely worth clarifying.)

@wangxianzhu
Copy link

wangxianzhu commented Jun 20, 2023

The original comment has been moved into #518.

1 similar comment
@wangxianzhu
Copy link

wangxianzhu commented Oct 31, 2023

The original comment has been moved into #518.

@wangxianzhu
Copy link

The current spec (https://www.w3.org/TR/intersection-observer/) no longer mentions "overflow clip" / "overflow clipping". Should we close this issue?

@szager-chromium

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants