Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add search to dropdown component #830

Merged
merged 7 commits into from
Aug 2, 2019

Conversation

cgregs32
Copy link
Contributor

@cgregs32 cgregs32 commented Jul 26, 2019

#832
Adds the ability to include search component to select component via withSearch prop

Utilizes Search component within select component to be able to filter longer lists of options.
withSearch default to false, so this change will not affect existing calls to the select component

Screen Shot 2019-07-25 at 2 52 50 PM

@cgregs32 cgregs32 force-pushed the chris/add-search-to-dropdown-component branch 3 times, most recently from 200bbda to 3e7b912 Compare July 26, 2019 19:40
src/components/Select.js Outdated Show resolved Hide resolved
src/components/Select.js Outdated Show resolved Hide resolved
src/components/Select.js Outdated Show resolved Hide resolved
@cgregs32 cgregs32 force-pushed the chris/add-search-to-dropdown-component branch from 73ede10 to f7ae456 Compare July 30, 2019 21:37
@cgregs32 cgregs32 force-pushed the chris/add-search-to-dropdown-component branch from 2fe27e8 to 8d564ac Compare August 1, 2019 20:37
@cgregs32
Copy link
Contributor Author

cgregs32 commented Aug 1, 2019

@cerinman
Please take another look!

@cerinman
Copy link
Contributor

cerinman commented Aug 1, 2019

:shipit:

@cgregs32 cgregs32 merged commit 44b276f into master Aug 2, 2019
@cgregs32 cgregs32 deleted the chris/add-search-to-dropdown-component branch August 2, 2019 15:38
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.

None yet

3 participants