Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Update demo components to be compatible with React 15.2 and up #118

Merged
merged 1 commit into from
Sep 19, 2016

Conversation

tonybaroneee
Copy link
Member

No description provided.

@tonybaroneee tonybaroneee force-pushed the fix-unknown-props-warnings branch 2 times, most recently from 6f60bd1 to 74664be Compare August 5, 2016 14:55
@@ -87,7 +87,7 @@ var ToggleBox = React.createClass({
return (
<div className="flex-box flex-column flex-1 align-items-center">
<div>
<select rvalue={this.state.effect} onChange={this.whenSelectChanged}>{this.renderEffects()}</select>
Copy link
Member Author

Choose a reason for hiding this comment

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

Wondering what the intent of this property was, can't find information on it anywhere. Plus it through an "Unknown property" warning (same thing down below in trigger-box).

Copy link
Collaborator

Choose a reason for hiding this comment

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

It should probably just be value.

On Fri, Aug 5, 2016 at 10:56 AM, Anthony Barone notifications@github.com
wrote:

In demo/examples/toggle-box.jsx
#118 (comment)
:

@@ -87,7 +87,7 @@ var ToggleBox = React.createClass({
return (


  •      <select rvalue={this.state.effect} onChange={this.whenSelectChanged}>{this.renderEffects()}</select>
    

Wondering what the intent of this property was, can't find information on
it anywhere. Plus it through an "Unknown property" warning (same thing down
below in trigger-box).


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/twitter-fabric/velocity-react/pull/118/files/74664beda670a05df4be99b0f5a185b0e8bbdc89#r73705754,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP65QXPOjmp9spInqB8QIsfw0P_2fi8ks5qc08rgaJpZM4JdwiN
.

@tonybaroneee
Copy link
Member Author

Related to #114

@tonybaroneee tonybaroneee force-pushed the fix-unknown-props-warnings branch from 74664be to 2b82374 Compare August 5, 2016 21:00
@fionawhim fionawhim merged commit 4345136 into master Sep 19, 2016
@fionawhim fionawhim deleted the fix-unknown-props-warnings branch September 19, 2016 19:24
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.

2 participants