Megabit / Blazorise

Blazorise is a component library built on top of Blazor with support for CSS frameworks like Bootstrap, Tailwind, Bulma, AntDesign, and Material.
https://blazorise.com/
Other
3.3k stars 533 forks source link

[Bug]: Dropdown Item Checkbox does not always toggle on click #5486

Closed InzeNL closed 6 months ago

InzeNL commented 6 months ago

Blazorise Version

1.5.2

What Blazorise provider are you running on?

Bootstrap4

Link to minimal reproduction or a simple code snippet

https://github.com/InzeNL/DropdownCheckboxBug

Steps to reproduce

What is expected?

Clicking this area inbetween the checkbox should check the checkbox, as the entire rest of the area of the dropdown item does. The padding around the text and checkbox, the text, and the checkbox itself, all toggle the checkbox.

What is actually happening?

The marked area is the only area that doesn't actually toggle the checkbox

What browsers do you see the problem on?

Chrome, Microsoft Edge, Firefox, Brave

Any additional comments?

No response

David-Moreira commented 6 months ago

This happens on both bootstrap and bootstrap5. And also on the regular checkbox.

This does happen in the bootstrap5 framework itself https://getbootstrap.com/docs/5.0/forms/checks-radios/#checks

While we could easily fix it by making the hidden checkbox input go a little wider to catch that blank space, I have to think, that maybe this is working as intended? Or that it should be fixed by the css framework itself? @stsrki thoughts?


@InzeNL As a workaround you can apply the fix yourself on your end try:

.custom-checkbox .custom-control-input{
width: 1.5rem
}

That should make the hidden input grow wide enough for the click detection to go through image

stsrki commented 6 months ago

I would rather that is behaves as per bootstrap documentation. It might be better to report it on their side and see if it is really a bug.

David-Moreira commented 6 months ago

@InzeNL As we generally try not to make too many changes to the css framework unless we deem necessary due to specific use cases with blazor + blazorise components, would you be so kind to report this issue on bootstrap? I'm going ahead and closing the issue. But feel free to reopen if you feel there's anything else in Blazorise side we can help you with regarding this matter.