Skip to content

allow jsx identifiers to start with keywords#4705

Merged
vladima merged 2 commits into
masterfrom
keywordInJsxIdentifier
Sep 10, 2015
Merged

allow jsx identifiers to start with keywords#4705
vladima merged 2 commits into
masterfrom
keywordInJsxIdentifier

Conversation

@vladima
Copy link
Copy Markdown
Contributor

@vladima vladima commented Sep 9, 2015

fixes #4644

Comment thread src/compiler/parser.ts Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be exported from the scanner (and marked internal).

@DanielRosenwasser
Copy link
Copy Markdown
Member

Can you add an example of the property being a keyword entirely? Like class="FooAligned".

@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Sep 10, 2015

👍 i would merge this into master and not release-1.6

vladima added a commit that referenced this pull request Sep 10, 2015
allow jsx identifiers to start with keywords
@vladima vladima merged commit c90276f into master Sep 10, 2015
@vladima vladima deleted the keywordInJsxIdentifier branch September 10, 2015 22:00
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Jsx attributes that start with keyword are not handled correctly

4 participants