Skip to content

Discourage document.writeln() like document.write() #11384

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
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Discourage document.writeln() like document.write() #11384

wants to merge 2 commits into from

Conversation

caugner
Copy link

@caugner caugner commented Jun 17, 2025 •
edited by pr-preview bot
Loading

Discourage document.writeln() for the same reasons as document.write()

See: #7977 (comment)

  • At least two implementers are interested (and none opposed):
    • …
    • …
  • Tests are written and can be reviewed and commented upon at:
    • …
  • Implementation bugs are filed:
    • Chromium: …
    • Gecko: …
    • WebKit: …
    • Deno (only for timers, structured clone, base64 utils, channel messaging, module resolution, web workers, and web storage): …
    • Node.js (only for timers, structured clone, base64 utils, channel messaging, and module resolution): …
  • Corresponding HTML AAM & ARIA in HTML issues & PRs:
  • MDN issue is filed: …
  • The top of this comment includes a clear commit message to use.

(See WHATWG Working Mode: Changes for more details.)

/dynamic-markup-insertion.html ( diff )

@caugner caugner force-pushed the 7977-discourage-document-writeln branch from 3ad0cb8 to 92af19a Compare June 17, 2025 12:14
This was referenced Jun 17, 2025
Open
Merged
@caugner caugner force-pushed the 7977-discourage-document-writeln branch from 92af19a to f5bc070 Compare June 17, 2025 12:25
annevk
annevk approved these changes Jun 17, 2025
View reviewed changes
Copy link
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

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

Modulo formatting this seems okay to me.

@caugner
Apply suggestions from code review
9f74eea 
Co-authored by: Anne van Kesteren <annevk@annevk.nl>
@annevk
Copy link
Member

annevk commented Jun 18, 2025

@zcorpan could you please work with @caugner on getting the Participation check resolved?

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

@annevk annevk annevk approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@caugner @annevk