zilpay / zil-pay

ZilPay browser extension enables browsing Zilliqa blockchain enabled websites.
https://zilpay.io
Other
3 stars 1 forks source link

Sometimes users can get confused with the multiple windows #52

Closed ghost closed 2 years ago

ghost commented 3 years ago

Is your feature request related to a problem? Please describe.

https://github.com/Zilliqa/neo-savant/issues/123 is a case where a user can get confused with the UI:

A case where Alice get confused with multiple windows:

  1. Alice sets gas limit 1 and tries to deploy a contract to see it fails by opening a window (w1).
  2. Alice notices the error message Gas limit(1) lower than minimum for creating contract (1163)
  3. Alice clicks the outside of w1 and doesn't close it
  4. Alice sets the gas price to 2500 and tries to deploy a contract to see it succeeded by opening a window (w2).
  5. Alice sees the previous window without the error message Gas limit(1) lower than minimum for creating contract (1163)
  6. Alice thinks the gas limit is reverted to previous value in w2.

What happened actually: We know that actually the w2 window just was closed and the w1 is being displayed.

Here is the screen recording which can elaborate the above confusion. https://github.com/Zilliqa/neo-savant/issues/123#issuecomment-925485405

Key Question: Is there any good solution to prevent this kind of confusion?

hicaru commented 3 years ago

Hello thanks for your issue, now Zilliqa have no API for calculate gasLimit

hicaru commented 2 years ago

fixed in 1.0.0 version