Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Missing autofocus in DOMProperty.js standard properties enumeration. #149

Closed
steida opened this issue Jul 2, 2013 · 3 comments
Closed

Missing autofocus in DOMProperty.js standard properties enumeration. #149

steida opened this issue Jul 2, 2013 · 3 comments
Assignees
@zpao

Comments

@steida
Copy link

steida commented Jul 2, 2013

Not sure if it's bug or feature.
@ghost ghost assigned zpao Jul 5, 2013
@zpao
Copy link
Member

zpao commented Jul 5, 2013

I'll be adding this and a couple other known missing properties. Thanks for the heads up!

FWIW, I only see autofocus being valuable for server-side rendering. Browsers are only supposed to look for this property on page load, so adding it with client-side rendering shouldn't do anything.

@steida
Copy link
Author

steida commented Jul 6, 2013

It shouldn't, but it does. At least on my Chrome. Check http://jsfiddle.net/k8Y2c/2.

@zpao
Copy link
Member

zpao commented Jul 8, 2013

I added this and a few others in 203dba2

@zpao zpao closed this as completed Jul 8, 2013
bvaughn pushed a commit to bvaughn/react that referenced this issue Aug 13, 2019
Merge pull request facebook#149 from gaearon/dont-reset
826cac0 
Don't reset selected item on search mismatch or exit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zpao zpao

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zpao @steida