Skip to content
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

[Redesign] UI polish for Redeem view #2787

Conversation

emreboga
Copy link
Collaborator

@emreboga emreboga commented Oct 2, 2024

This PR addresses the UI polish requests which includes a redesign of the Circular Progress bar for both determinate and indeterminate cases (see below). It also cleans up a few more React warnings/errors.

You can see the update for smoother progress bar transition here:
https://www.loom.com/share/67634ad8fceb4d99a82804b39cefb3eb?sid=ba85d219-3077-4940-befc-6a48227d0af9

Screenshots for all UI states:

Screenshot 2024-10-02 at 3 42 11 PM Screenshot 2024-10-02 at 3 49 46 PM Screenshot 2024-10-02 at 5 10 23 PM Screenshot 2024-10-02 at 5 15 23 PM

Copy link

netlify bot commented Oct 2, 2024

👷 Deploy request for wormhole-connect pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 685479d

Copy link

netlify bot commented Oct 2, 2024

👷 Deploy request for wormhole-connect-mainnet pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 685479d

Signed-off-by: Emre Bogazliyanlioglu <emre@wormholelabs.xyz>
Signed-off-by: Emre Bogazliyanlioglu <emre@wormholelabs.xyz>
@emreboga emreboga force-pushed the emre/redesign-redeem-view-polish branch from 87087c5 to f68979b Compare October 2, 2024 19:41
Signed-off-by: Emre Bogazliyanlioglu <emre@wormholelabs.xyz>
Signed-off-by: Emre Bogazliyanlioglu <emre@wormholelabs.xyz>
@emreboga emreboga merged commit d317894 into wormhole-foundation:development Oct 7, 2024
8 checks passed
@emreboga emreboga deleted the emre/redesign-redeem-view-polish branch October 7, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants