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

Update selectize.js #1520

Closed
wants to merge 1 commit into from
Closed

Conversation

ymeric
Copy link

@ymeric ymeric commented Feb 5, 2020

isFull function is returning true although create option is false. This blocks search feature in certain scenarios. This change will make sure the create option is true before comparing maxItems to length of items.

isFull function is returning true although create option is false. This blocks search feature in certain scenarios. This change will make sure the create option is true before comparing maxItems to length of items.
@Pictor13
Copy link
Contributor

I remember some bug like that....
I've never had the time to do a pull-request; thanks!

I think adding a test case to ensure it works would be great.

Looks like there's a new maintainer now, so PR are being evaluated again :)
But the PR should pass the tests, I guess.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 1, 2021

Stale pull request message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants