Skip to content

Fix redefinitions of legacy iframe attributes#42542

Merged
chrisdavidmills merged 1 commit intomdn:mainfrom
bban160:fix-iframe-redefined-attributes
Jan 12, 2026
Merged

Fix redefinitions of legacy iframe attributes#42542
chrisdavidmills merged 1 commit intomdn:mainfrom
bban160:fix-iframe-redefined-attributes

Conversation

@bban160
Copy link
Contributor

@bban160 bban160 commented Dec 26, 2025

Description

Correct the stated redefinitions of allowfullscreen and allowpaymentrequest as per the relevant specs.

Motivation

The existing text states that allowfullscreen is redefined as allow="fullscreen" (shorthand for allow="fullscreen 'src'") and allowpaymentrequest is redefined as allow="payment" (shorthand for allow="payment 'src'"). In reality, both have allowlists of *, which makes a difference if the iframe is navigated away from the origin defined in its src attribute.

Additional details

allowfullscreen:

allowpaymentrequest:

@bban160 bban160 requested a review from a team as a code owner December 26, 2025 17:59
@bban160 bban160 requested review from chrisdavidmills and removed request for a team December 26, 2025 17:59
@github-actions github-actions bot added Content:HTML Hypertext Markup Language docs size/xs [PR only] 0-5 LoC changed labels Dec 26, 2025
@github-actions
Copy link
Contributor

@bban160
Copy link
Contributor Author

bban160 commented Jan 12, 2026

@chrisdavidmills Please don't forget about this PR 🙂

Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

LGTM; thanks, @bban160!

@chrisdavidmills chrisdavidmills merged commit da2d33b into mdn:main Jan 12, 2026
6 checks passed
@bban160 bban160 deleted the fix-iframe-redefined-attributes branch January 12, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTML Hypertext Markup Language docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants