Metamask-extension: Logged on testnet, then logged on main network, but showing yet the balance of testnet instead of a loading spinner

[DaniellMesquita]

[bdresser]

@DaniellMesquita I can't reproduce this – could you include more detailed steps on how you got in this state?

[DaniellMesquita]

The steps I remember are as follows in the title. Detail that I use a limited Internet connection with 3KBps. But it should show a spinner for loading balances when switching between networks.

[danjm]

@bdresser I was able to reproduce as described by @DaniellMesquita

Fix forthcoming

[danjm]

Actually, just adding a loading screen really harms the experience for users with faster connections. A flash of a loading screen on every network switch is non-ideal.

We perhaps need an animated transition of definite duration on the loading screen. Or in the case of a network change, instead of overlaying an entire loading screen, we could add a simple animated transition to the balance (e.g. fade out, fade in)

I'll put together an MVP of the latter and then run it by @bdresser and @cjeria

[bdresser]

@danjm is this solved by https://github.com/MetaMask/metamask-extension/pull/5854, or possible to include the cached indicator for wrong-network balances (in addition to stale ones) ?

[danjm]

@bdresser this is not fixed by the cached balances changes I made. I will do the 'animated transition' type solution I described previously and share a gif here within the next 24 hours.

[danjm]

@bdresser @DaniellMesquita Let me know what you think of the screen captures here: https://github.com/MetaMask/metamask-extension/pull/6193

Users will always see the loading spinner and fade out/in as is shown in those videos. The spinner will be shown until the network request resolves, so users will see it for as long as is needed.

[tmashuang]

Closing due to inactivity, my recent attempt to reproduce has been unsuccessful. If this issue has persisted please open a new issue.