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

Progressively load gas estimations on confirm screen #6591

Closed
bdresser opened this issue May 9, 2019 · 1 comment
Closed

Progressively load gas estimations on confirm screen #6591

bdresser opened this issue May 9, 2019 · 1 comment
Assignees
Labels
area-transactions area-UI Relating to the user interface.

Comments

@bdresser
Copy link
Contributor

bdresser commented May 9, 2019

Right now, we block rendering the confirmation screen on gas estimation. We should render the screen, then fill in gas estimation inline to improve the loading time of confirmation screens.

@bdresser bdresser added area-transactions area-UI Relating to the user interface. labels May 9, 2019
@bdresser bdresser added this to the UI Sprint 12 [May 13] milestone May 9, 2019
@PhABC
Copy link

PhABC commented May 11, 2019

Please also allow for manual imput even if gas estimation and gas price aren't loaded. Right now, I personally can't manually set the gas price as my confirmation screen is stuck, as follow.

image

I am unable to manually change the gas price and gas limit value until the charts are loaded, which takes a very long.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-transactions area-UI Relating to the user interface.
Projects
None yet
Development

No branches or pull requests

4 participants