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: Loader is not displayed in Save button while creating an entry for the collection-type #15188
fix: Loader is not displayed in Save button while creating an entry for the collection-type #15188
Conversation
Codecov ReportBase: 60.52% // Head: 60.52% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #15188 +/- ##
=======================================
Coverage 60.52% 60.52%
=======================================
Files 1352 1352
Lines 33172 33172
Branches 6334 6334
=======================================
Hits 20078 20078
Misses 11263 11263
Partials 1831 1831
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. |
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.
nice catch, thank you 💃
This PR was pushed through Gitstart, with contributions from @Beejay-bot, @raph941, @phunguyenmurcul. |
What does it do?
Use
loading
instead ofisLoading
(wrong name) in Save button props binding.Why is it needed?
The loading icon cannot be displayed when the save button is hit because the isLoading prop is not a valid attribute. Loading is the appropriate and correct prop because it enables the display of the loading icon while performing an action.
How to test it?
Related issue(s)/PR(s)
Fixes #15177
Demo
Screen.Recording.2022-12-15.at.4.33.45.PM.mov