-
Notifications
You must be signed in to change notification settings - Fork 2
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
[IOAPPFD0-171] Add the new LoadingSpinner
component
#97
Conversation
@LazyAfternoons Yes, I'm aware of the difference. But we could still release it with the prospect of refining it later. Also we're going to change the following things:
|
@LazyAfternoons Improved it by making it look like the designed one. It doesn't use the border hack anymore, but everything is made with a custom SVG. To summarize everything:
@CrisTofani Fixed the TS issue with the new page by deleting it. YOLO. |
Thank you, it looks good to me. I think the |
Yes, it's missing because we have a TS issue when we add the new page. But it's a minor issue because the official DS page will be in the official repository |
Short description
This PR adds the new
LoadingSpinner
component, derived fromItwLoadingSpinner
developed specifically for the IT Wallet flow. To learn more about the original component:List of changes proposed in this pull request
Preview
loading-spinner.mp4
How to test