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

Checkbox hover circle overlap in MudTable when table is 'dense' #2454

Closed
MaRRiK74 opened this issue Aug 10, 2021 · 5 comments
Closed

Checkbox hover circle overlap in MudTable when table is 'dense' #2454

MaRRiK74 opened this issue Aug 10, 2021 · 5 comments
Assignees
Labels
bug Something does not work as intended/expected fixed Fix has been merged
Milestone

Comments

@MaRRiK74
Copy link

MaRRiK74 commented Aug 10, 2021

Describe the bug
Checkboxes rendered within a MudTable are not dense when the MudTable is set to be 'dense'. The hover circle will get cut off or overlap rows and headers.

Expected behavior
The checkbox to adapt to the dense setting of the MudTable preventing the overlap.

**Screenshot taken from https://mudblazor.com/components/table#grouping-(basic) **
image

@Garderoben
Copy link
Member

Use a dense checkbox. https://mudblazor.com/components/checkbox#dense

@MaRRiK74
Copy link
Author

MaRRiK74 commented Aug 10, 2021

Those checkboxes are rendered by MudTable (multiselect) not added to each row! How do you replace them with the dense version? @Garderoben

@Garderoben
Copy link
Member

Those checkboxes are rendered by MudTable (multiselect) not added to each row! How do you replace them with the dense version? @Garderoben

Ops, My bad there went a little to fast.
I will fix it so dense table uses dense checkboxes.

Sorry for the misunderstanding.

@Garderoben
Copy link
Member

@MaRRiK74 Again, sorry for closing this, misunderstanding from my part. Good that you answered!
I fixed it now, and its in current dev and will be in next release.
TableCheckBoxDense

@Garderoben Garderoben added bug Something does not work as intended/expected fixed Fix has been merged and removed triage labels Aug 11, 2021
@Garderoben Garderoben added this to the 5.1.1 milestone Aug 11, 2021
@Garderoben
Copy link
Member

You can test it here: https://dev.mudblazor.com/components/table#grouping-(multi-levels)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something does not work as intended/expected fixed Fix has been merged
Projects
None yet
Development

No branches or pull requests

2 participants