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

Teleport v2 improvements #7363

Closed
exezbcz opened this issue Sep 25, 2023 · 7 comments · Fixed by #7946
Closed

Teleport v2 improvements #7363

exezbcz opened this issue Sep 25, 2023 · 7 comments · Fixed by #7946
Assignees
Labels
A-teleport related teleportation assets between para chains p3 non-core, affecting less than 40%

Comments

@exezbcz
Copy link
Member

exezbcz commented Sep 25, 2023

let's make some small improvements and refine the current teleport page; with more and more chains, it's becoming pretty hardly usable, and I believe it could be better
image

Main

Design

  • from top to bottom, not many changes, mainly visual and adding dropdowns instead of tabs
  • keeping the teleport to your address only. In future it would be somehow merged with transfer anyway

Top part

image

Source and destination

image

  • its needed to fill out these dropdowns. They are rounded this time

  • on the right from the designs, there is more on the dropdowns
    image

  • I think there could be polkadot -> polkadothub by default, as mentioned here: Wrong currency on the Teleport page #6740

Asset Amount

image

  • here, you will specify the asset amount
  • right now, its fixed to the one asset we support on both networks
  • under is balance with max button - this button has the same hover effects as in the profile sidebar
  • the input changes according to the designs - active black stroke

Proceed to confirmation

image

  • big button
  • we can keep the disabled states from the old version of teleport - insufficient funds, select destination chain etc.

Context message

image

  • same as the old version with "(you are owner) message

That is it. Feel free to ask, thanks!

@exezbcz exezbcz added p3 non-core, affecting less than 40% design-request A-teleport related teleportation assets between para chains labels Sep 25, 2023
@yangwao
Copy link
Member

yangwao commented Sep 26, 2023

Highlighting this as p1 as its currently major UX break for creators

@exezbcz
Copy link
Member Author

exezbcz commented Sep 28, 2023

updated and ready guys 🚀

@exezbcz exezbcz changed the title Teleport improvements Teleport v2 improvements Sep 28, 2023
@yangwao
Copy link
Member

yangwao commented Sep 28, 2023

Why are we redesigning the screen, which will be dissolved in the future?

Why should people visit one more extra screen? I think that will confuse them even more.

In favour, the teleport screen will go away in teleportation in action buttons

@exezbcz
Copy link
Member Author

exezbcz commented Sep 28, 2023

@yangwao because the user should have the option to teleport even if it's not combined with any action (auto bridging in action buttons - buy, make offer, mint...)

Why should people visit one more extra screen? I think that will confuse them even more.

They won't

In favour, the teleport screen will go away in teleportation in action buttons

What would you do if you just wanted to move your assets?

@exezbcz
Copy link
Member Author

exezbcz commented Oct 27, 2023

image

👀

@newraina
Copy link
Member

newraina commented Nov 3, 2023

👋🏻

@kodabot
Copy link
Collaborator

kodabot commented Nov 3, 2023

ASSIGNED - @newraina 🔒 LOCKED -> Saturday, November 4th 2023, 19:52:43 UTC -> 36 hours

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-teleport related teleportation assets between para chains p3 non-core, affecting less than 40%
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants