Skip to content
This repository was archived by the owner on Jul 28, 2023. It is now read-only.

Treat nodeType 7 like comments#121

Merged
SantosGuillamot merged 1 commit intomain-wp-directives-pluginfrom
handle-xpacket
Dec 16, 2022
Merged

Treat nodeType 7 like comments#121
SantosGuillamot merged 1 commit intomain-wp-directives-pluginfrom
handle-xpacket

Conversation

@SantosGuillamot
Copy link
Copy Markdown
Contributor

@SantosGuillamot SantosGuillamot commented Dec 16, 2022

As explained in #117 , there are some sites using xpacket inside SVG nodes, which returns a hydration error. As David suggests here, and following MDN docs, I am filling this PR to treat them like comments and removing them.

I am planning to add tests for this and other use cases in other PR.

Copy link
Copy Markdown
Member

@luisherranz luisherranz left a comment

Choose a reason for hiding this comment

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

LGTM!

@SantosGuillamot SantosGuillamot merged commit 5927078 into main-wp-directives-plugin Dec 16, 2022
@SantosGuillamot SantosGuillamot deleted the handle-xpacket branch December 16, 2022 11:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants