-
Notifications
You must be signed in to change notification settings - Fork 17
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
Allow user to dismiss spinner early #7
Comments
Hi @dt2patel can I try this? |
Sure! |
Hi @dt2patel can you tell me in which component this spinner is? |
@mtwn105 You need to handle the situation in all the components where spinner is used and make them dismissible so, if the API takes too long to respond then the user can dismiss the spinner. |
@ymaheshwari1 can you share a reference link for @mtwn105 to sample. |
@mtwn105 Like here (https://github.com/hotwax/clienteling/blob/main/src/app/pages/product/product.ts#L294) when moving to the product-review page, we have started showing the spinner but in case if the routing is not completed then the user will be blocked by the spinner, so need to handle all such cases. |
Thanks. Created PR #11 |
Some times spinners with API calls can get stuck blocking the app. need to make these dismissible or not app blocking.
The text was updated successfully, but these errors were encountered: