Skip to content

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

Closed
smuuf wants to merge 1 commit into
nette:masterfrom
smuuf:patch-1
Closed

smuuf wants to merge 1 commit into
nette:masterfrom
smuuf:patch-1

Conversation

@smuuf

@smuuf smuuf commented Nov 22, 2019

Copy link
Copy Markdown
Contributor
  • 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

dg commented Nov 28, 2019

Copy link
Copy Markdown
Member

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 January 24, 2020 13:18
@genesiscz

Copy link
Copy Markdown

@smuuf are you gonna complete this one or not?

@smuuf

smuuf commented Feb 17, 2020

Copy link
Copy Markdown
Contributor Author

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

@dg

dg commented Feb 19, 2020

Copy link
Copy Markdown
Member

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants