Skip to content

Comments

Rewrite syntax for createelementns method#14619

Merged
teoli2003 merged 2 commits intomdn:mainfrom
MrBrain295:MrBrain295-patch-1
Apr 3, 2022
Merged

Rewrite syntax for createelementns method#14619
teoli2003 merged 2 commits intomdn:mainfrom
MrBrain295:MrBrain295-patch-1

Conversation

@MrBrain295
Copy link
Contributor

Summary

Replace var with let.

Motivation

It was wrong.

Supporting details

Related issues

Metadata

  • Adds a new document
  • Rewrites (or significantly expands) a document
  • Fixes a typo, bug, or other error

@MrBrain295 MrBrain295 requested a review from a team as a code owner April 3, 2022 14:46
@MrBrain295 MrBrain295 requested review from Rumyra and removed request for a team April 3, 2022 14:46
@github-actions github-actions bot added the Content:WebAPI Web API docs label Apr 3, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Apr 3, 2022

Preview URLs

Flaws

URL: /en-US/docs/Web/API/Document/createElementNS
Title: Document.createElementNS()
on GitHub
Flaw count: 5

  • macros:
    • wrong xref macro used (consider changing which macro you use)
    • /en-US/docs/Web/API/Element/nodeName does not exist
  • broken_links:
    • Can't resolve /en-US/docs/Mozilla/Tech/XUL
    • Can't resolve /en-US/docs/Mozilla/Tech/XBL
    • Can't resolve /en-US/docs/Mozilla/Tech/XUL

External URLs

URL: /en-US/docs/Web/API/Document/createElementNS
Title: Document.createElementNS()
on GitHub

No new external URLs

(this comment was updated 2022-04-03 15:23:53.392516)

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

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

👍

@teoli2003 teoli2003 changed the title Replace var with let Rewrite syntax for createelementns method Apr 3, 2022
@teoli2003 teoli2003 merged commit 11f9240 into mdn:main Apr 3, 2022
@MrBrain295 MrBrain295 deleted the MrBrain295-patch-1 branch April 3, 2022 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebAPI Web API docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants