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

Network spinner blocks custom RPC form #4479

Closed
danfinlay opened this issue Jun 4, 2018 · 4 comments
Closed

Network spinner blocks custom RPC form #4479

danfinlay opened this issue Jun 4, 2018 · 4 comments
Labels

Comments

@danfinlay
Copy link
Contributor

Story:

  • User wants to connect to a custom RPC
  • They enter the RPC wrong
  • The spinner appears
  • They click the network menu again to select custom again
  • They are taken back to settings, but cannot edit the form:

screen shot 2018-06-04 at 9 11 19 am

This enhancement needs design first:
MetaMask/Design#21

@bdresser
Copy link
Contributor

bdresser commented Jun 6, 2018

@danfinlay @cjeria when I put in a custom RPC, it adds that new URL to the list of networks.

If I click "Custom RPC" again, it feels like the extension should stop trying to connect to the previously entered URL, cancel the spinner, and drop me on a fresh settings page. (Right now it just continues to try connecting to the URL I entered)

Potentially a design-free solution?

@danfinlay
Copy link
Contributor Author

Right now this bug doesn't exist on old-UI because it just doesn't show the spinner on the settings page, which has the same ultimate effect: People can pick a new custom RPC even when they were just searching for one. I like this solution because it's easier to implement, but either would be good.

@cjeria
Copy link
Contributor

cjeria commented Jun 18, 2018

This sounds like a good opportunity to time-out with a message to "Try again" and link to the settings page per @bdresser suggestion. Easy enough to create a design for if this sounds like a desirable implementation.

@bdresser
Copy link
Contributor

closing in favor of #3776

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants