status-im / ens-usernames

DApp to register usernames for Status Network
MIT License
19 stars 11 forks source link

Redesign error screen for user not on mainnet #22

Closed rachelhamlin closed 5 years ago

rachelhamlin commented 5 years ago

The DApp will be launched publicly on mainnet. Any user who interacts with it on a testnet, like Ropsten, will be unable to search.

Use the styling below to present a warning message that says,

Please connect to Ethereum mainnet to continue.

(Font + sizing info in Figma)

image

This will replace the current error message for the testnet version of the DApp:

image

bgits commented 5 years ago

It's already there, when we deploy it to mainnet, Ropsten will be replaced with mainnet.

bgits commented 5 years ago

The functionality, not the new design

rachelhamlin commented 5 years ago

@bgits wdyt about borrowing this other error screen for the design?

I know the copy isn't super specific—e.g. if I'm not in a DApp browser at all I won't understand—but we can address that later as well

bgits commented 5 years ago

We already have an error screen for when the user is not on the correct network. It's used in two dapps, go to vote.status.im while connected to any network but mainnet. Is this purely an aesthetics issue?

rachelhamlin commented 5 years ago

Yup, I think it makes sense to have consistent styling across all screens. From what I remember the other error screen doesn't fit mobile either.

bgits commented 5 years ago

So maybe this issue should be called, redesign error screen? In terms of priorities where do you see this?

rachelhamlin commented 5 years ago

Sure thing, and pretty low. Leo or even Alex might tackle? Will update when I’m at my computer later.

On Mon, Oct 1, 2018 at 5:30 PM Barry G notifications@github.com wrote:

So maybe this issue should be called, redesign error screen? In terms of priorities where do you see this?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/status-im/ens-usernames/issues/22#issuecomment-426069928, or mute the thread https://github.com/notifications/unsubscribe-auth/AOwPYzbXJgq8YdxcRyxdPeW6LF1OYaDdks5ugol3gaJpZM4W-PGp .