Skip to content

Comments

React 15#700

Closed
robertkowalski wants to merge 4 commits intoapache:masterfrom
robertkowalski:react-15
Closed

React 15#700
robertkowalski wants to merge 4 commits intoapache:masterfrom
robertkowalski:react-15

Conversation

@robertkowalski
Copy link
Member

Fixes and upgrades for React 15

function (app, FauxtonAPI, React, Stores, Actions, Components, Documents, FauxtonComponents,
ReactBootstrap, ReactSelect, ReactCSSTransitionGroup) {

require('react-select/dist/react-select.css');
Copy link
Member

Choose a reason for hiding this comment

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

Why not just add this to the other defines?

Copy link
Member Author

Choose a reason for hiding this comment

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

ah good call!

@garrensmith
Copy link
Member

+1 once tests pass.

@robertkowalski robertkowalski force-pushed the react-15 branch 2 times, most recently from 2f2cd47 to 73a5161 Compare April 13, 2016 12:04
react-select is easier to extend and customize and it also
supports rendering the result-list after a click.

it also supports react 15 and is actively maintained.
asfgit pushed a commit that referenced this pull request Apr 14, 2016
PR: #700
PR-URL: #700
Reviewed-By: garren smith <garren.smith@gmail.com>
asfgit pushed a commit that referenced this pull request Apr 14, 2016
PR: #700
PR-URL: #700
Reviewed-By: garren smith <garren.smith@gmail.com>
asfgit pushed a commit that referenced this pull request Apr 14, 2016
PR: #700
PR-URL: #700
Reviewed-By: garren smith <garren.smith@gmail.com>
asfgit pushed a commit that referenced this pull request Apr 14, 2016
react-select is easier to extend and customize and it also
supports rendering the result-list after a click.

it also supports react 15 and is actively maintained.

PR: #700
PR-URL: #700
Reviewed-By: garren smith <garren.smith@gmail.com>
@robertkowalski robertkowalski deleted the react-15 branch April 14, 2016 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants