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

New onboarding/notifications popup item buttons #2922

Closed
1 of 6 tasks
stackingsaunter opened this issue Dec 11, 2023 · 1 comment · Fixed by #2983
Closed
1 of 6 tasks

New onboarding/notifications popup item buttons #2922

stackingsaunter opened this issue Dec 11, 2023 · 1 comment · Fixed by #2983
Labels
frontend All things frontend development related

Comments

@stackingsaunter
Copy link
Contributor

stackingsaunter commented Dec 11, 2023

Part of #2917 epic

Component used in the current "Receive" screens, after some updates will be reused instead of current "onboarding cards".

See Figma file for detailed design inspect

Styling

CleanShot 2024-01-12 at 11 16 37@2x

CleanShot 2023-12-13 at 02 20 26@2x

To do

  • corner radius to rounded-xl (12px)
  • use Popicons icons and update "Receive" icons too Change iconset to Popicons #2920 (comment) (excep Chevrons, as there's no one in Popicons atm)
  • copy colors
  • chevron color change to gray400 / dark: neutral500
  • spacing between cards sm (8px)
  • make sure Receive screen uses the same component

image

image

Behaviour

  1. Whenever connected wallet has no transactions, 4 cards should be visible (3 + 1 after scroll)
    (We can skip "Buy bitcoin" card for now since onramps are turned off for the moment)

CleanShot 2023-12-13 at 02 41 20@2x

  1. When transaction list is loaded, then:
    a) If user with this connected wallet went through Master Key backup or import flow, we don't show any cards
    b) If user hasn't finished either import or backup flow, we always show "backup" and "import" cards

image

@MayaSatishRao
Copy link

@stackingsaunter i want to contribute to this issue? Can i work on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend All things frontend development related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants