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

(Optionally) support indeterminate (tristate) checkbox on select all #1

Open
harvimt opened this issue Feb 24, 2014 · 1 comment
Open

Comments

@harvimt
Copy link

harvimt commented Feb 24, 2014

Checkboxes have an indeterminate state that can be checked with .prop('indeterminate'). This should be used by the select all box when some but not all checkboxes are checked to better conform with Human Interface Guidelines (in most cases).

It gets funky pretty fast though, since you need indeterminate & checked set to show an indeterminate checkbox, but if the checkbox is checked it freaks out the rest of the code.

I'm working on a solution for myself, I'll see what I come up with.

@nylen
Copy link
Owner

nylen commented Feb 26, 2014

I looked at this a little bit yesterday. I'm in favor of supporting the indeterminate property - probably as an additional config option - but there are a lot of edge cases that would need to be handled correctly.

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