sui icon indicating copy to clipboard operation
sui copied to clipboard

Published 20 hours ago • verified publisher icon MystenLabs

[Wallet] Ability to change current page via progress bar

Open avently opened this issue 2 years ago • 3 comments

Description

When a user creates new transaction in the wallet and wants to go to the previous screen to change amount, he must use back button which is small and is not convenient to press. Also the user can think that pressing on the back button will drop him to the balance page. I think it would be better to allow pressing on a progress bar at the top.

So, for example, when you entered a transaction amount and pressed Continue, you may press on the Amount tab at the top which will show the first page with amount selection.

Pressing on the next pages in the progress bar will do nothing since data on pages should be verified first.

Test Plan

I cloned the repo, made changes while viewing the results in Chrome browser as an unpacked extension.

Type of Change (Check all that apply)

  • [x] user-visible impact
  • [ ] breaking change for a client SDKs
  • [ ] breaking change for FNs (FN binary must upgrade)
  • [ ] breaking change for validators or node operators (must upgrade binaries)
  • [ ] breaking change for on-chain data layout
  • [ ] necessitate either a data wipe or data migration

Release notes

  • Ability to change an active page in creating transaction screen by clicking on a top progress bar (that shows all pages in one line). If the previous page was selected, it will be opened. Otherwise, nothing will happen (next pages have no reaction) sui-wallet

avently avatar Feb 23 '23 03:02 avently

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Ignored Deployments
Name Status Preview Comments Updated
explorer ⬜️ Ignored (Inspect) Feb 23, 2023 at 3:26AM (UTC)
explorer-storybook ⬜️ Ignored (Inspect) Feb 23, 2023 at 3:26AM (UTC)
frenemies ⬜️ Ignored (Inspect) Feb 23, 2023 at 3:26AM (UTC)
wallet-adapter ⬜️ Ignored (Inspect) Feb 23, 2023 at 3:26AM (UTC)

vercel[bot] avatar Feb 23 '23 03:02 vercel[bot]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Apr 25 '23 01:04 github-actions[bot]

Are you interested in merging this?

avently avatar Apr 25 '23 07:04 avently