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.

Sign up for GitHub

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

Jump to bottom

Clicking an indeterminate select all -checkbox should select everything on "multi" mode #42

Closed
tomivirkki opened this issue Jun 29, 2015 · 0 comments
Closed

Clicking an indeterminate select all -checkbox should select everything on "multi" mode #42

tomivirkki opened this issue Jun 29, 2015 · 0 comments
Assignees
@anezthes
Labels
bug Something isn't working
Milestone
0.3.0-beta4

Comments

@tomivirkki
Copy link
Member

When the selection mode is set to multi and some items are selected, the select all checkbox goes into indeterminate mode. Now when it's clicked, the mode should switch to "all" but instead it just clears the select all -checkbox.
It works the other way around though ("all" -> "multi")
@tomivirkki tomivirkki added bug Something isn't working <v-grid> labels Jun 29, 2015
@jouni jouni modified the milestone: 0.3.0 Jun 30, 2015
anezthes added a commit that referenced this issue Jul 1, 2015
@anezthes
Issue #42. Removed selection checks from IndexBasedSelectionModel*. i…
6790783 
…sChecked triggers event in in GridComponent.
@anezthes anezthes mentioned this issue Jul 1, 2015
Merged
anezthes added a commit that referenced this issue Jul 1, 2015
@anezthes
Should fix issue #42.
2d8dca6
Saulis added a commit that referenced this issue Jul 2, 2015
@Saulis
Merge pull request #44 from vaadin/fix/42
6e7cb27 
Issue #42. Removed selection checks from IndexBasedSelectionModel*.
@jouni jouni modified the milestones: 0.3.0, Next Jul 3, 2015
@Saulis Saulis modified the milestones: Next, 0.3.0-beta4 Jul 3, 2015
HJK181 pushed a commit to CommerceExperts/vaadin-grid that referenced this issue Jul 27, 2020
@tomivirkki
Merge pull request vaadin#42 from vaadin/feature/material-design
10ae227 
Apply Material Design styles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anezthes anezthes

Labels
bug Something isn't working
Projects
None yet
Milestone
0.3.0-beta4
Development

No branches or pull requests
4 participants
@jouni @Saulis @tomivirkki @anezthes