Skip to content

Don't emit autoFocus={false} attribute on the server - #11543

Merged
gaearon merged 1 commit into
react:masterfrom
gaearon:autofocus-false
Nov 13, 2017
Merged

Don't emit autoFocus={false} attribute on the server#11543
gaearon merged 1 commit into
react:masterfrom
gaearon:autofocus-false

Conversation

@gaearon

@gaearon gaearon commented Nov 13, 2017

Copy link
Copy Markdown
Contributor

Seems like I made a mistake in #11192.
Browsers don't treat autofocus="false" as truthy so we should use HAS_BOOLEAN_VALUE here.

Fixes the issue described in #11192 (comment).

@trueadm trueadm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@gaearon
gaearon merged commit afc5fab into react:master Nov 13, 2017
Ethan-Arrowood pushed a commit to Ethan-Arrowood/react that referenced this pull request Dec 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants