Skip to content

Commit

Permalink
#36 Solve react-unknown-prop when using copyKeys
Browse files Browse the repository at this point in the history
Upgraded React version in order to replicate
  • Loading branch information
mderrick committed Jul 17, 2016
1 parent d573310 commit 0591602
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
4 changes: 2 additions & 2 deletions demo/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
"dependencies": {
"css-reset": "git+https://github.com/60frames/css-reset.git",
"lodash.throttle": "^3.0.4",
"react": "^15.0.0",
"react-dom": "^15.0.0"
"react": "^15.2.0",
"react-dom": "^15.2.0"
},
"devDependencies": {
"babel-core": "^5.8.25",
Expand Down
4 changes: 3 additions & 1 deletion src/components/video/Video.js
Original file line number Diff line number Diff line change
Expand Up @@ -338,7 +338,9 @@ var Video = React.createClass({
render() {
// If controls prop is provided remove it
// and use our own controls.
var {controls, ...otherProps} = this.props;
// Leave `copyKeys` here even though not used
// as per issue #36.
var {controls, copyKeys, ...otherProps} = this.props;
return (
<div className={this.getVideoClassName()}
tabIndex="0"
Expand Down

0 comments on commit 0591602

Please sign in to comment.