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

Improve UI on log-in modal. #350

Open
wants to merge 9 commits into
base: polygon-prototype-staging
Choose a base branch
from

Conversation

gianfra-t
Copy link
Contributor

@gianfra-t gianfra-t commented Jan 3, 2025

Closes: #351 (and #362)

A re-use of the signing box needs to show when the user attempts to login. This box should not show a progress bar on this stage.

Changes

  • Modify the SigningBox component to allow for this new "phase". No progress bar is shown.
  • Modify useSiweSignature hook to work without the modal.
  • Improve error display and recovery when user rejects signing/network change requests.

Copy link

netlify bot commented Jan 3, 2025

Deploy Preview for pendulum-pay ready!

Name Link
🔨 Latest commit 9b96dfa
🔍 Latest deploy log https://app.netlify.com/sites/pendulum-pay/deploys/67851d1eaca1f40008d54e53
😎 Deploy Preview https://deploy-preview-350--pendulum-pay.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@gianfra-t gianfra-t linked an issue Jan 3, 2025 that may be closed by this pull request
@gianfra-t gianfra-t changed the title [DRAFT] Improve UI on log-in modal. Improve UI on log-in modal. Jan 8, 2025
@gianfra-t gianfra-t changed the title Improve UI on log-in modal. [DRAFT] - Improve UI on log-in modal. Jan 8, 2025
@gianfra-t gianfra-t changed the title [DRAFT] - Improve UI on log-in modal. Improve UI on log-in modal. Jan 13, 2025
@gianfra-t gianfra-t requested a review from a team January 13, 2025 13:46
@@ -368,7 +374,7 @@ export const SwapPage = () => {
from,
selectedNetwork,
fromAmountString,
requiresSquidRouter: selectedNetwork === Networks.Polygon,
requiresSquidRouter: isNetworkEVM(selectedNetwork),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not related to this change, but we need this after support of other EVM networks.

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.

Improve UX on sign-in modal.
2 participants