-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
disabled attribute doesn't work on <Button> #7905
Comments
Wanted to add that this ended up being a big issue for our company. We upgraded Chakra a few weeks ago, not realizing that this update broke most of our form buttons from disabling during dialog submissions. So we've had all kinds of data issues in production from people double-clicking the submit button, requiring time from people in the ops department, coordinating with multiple clients, etc. Not only is it a semver issue, but it doesn't look like the change even got into the changelog? I searched through each version on https://chakra-ui.com/changelog/ and don't see the note from Yes, the docs don't seem to mention |
This reverts commit 0f37665. Fixes chakra-ui#7269, chakra-ui#7816 and chakra-ui#7905 and also enables the only currently known workaround for chakra-ui#7965. Removing the ability to use the disabled-prop posed a breaking change, which is hereby fixed.
* Revert "refactor: button disabled prop" This reverts commit 0f37665. Fixes #7269, #7816 and #7905 and also enables the only currently known workaround for #7965. Removing the ability to use the disabled-prop posed a breaking change, which is hereby fixed. * docs: update --------- Co-authored-by: Segun Adebayo <joseshegs@gmail.com>
@BANOnotIT #8462 has been merged, If satisfied, you may close this issue, the fix should be shipping with Chakra 2.8.3. |
It's great! But since this thing took so long to revert we just migrated to isDisabled so I don't have code to check whether it is fully compatible. I suppose the issue can be closed |
Description
When I update
@chakra-ui/button
from 2.0.13 to 2.1.0 disable attribute doesn't work because of 0f37665. That's violation of SemVer because:Link to Reproduction
https://example.com
Steps to reproduce
No response
Chakra UI Version
2.8.0
Browser
No response
Operating System
Additional Information
No response
The text was updated successfully, but these errors were encountered: