search

omni / bridge-ui

UI for TokenBridge, an interoperability solution between Ethereum networks for native and ERC tokens
https://bridge.poa.net/
41 stars 51 forks source link

UI update required #133

Closed Martin1209 closed 6 years ago

Martin1209 commented 6 years ago

image

What's metamask... ;)

rstormsf commented 6 years ago

metamask @Martin1209

@patitonar we should probably have it as link

acravenho commented 6 years ago

@rstormsf I think they're referring to updating to Nifty Wallet

Martin1209 commented 6 years ago

Sorry - I should have clarified!

I did mean that we should replace the references to Nifty wallet references, and perhaps a link too as you said.

akolotov commented 6 years ago

Hmm. But MetaMask will also work, right? Does it make sense to use a common name like "an extension providing Web3 interface" and links it to a page which provide the list of supported extensions?

acravenho commented 6 years ago

I think it might be a better user experience to list them both directly in the popup.

"You need to install Nifty Wallet or MetaMask and select an account."

Martin1209 commented 6 years ago

I agree with both of those, so maybe even a combination - "An extension providing Web3 interface such as Nifty Wallet or Metamask"

akolotov commented 6 years ago

Addressed in #138