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

Remove all options when search input is empty to fix #25 #26

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

Conversation

pjfsilva
Copy link

Fixes #25

This adds code to remove all options when search input is empty. This solves a problem where the options can be left there if you type something, get results and then delete everything from input

@pjfsilva pjfsilva changed the title master Remove all options when search input is empty to fix #25 Mar 26, 2017
…till kept to the last results from server

Fixes a problem where all selected options where gone after input is set to empty
@ksykulev
Copy link
Owner

ksykulev commented Mar 30, 2017

@pjfsilva - thanks for your contribution! This is great! Would it be possible for you to write a test for this in https://github.com/ksykulev/chosen-ajax-addition/blob/master/test/main.js ? That way if I upgrade chosen or make any additional changes I can know this feature will continue to work as intended. If you need help let me know.

@seyfer
Copy link

seyfer commented Sep 5, 2017

@ksykulev @pjfsilva would be great to finish it

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.

3 participants