Skip to content

Commit

Permalink
Kill ReactLink, LinkedStateMixin, valueLink and checkedLink (#8165)
Browse files Browse the repository at this point in the history
This should be safe because we've been warning for this. The LinkedStateMixin is technically exposed on React.addons without a warning but presumably you wouldn't be using it without a valueLink or checkedLink.

I do this primarily to clear up what the custom onChange listeners are doing.

Renamed the final prop type helper to ReactControlledValuePropTypes.
  • Loading branch information
sebmarkbage authored Oct 31, 2016
1 parent f538544 commit 34fd4f4
Show file tree
Hide file tree
Showing 13 changed files with 29 additions and 640 deletions.
2 changes: 0 additions & 2 deletions src/addons/ReactWithAddons.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@

'use strict';

var LinkedStateMixin = require('LinkedStateMixin');
var React = require('React');
var ReactAddonsDOMDependencies = require('ReactAddonsDOMDependencies');
var ReactComponentWithPureRenderMixin =
Expand All @@ -25,7 +24,6 @@ var update = require('update');

React.addons = {
CSSTransitionGroup: ReactCSSTransitionGroup,
LinkedStateMixin: LinkedStateMixin,
PureRenderMixin: ReactComponentWithPureRenderMixin,
TransitionGroup: ReactTransitionGroup,

Expand Down
38 changes: 0 additions & 38 deletions src/addons/link/LinkedStateMixin.js

This file was deleted.

73 changes: 0 additions & 73 deletions src/addons/link/ReactLink.js

This file was deleted.

46 changes: 0 additions & 46 deletions src/addons/link/__tests__/LinkedStateMixin-test.js

This file was deleted.

159 changes: 0 additions & 159 deletions src/addons/link/__tests__/ReactLinkPropTypes-test.js

This file was deleted.

2 changes: 0 additions & 2 deletions src/renderers/dom/shared/hooks/ReactDOMUnknownPropertyHook.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,7 @@ if (__DEV__) {

autoFocus: true,
defaultValue: true,
valueLink: true,
defaultChecked: true,
checkedLink: true,
innerHTML: true,
suppressContentEditableWarning: true,
onFocusIn: true,
Expand Down
Loading

0 comments on commit 34fd4f4

Please sign in to comment.