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 click behavior change after double clicked disable checkbox items #3528

Closed
MinChanSike opened this issue Jul 22, 2024 · 7 comments · Fixed by #3552
Closed

Checkbox click behavior change after double clicked disable checkbox items #3528

MinChanSike opened this issue Jul 22, 2024 · 7 comments · Fixed by #3552
Labels
📦 Scope : Components Related to the components 🐛 Type: Bug Something isn't working

Comments

@MinChanSike
Copy link

MinChanSike commented Jul 22, 2024

NextUI Version

2.4.2 2.4.6

Describe the bug

Need double click to check/uncheck normal checkbox after clicking isDisabled checkbox item.

Screen.Recording.22-7-2024.4-47-00.Pm.mp4

Please take a look sample here.
https://codesandbox.io/p/devbox/ancient-hooks-7h9mnn

Your Example Website or App

No response

Steps to Reproduce the Bug or Issue

To simulate, only click checkbox check/uncheck container, does not click the label.

Expected behavior

Click behavior should not change to check/uncheck checkbox.

Screenshots or Videos

No response

Operating System Version

Windows

Browser

Chrome

Copy link

linear bot commented Jul 22, 2024

@wingkwong
Copy link
Member

Able to reproduce even in 2.4.6.

@wingkwong wingkwong added 🐛 Type: Bug Something isn't working 📦 Scope : Components Related to the components labels Jul 22, 2024
@sohan01fw
Copy link
Contributor

@MinChanSike i tried this in firefox it works fine but not in chrome browser.

@awesome-pro
Copy link
Contributor

@wingkwong bug reproducible with RadioGroup and. Switch too

@awesome-pro
Copy link
Contributor

@MinChanSike i tried this in firefox it works fine but not in chrome browser.

Only chrome is showing this behaviour, not in safari too

@zymalvin
Copy link

Is there anyone who has a solution?

@awesome-pro
Copy link
Contributor

@zymalvin issue solved :), you'll get it as soon it's merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 Scope : Components Related to the components 🐛 Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants