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

feat: Wallet account view 4 #678

Open
wants to merge 52 commits into
base: develop
Choose a base branch
from
Open

feat: Wallet account view 4 #678

wants to merge 52 commits into from

Conversation

janmichek
Copy link
Collaborator

Description

fixes #536

There are 2 minor UX issues I will try to resolve in the following (Portfolio) issue, so I don't block the review

  1. After successful wallet connection there is no loader when redirecting to connected account detail
  2. When clicking on Try again, the whole page gets reloaded, which is not necessary. Only scanning again should work, but for some reason it's not working Wallet re-connection #572

Demo

firefox_BZzKh8r8cp.mp4

Checklist:

Copy link

Copy link
Collaborator

@michele-franchi michele-franchi left a comment

Choose a reason for hiding this comment

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

Generally it looks good but I would adjust the following:

  1. The "Connect Wallet" button should have the same height of the network switcher:
image

padding: var(--space-1); on the button solves it.

  1. The "Exit Wallet" dropdown should be visually similar to the menu dropdowns.
image

Example:

image

Without the min-width of the other menu items.

  1. When you connect the wallet and you refresh the page the connection is lost.

.env.example Outdated Show resolved Hide resolved
nuxt.config.ts Show resolved Hide resolved
src/components/TheWalletAccountControls.vue Outdated Show resolved Hide resolved
src/components/WalletConnectionPanel.vue Outdated Show resolved Hide resolved
src/utils/hints/walletHints.js Outdated Show resolved Hide resolved
janmichek and others added 6 commits March 19, 2024 09:22
Co-authored-by: Michele F. <michele-franchi@users.noreply.github.com>
Co-authored-by: Michele F. <michele-franchi@users.noreply.github.com>
@janmichek
Copy link
Collaborator Author

Generally it looks good but I would adjust the following:

1. The "Connect Wallet" button should have the same height of the network switcher:
image

padding: var(--space-1); on the button solves it.

2. The "Exit Wallet" dropdown should be visually similar to the menu dropdowns.
image

Example:
image

Without the min-width of the other menu items.

3. When you connect the wallet and you refresh the page the connection is lost.

I have been discussing this with Denis. Currently is not possible to achieve this seamlessly. The app can remember last used account, but we cannot go over the user prompt to connect to the wallet again like on superhero.com
Implementation-wise, the SDK can do something to simplify it, but fundamentally will stay the same.

For now, I suggest not to implement it, wait for SDK completion, and the implement the persistency of last logged address @michele-franchi

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.

Wallet account view
2 participants