Skip to content

Clarify overscroll-behaviour for iframe elements#20706

Merged
estelle merged 3 commits intomdn:mainfrom
temach:patch-1
Sep 22, 2022
Merged

Clarify overscroll-behaviour for iframe elements#20706
estelle merged 3 commits intomdn:mainfrom
temach:patch-1

Conversation

@temach
Copy link
Copy Markdown
Contributor

@temach temach commented Sep 14, 2022

Description

Clarify that using overscroll-property on iframe elements does not have an effect.

Motivation

It might seem that overscroll-property can solve scroll-chaining for iframes, but iframes with scroll bars are not scroll container elements, so this property does not apply. This was surprising and I wasted a lot of time to find out why.

Additional details

firefox issue and chrome issue.

Related issues and pull requests

@temach temach requested a review from a team as a code owner September 14, 2022 06:06
@temach temach requested review from estelle and removed request for a team September 14, 2022 06:06
@github-actions github-actions bot added the Content:CSS Cascading Style Sheets docs label Sep 14, 2022
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 15, 2022

Preview URLs

(this comment was updated 2022-09-20 22:10:09.636509)

Copy link
Copy Markdown
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your PR!

It's definitely worth noting this, but I don't think it needs Note styling or references to those bugs.

Co-authored-by: wbamberg <will@bootbonnet.ca>
Copy link
Copy Markdown
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@estelle estelle enabled auto-merge (squash) September 20, 2022 22:07
Copy link
Copy Markdown
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants