Skip to content
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.

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

Demix NonDocumentTypeChildNode #9065

Merged
merged 2 commits into from
Feb 14, 2021

Conversation

Elchi3
Copy link
Member

@Elchi3 Elchi3 commented Feb 11, 2021

NonDocumentTypeChildNode, read that again, NonDocumentTypeChildNode ...

So, what do you think it does? If you break it down, it is relatively simple: It adds two methods to Element and CharacterData. (why don't just say so, right?)

When documenting this as a mixin instead, you get this:
https://developer.mozilla.org/en-US/docs/Web/API/NonDocumentTypeChildNode

This PR re-arranges this piece of obscure art and I will definitely delete/re-write the MDN pages soon.

@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Feb 11, 2021
Copy link
Collaborator

@foolip foolip left a comment

Choose a reason for hiding this comment

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

So much nicer! I can check if the collector would make any updates, but fee free to merge it if I don't get to it today.

Copy link
Collaborator

@foolip foolip left a comment

Choose a reason for hiding this comment

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

Again the situation was quite different for different interfaces, I'm glad we're splitting this!

api/CharacterData.json Outdated Show resolved Hide resolved
api/CharacterData.json Outdated Show resolved Hide resolved
api/CharacterData.json Outdated Show resolved Hide resolved
api/CharacterData.json Outdated Show resolved Hide resolved
api/CharacterData.json Outdated Show resolved Hide resolved
api/Element.json Outdated Show resolved Hide resolved
api/Element.json Outdated Show resolved Hide resolved
api/Element.json Show resolved Hide resolved
api/Element.json Outdated Show resolved Hide resolved
api/Element.json Outdated Show resolved Hide resolved
Co-authored-by: Philip Jägenstedt <[email protected]>
@Elchi3
Copy link
Member Author

Elchi3 commented Feb 14, 2021

Interesting! Thanks for your research, Philip! Having this split up is so good. Can't wait to make this more clear on MDN, too.

@foolip foolip merged commit 2684ecd into mdn:master Feb 14, 2021
@Elchi3 Elchi3 deleted the demix-nondocumenttypechildnode branch February 14, 2021 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants