Skip to content

add jsx factory and hold text in jsxtext node - #29439

Merged
Wesley Wigham (weswigham) merged 3 commits into
microsoft:masterfrom
Kingwl:jsx_text_node
Mar 11, 2019
Merged

add jsx factory and hold text in jsxtext node#29439
Wesley Wigham (weswigham) merged 3 commits into
microsoft:masterfrom
Kingwl:jsx_text_node

Conversation

@Kingwl

Copy link
Copy Markdown
Contributor

Fixes #29425

Comment thread src/compiler/factory.ts Outdated
Comment thread src/compiler/factory.ts Outdated
@Kingwl

Copy link
Copy Markdown
Contributor Author

🆙

@weswigham
Wesley Wigham (weswigham) merged commit b97b1a8 into microsoft:master Mar 11, 2019
@Kingwl
Wenlu Wang (Kingwl) deleted the jsx_text_node branch March 12, 2019 07:38
@dsherret

Copy link
Copy Markdown
Contributor

@weswigham

Wesley Wigham (weswigham) commented Mar 30, 2019

Copy link
Copy Markdown
Member

@Kingwl

Copy link
Copy Markdown
Contributor Author

the only things has been break is the containsOnlyWhiteSpaces😅

@dsherret

David Sherret (dsherret) commented Mar 31, 2019

Copy link
Copy Markdown
Contributor

That’s ok! Just making sure it's documented. Thanks for adding the factories 🙂

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.

4 participants