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
fix(Rating): fix issue with state snapshots and therefore provide decent support for mobile phones #2945
base: main
Are you sure you want to change the base?
Conversation
…ent support for mobile phones
Someone is attempting to deploy a commit to the rsuite Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 9eb8133:
|
Codecov ReportBase: 92.89% // Head: 92.87% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #2945 +/- ##
==========================================
- Coverage 92.89% 92.87% -0.03%
==========================================
Files 557 557
Lines 19898 19908 +10
Branches 2707 2709 +2
==========================================
+ Hits 18484 18489 +5
- Misses 764 766 +2
- Partials 650 653 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@simonguo or @SevenOutman can you please have a look? :) |
hi @LisaKirchhofer
|
Hi @simonguo! |
Corresponding issue can be found here: #2917
The problem was the following: 2 functions operating on the same state were wrapped within 1 parent function -> one of them setting a state, the other one reading the state. The reading function was expecting to get the new value from the state, set by the previous function. As react operates on snapshots, the reading function retrieved the old value instead of the new one. I took the opportunity and tried to refactor those parts.