Skip to content
This repository has been archived by the owner on Aug 19, 2022. It is now read-only.

ensuring that defaultProps get transferred in every browser #241

Merged
merged 3 commits into from
Jul 7, 2015

Conversation

bobbyrenwick
Copy link
Contributor

This fixes #235, and doesn't require a move away from using the subclass method in order to enhance components.

@ianobermiller
Copy link
Contributor

Don't we need to do the same for propTypes, contextTypes, and childContextTypes?

@bobbyrenwick
Copy link
Contributor Author

@ianobermiller good thinking, that's done too

@ianobermiller
Copy link
Contributor

Can you just add a comment why we are doing this? Just say this is for IE <10 and link to the babel caveats page. Then this should be good to go. Thanks again!

@bobbyrenwick
Copy link
Contributor Author

@ianobermiller done!

@ianobermiller
Copy link
Contributor

Nice work, thanks @bobbyrenwick!

ianobermiller added a commit that referenced this pull request Jul 7, 2015
ensuring that defaultProps get transferred in every browser
@ianobermiller ianobermiller merged commit ac4f020 into FormidableLabs:master Jul 7, 2015
AnSavvides pushed a commit to AnSavvides/radium that referenced this pull request Aug 11, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Radium breaks getDefaultProps on IE
2 participants