Skip to content
This repository has been archived by the owner on Nov 8, 2023. It is now read-only.

rename l1/l2 pay and add chain / currency sumbols to UI #148

Closed
wants to merge 3 commits into from
Closed

Conversation

geoknee
Copy link
Contributor

@geoknee geoknee commented Oct 19, 2023

Closes #120

Screenshot 2023-10-19 at 10 22 11

@@ -280,7 +289,7 @@ const Wallet: React.FunctionComponent<{ role: Role }> = (props: {
});
}}
>
<BoltIcon /> L2 Pay
<BoltIcon /> Bridge to {myPeerChain.symbol}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could use the "name" field here, although that is typically a little too long to make the button look good...

We could just use "bridge" in the button, and display the chain name (and/or logo?) somewhere else on the UI?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i.e. Host Network / Target Network

@geoknee geoknee changed the title rename l1/l2 pay rename l1/l2 pay and add chain / currency sumbols to UI Oct 19, 2023
@NiloCK
Copy link
Contributor

NiloCK commented Oct 19, 2023

Slightly mixed minds about this - this is a tech demo for a tech audience, but a big benefit of this system is that it abstracts away "guts" from the end user, who probably benefits most from a simple "Pay" button denominated in their local currency.

I'd lean toward putting this in, but I wonder if there's a different way to emphasize the bridging / network hopping, while also demonstrating a very friendly UI to the end user.

@geoknee
Copy link
Contributor Author

geoknee commented Oct 19, 2023

"chain pay" / "bridge pay"?

@geoknee
Copy link
Contributor Author

geoknee commented Oct 19, 2023

And for consistency, "host chain" ("target chain" even)

@geoknee geoknee closed this Oct 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rename "L1/L2 pay"
2 participants