Skip to content

Conversation

@yutin1987
Copy link

2015-11-22 1 29 53

when Toggle add valueLink props

Copy link
Member

Choose a reason for hiding this comment

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

could you use the getDefaultProps instead for the false value?

@oliviertassinari
Copy link
Member

Soulds like a good feature since it's documented by facebook here https://facebook.github.io/react/docs/two-way-binding-helpers.html.
Could you add an example in the documentation, the propery in the documentation, and a PropType following (https://github.com/facebook/react/blob/401e6f10587b09d4e725763984957cf309dfdc30/src/addons/link/ReactLink.js#L57)?

@oliviertassinari
Copy link
Member

Actually, the valueLink is now deprecated by facebook. See facebook/react#2302 and facebook/react#5032
Thanks for you effort anyway.

@zannager zannager added the scope: toggle button Changes related to the toggle button. label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: toggle button Changes related to the toggle button.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants