Skip to content
New issue

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

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

Already on GitHub? # to your account

transferPropsTo is now deprecated in React #47

Closed
tonsky opened this issue Jan 11, 2015 · 3 comments
Closed

transferPropsTo is now deprecated in React #47

tonsky opened this issue Jan 11, 2015 · 3 comments

Comments

@tonsky
Copy link
Contributor

tonsky commented Jan 11, 2015

Hi!

transferPropsTo is now deprecated in React: https://gist.github.com/sebmarkbage/a6e220b7097eb3c79ab7

It’s used in interpreter/wrap-form-element
https://github.com/r0man/sablono/blob/master/src/sablono/interpreter.cljx#L35

It generates warning every time form element is used:

Warning: transferPropsTo is deprecated. See http://fb.me/react-transferpropsto for more information.

BTW may I ask, what is the motivation behind wrapping React input elements with wrapper? Why don’t just use raw React inputs?

@r0man
Copy link
Owner

r0man commented Jan 12, 2015

Hi tonsky,

thanks, I'll update this in the next days. The reason input
elements are wrapped is because there were problems with the
cursor in input elements beeing reseted when typing. This fix, is
actually copied from om, and the original issue is here:

#10

Roman.

@r0man
Copy link
Owner

r0man commented Jan 12, 2015

@tonsky This should be fixed in 0.3.0-SNAPSHOT. Can you verify this is working for you?

@tonsky
Copy link
Contributor Author

tonsky commented Jan 12, 2015

Confirm — all works, warnings are gone, thanks!=

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants