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

Single select shouldn't show "Check all" and "Uncheck all" #21

Closed
alexwhitman opened this issue Sep 30, 2010 · 3 comments
Closed

Single select shouldn't show "Check all" and "Uncheck all" #21

alexwhitman opened this issue Sep 30, 2010 · 3 comments

Comments

@alexwhitman
Copy link

As the title says, when using single select {multiple: false} the "Check all" and "Uncheck all" links shouldn't be shown as it doesn't really make sense.

@alexwhitman
Copy link
Author

Can be easily fixed by changing line 64 to:
if(o.header === true && o.multiple === true){

@ehynds
Copy link
Owner

ehynds commented Sep 30, 2010

no, but the header should still be shown so the user can set it to whatever they want. I've documented using the header attribute with the multiple option.

@ehynds
Copy link
Owner

ehynds commented Sep 30, 2010

on second thought, I agree. header option is still available anyway. landed: http://github.com/ehynds/jquery-ui-multiselect-widget/commit/43fc35d413499844bb513cdb6b292180e7a5b794

This issue was closed.
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

No branches or pull requests

2 participants