Skip to content

Don't warn for upper case tag name if it's a custom element#21156

Merged
sebmarkbage merged 1 commit intofacebook:masterfrom
sebmarkbage:dontwarnforcustomelementcasing
Apr 1, 2021
Merged

Don't warn for upper case tag name if it's a custom element#21156
sebmarkbage merged 1 commit intofacebook:masterfrom
sebmarkbage:dontwarnforcustomelementcasing

Conversation

@sebmarkbage
Copy link
Copy Markdown
Contributor

Alternative to #21076

This replicates what we do on the client already.

@sebmarkbage sebmarkbage requested a review from gaearon April 1, 2021 00:56
@facebook-github-bot facebook-github-bot added React Core Team Opened by a member of the React Core Team CLA Signed labels Apr 1, 2021
Copy link
Copy Markdown
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

okay

This replicates what we do on the client.
@sebmarkbage sebmarkbage force-pushed the dontwarnforcustomelementcasing branch from 2266075 to b218955 Compare April 1, 2021 01:00
@gaearon
Copy link
Copy Markdown
Collaborator

gaearon commented Apr 1, 2021

Why is it legit to have a custom element with non-lowercase name? What's the point?

@sizebot
Copy link
Copy Markdown

sizebot commented Apr 1, 2021

Comparing: 9ed0167...b218955

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 122.65 kB 122.65 kB = 39.35 kB 39.36 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 129.21 kB 129.21 kB = 41.43 kB 41.43 kB
facebook-www/ReactDOM-prod.classic.js = 405.89 kB 405.89 kB = 75.39 kB 75.39 kB
facebook-www/ReactDOM-prod.modern.js = 394.15 kB 394.15 kB = 73.53 kB 73.53 kB
facebook-www/ReactDOMForked-prod.classic.js = 405.89 kB 405.89 kB = 75.39 kB 75.39 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against b218955

@sebmarkbage sebmarkbage merged commit 588db2e into facebook:master Apr 1, 2021
acdlite pushed a commit to acdlite/react that referenced this pull request Apr 11, 2021
acdlite pushed a commit to acdlite/react that referenced this pull request Apr 13, 2021
acdlite pushed a commit to acdlite/react that referenced this pull request Apr 16, 2021
acdlite pushed a commit to acdlite/react that referenced this pull request Apr 16, 2021
acdlite pushed a commit to acdlite/react that referenced this pull request Apr 19, 2021
koto pushed a commit to koto/react that referenced this pull request Jun 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants