search

purescript-halogen / purescript-halogen-template

Template project for getting started with Halogen
Apache License 2.0
128 stars 56 forks source link

xmlns:foo? #18

Closed JamieBallingall closed 5 years ago

JamieBallingall commented 5 years ago

https://github.com/slamdata/purescript-halogen-template/blob/cebbb9fd48d438b5223c07630cc6104ec0e5d4cd/dist/index.html#L2

creates a namespace called 'foo'. Is that required? Given it the connotations of 'foo' as a placeholder name it creates confusion around the role of SVG in Halogen apps.

I believe it could simply be removed, since SVG would be within the HTML document and therefore unable to have its own namespace anyway.

garyb commented 5 years ago

Uh yes, absolutely, I have no idea why that is in there!