-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
JSX namespace support #702
Comments
Well React doesn't support them and I'm not aware of anything else supporting them either, so it's not clear to me how they are supposed to be translated to JavaScript. I can certainly parse them and turn them into a prettier syntax error if that's what you mean. |
@evanw non-React JSX implementations could use them in their configured factories. I think the most natural thing to do would be to simply allow the syntax and pass it as a single colon-separated attribute name. |
This change has been released by the way. It should work in version 0.8.36. I'm somewhat curious about your use case. Was it something specific? Or just checking for compliance with the spec? |
@evanw it’s for glimmer 3 POC, https://github.com/wycats/shimmer |
@lifeart is correct 😄 |
While React doesn't support JSX namespaces, they're explicitly part of the JSX spec. Is there a specific reason that esbuild doesn't parse namespaces?
The text was updated successfully, but these errors were encountered: