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

Allow SVG elements in DOM (close #354). #416

Closed
wants to merge 1 commit into from
Closed

Conversation

smuuf
Copy link

@smuuf smuuf commented Nov 22, 2019

  • bug fix
  • BC break? no

Added svg to the list of supported HTML tags. Also added the rest of known SVG elements as per MDN SVG Elements page.

@dg
Copy link
Member

@dg dg commented Nov 28, 2019

There is also a lot of SVG elements https://developer.mozilla.org/en-US/docs/Web/SVG/Element

@dg dg force-pushed the master branch 10 times, most recently from f2dae8a to 20b7c0b Compare Jan 24, 2020
@genesiscz
Copy link

@genesiscz genesiscz commented Feb 9, 2020

@smuuf are you gonna complete this one or not?

@smuuf
Copy link
Author

@smuuf smuuf commented Feb 17, 2020

I added the rest of known SVG elements as per the MDN SVG elements page + some tests.

@dg
Copy link
Member

@dg dg commented Feb 19, 2020

Thanks. When I think about it, it may be better to allow all the elements. Because there are MathML and Custom Elements…

@dg dg closed this in 0b33a93 Feb 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants