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

multiple select #18

Closed
tallesdantas opened this issue Jan 9, 2016 · 10 comments
Closed

multiple select #18

tallesdantas opened this issue Jan 9, 2016 · 10 comments
Labels
bug

Comments

@tallesdantas
Copy link

@tallesdantas tallesdantas commented Jan 9, 2016

The multiple select doesnt work with semantcui { http://semantic-ui.com/ }, i think that it also doest work with bootstrap , when i use the $ctrl.select2({}); in init it works but it buggs the layout just working with the select2 api , please help.

@ermouth
Copy link
Owner

@ermouth ermouth commented Jan 9, 2016

please help

Help to do what?

@tallesdantas
Copy link
Author

@tallesdantas tallesdantas commented Jan 9, 2016

how can i use the multiple select of semantic ui http://semantic-ui.com/modules/dropdown.html with it?, its like bottstrap ,if i use $ctrl.select2({}); it glitchs the layout

@ermouth
Copy link
Owner

@ermouth ermouth commented Jan 9, 2016

Seems to work fine if you create it using conventional SELECT, not a bunch of divs. Can you provide a code to reproduce your case?

@tallesdantas
Copy link
Author

@tallesdantas tallesdantas commented Jan 9, 2016

i've tyed to use select too , but when i add a value and cancell it it holds the last one , becouse the default "select multiple(html pure)" aways holds a value, try add multiple values then cancell eatch one.

@ermouth
Copy link
Owner

@ermouth ermouth commented Jan 10, 2016

Well, I see. It‘s indeed a bug. Will fix in 1.2.1 quiet soon, thank for pointing out.

@tallesdantas
Copy link
Author

@tallesdantas tallesdantas commented Jan 10, 2016

thanks, btw nice plugin it's mutch better than downloading a whole library like knowcout, makes me easyer to program, as soon as you fix it please tellme =D

@ermouth
Copy link
Owner

@ermouth ermouth commented Jan 10, 2016

As a quick fix you can try to add return $o.val()||[]; right after line 424 with } of jquerymy.js. Hope it fix an issue at least in your particular case.

@tallesdantas
Copy link
Author

@tallesdantas tallesdantas commented Jan 10, 2016

Works like a charm , thanks =D

@ermouth ermouth added the bug label Jan 12, 2016
@ermouth
Copy link
Owner

@ermouth ermouth commented Jan 12, 2016

Reopen as a bug to fix into 1.2.1
Invalid behavior for select[multiple], does not allow to uncheck last elt in the list and make list empty. Takes place due to $('').val() returns null, not []

@ermouth ermouth reopened this Jan 12, 2016
@ermouth
Copy link
Owner

@ermouth ermouth commented Jan 13, 2016

Fixed in 1.2.1

@ermouth ermouth closed this Jan 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.