-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Revamp useOptimistic docs #8264
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
Revamp useOptimistic docs #8264
Conversation
Size changesDetails📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
|
added some initial nits. examples add a lot of clarity, so great work on those. i'd just slightly trim everything and focus it down so each part explains one aspect |
stephan-noel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a really nice improvement. Just some small nits and questions, but didn't get to the Usage section yet. Will have more in depth look at that later if it's still open to try to give more meaningful feedback.
9e57f8c to
fbdc328
Compare
Preview
Fixes #7737
Fixes #7379
Fixes #7098
Fixes #6926