glimmer-vm
glimmer-vm copied to clipboard
Update list of allowed elements
Potentials lists we could use:
- https://www.npmjs.com/package/html-spec-tags?activeTab=readme
What we need:
- SVG Elements
- HTML Elements
- etc?
(or remove getTemplateLocals https://github.com/glimmerjs/glimmer-vm/issues/1550#issuecomment-1972012330 )
@NullVoxPopuli is this still needed?
Ya
babel template compilation plugin is not using getTemplateLocals anymore. so, maybe not needed? and just remove the whole list? basically, now, everything is a html element, except if it has a js local. How does it work for non-strict mode?
https://github.com/emberjs/babel-plugin-ember-template-compilation/pull/33
Ah! You're right! That fixes this repro from the linearGradient issue report https://github.com/glimmerjs/glimmer-vm/issues/1550
We do still need to fix math elements tho:
#1593