Skip to content

Fix useOptimistic demo bug #6422

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

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

kmiddleton14
Copy link
Contributor

@kmiddleton14 kmiddleton14 commented Nov 8, 2023

Before

Before.mp4

After

After.mp4

Copy link

github-actions bot commented Nov 8, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@kmiddleton14 kmiddleton14 marked this pull request as ready for review November 8, 2023 18:08
@kmiddleton14 kmiddleton14 merged commit 49ebed1 into reactjs:main Nov 8, 2023
@Vardhaman619
Copy link

useOptimistic demo form action runs when input value is empty
This PR fixes that issue.

subliun added a commit to subliun/react.dev that referenced this pull request Feb 22, 2025
This is the exact same fix as reactjs#6422 as the example is duplicated between the useOptimistic and <form> pages.
gaearon pushed a commit that referenced this pull request Feb 22, 2025
This is the exact same fix as #6422 as the example is duplicated between the useOptimistic and <form> pages.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants