Skip to content

fix(7410): Support JSX Element as JSX Attribute Value#47994

Merged
sandersn merged 1 commit intomicrosoft:mainfrom
a-tarasyuk:fix/7410
May 11, 2022
Merged

fix(7410): Support JSX Element as JSX Attribute Value#47994
sandersn merged 1 commit intomicrosoft:mainfrom
a-tarasyuk:fix/7410

Conversation

@a-tarasyuk
Copy link
Copy Markdown
Contributor

Fixes #7410

JSXAttributeValue:
    " JSXDoubleStringCharactersopt "
    ' JSXSingleStringCharactersopt '
    { AssignmentExpression }
    JSXElement
    JSXFragment

@RyanCavanaugh
Copy link
Copy Markdown
Member

Waiting for final confirmation on facebook/jsx#53 before merging/closing.

@RyanCavanaugh
Copy link
Copy Markdown
Member

@raghavthind2005 bulk-"approving" changes like what you're doing here is a little spammy; please refrain from doing this

@sandersn
Copy link
Copy Markdown
Member

sandersn commented May 9, 2022

@RyanCavanaugh are we still waiting on confirmation from facebook/jsx to merge this?

@RyanCavanaugh
Copy link
Copy Markdown
Member

Maybe let's just merge it. If the JSX spec changes, we'll have the same mess to clean up as Babel, which is fine

@sandersn sandersn merged commit c300fea into microsoft:main May 11, 2022
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

For Backlog Bug PRs that fix a backlog bug

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Support JSX Element as JSX Attribute Value

6 participants