Skip to content

Conversation

dallonf
Copy link

@dallonf dallonf commented Dec 28, 2017

I merged from the latest version of the async-pagination PR, which has been updated to support the module's recent v1.1.0 release. (it's finally out of 1.0 limbo!)

This also explicitly supports React 16 as a peerDependency (previously, I think React 16 worked, but raised a warning on install).

JedWatson and others added 30 commits September 13, 2017 03:48
Resolves PropType deprecation warning
Remove obsolete browserify-shim configuration
Remove obsolete browserify-shim configuration
fix bug, if wrote "hasOwnProperty"
…#1951.

Ran this locally and compared each of the demos against the live version
and saw no differences.
…selector-scoping

Fixing css states to be scoped with Select selector, closes JedWatson#1951.
Addresses JedWatson#2009.

There is a WIP test -- I can't figure out how to simulate the exact
scenario that this patch fixes in the test. The current test returns
positive even without my change. Suggestions would be appreciated --
thanks!
Addresses the second issue mentioned in the linked issue. While the
previous commit makes this change unnecessary for regular use cases, the
issue is still present when debouncing. By clearing out
`this._callback`, intermediary searches (e.g. 'Ti' when 'Tim' is already
fetched) won't cause the options for 'Tim' to change when the request
for 'Ti' completes.
Without key prop the input element unmounted and remounted after input changed, and the "onblur" handler make the panel dismissed soon.
It seems that the version stated on the README cannot be found, but react-select@1.0.0-rc.10 is the latest version.
Correct release candidate version on README
JedWatson and others added 27 commits December 13, 2017 17:19
…value

Clear input value on receiving props with another value.
Don't open drop down menu when clear values
onSelectResetsInput regression fixed
Pass placeholder prop to ValueComponent
…sets-input-for-single

Regression fix for single select with onSelectResetsInput=false
…stream-1.1.0

# Conflicts:
#	README.md
#	dist/react-select.css
#	dist/react-select.js
#	dist/react-select.min.css
#	dist/react-select.min.js
#	examples/dist/app.js
#	examples/dist/bundle.js
#	examples/dist/common.js
#	examples/dist/standalone.js
#	lib/Async.js
#	lib/Creatable.js
#	lib/Select.js
#	package.json
#	src/Creatable.js
#	src/Select.js
#	test/Async-test.js
@coveralls
Copy link

Coverage Status

Coverage increased (+3.4%) to 92.07% when pulling 22c41d2 on merge/upstream-1.1.0 into 2289c50 on master.

Copy link

@benbeadle benbeadle left a comment

Choose a reason for hiding this comment

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

Uh...this PR is like impossible to review, so...I approve! :D

@dallonf dallonf closed this Sep 18, 2019
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.