Skip to content

Add responsive iframe sizing demos#387

Merged
chrisdavidmills merged 9 commits into
mdn:mainfrom
chrisdavidmills:responsive-iframe-sizing
Jul 10, 2026
Merged

Add responsive iframe sizing demos#387
chrisdavidmills merged 9 commits into
mdn:mainfrom
chrisdavidmills:responsive-iframe-sizing

Conversation

@chrisdavidmills

Copy link
Copy Markdown
Contributor

Chrome 152 adds support for responsive iframe sizing; see https://chromestatus.com/feature/5108373464547328.

This relies on three new features:

This PR adds examples to demonstrate all three features.

@chrisdavidmills chrisdavidmills requested review from a team as code owners June 26, 2026 18:07
@chrisdavidmills chrisdavidmills requested a review from dipikabh June 26, 2026 18:07

@pepelsbey pepelsbey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good! Thank you 🙂

A few suggestions below, including an idea of how to simplify the JS demo.

Comment thread responsive-iframe-sizing/README.md
Comment thread responsive-iframe-sizing/js-request-resize/frame.html
Comment thread responsive-iframe-sizing/js-request-resize/index.html Outdated
Comment thread responsive-iframe-sizing/basic/frame.html Outdated
Comment thread responsive-iframe-sizing/js-request-resize/frame.html Outdated
Comment thread responsive-iframe-sizing/basic/index.html Outdated

@pepelsbey pepelsbey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One last thing and it's done 🙂

It will require a docs update, but it should be just a minor one.

Comment thread responsive-iframe-sizing/js-request-resize/frame.html Outdated
@chrisdavidmills chrisdavidmills merged commit 859935c into mdn:main Jul 10, 2026
1 check passed
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

Successfully merging this pull request may close these issues.

3 participants