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

supports multiselect #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

supports multiselect #15

wants to merge 3 commits into from

Conversation

elghali
Copy link
Collaborator

@elghali elghali commented Jul 26, 2017

please check this PR, as now it supports multiselect, maybe if you release different versions

@elghali elghali self-assigned this Jul 26, 2017
@ritz078
Copy link
Owner

ritz078 commented Jul 26, 2017

multi select should be optional. It should default to single select. Also in that case test is required.

@elghali
Copy link
Collaborator Author

elghali commented Jul 26, 2017

Sure it should, maybe add boolean prop "multi", do you care to do it?

@ritz078
Copy link
Owner

ritz078 commented Jul 26, 2017

I won't be able to do it now due to work. Will do once I have time. You can actually use react-select for that use case.

@elghali
Copy link
Collaborator Author

elghali commented Jul 26, 2017

React-select does not support fuzzy search , i need this functionality

@ritz078
Copy link
Owner

ritz078 commented Dec 27, 2020

@PatrickDesign can you please check if this is still relevant ?

@PatrickDesign
Copy link
Collaborator

@ritz078 Having looked at the code, I believe a multi-select could be built around our component (although the dropdown would close on each selection). Maybe the problem could be solved in a simpler way, i.e. a shouldStayOpenOnSelection prop.

Leaving this PR open in case anybody has more to chip in.

Copy link

@Emssui Emssui left a comment

Choose a reason for hiding this comment

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

Looks good to me

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.

4 participants