metamask-extension icon indicating copy to clipboard operation
metamask-extension copied to clipboard

Account display screen width error

Open wbt opened this issue 7 years ago • 15 comments

Describe the bug When an account has a ridiculously high ETH balance, the default MetaMask account display has width issues when displaying the balance. For example, see the left and right sides of this display:

still not wide enough in beta

To Reproduce Steps to reproduce the behavior:

  1. Use Geth to set up a simple private network in which one account has an extremely large balance.
  2. Use MetaMask to connect to that private network and install the private key for the rich account. (Note that the previous bug which prevented Metamask from connecting to a private network in the new UI appears to have been fixed.)
  3. View the account.

Expected behavior The text should fit in the display, especially since it's already being abbreviated with scientific notation.
A very large balance may be elided (e.g. "904...465.124 ETH") similar to how the address is (and you should be able to click on it to copy the full value, as with the address) OR the balance may use fewer significant digits in scientific notation. In the left side of the screenshot shown, the fiat conversion also doesn't fit on the screen despite scientific notation being used.
The elided option is preferred as it allows people to more easily track changes in the smaller-value digits where changes are more likely to occur and may be of interest.

Browser details (please complete the following information):

  • OS: Windows 10
  • Browser Chrome 69.0.3497.100
  • MetaMask Version 4.16.0
  • New UI

wbt avatar Oct 19 '18 21:10 wbt

Even with scientific notation, the font size is too big for the width of a large balance.

To Reproduce Steps to reproduce the behavior:

  1. Use Geth to set up a simple private network in which one account has an extremely large balance.
  2. Use MetaMask to connect to that private network and install the private key for the rich account.
  3. View the account: Screenshot or with government currency as primary: Screenshot

Expected behavior The number before the decimal point and the full currency symbol are visible, not just when hovering over the value. If there are fewer digits displayed, OK. If the ellipsis is moved farther left, that's OK too.

Browser details (please complete the following information):

  • OS: Windows 10
  • Browser: Chrome 70.0.3538.77 (latest stable official 64-bit)
  • MetaMask Version 5.0.2

wbt avatar Nov 13 '18 20:11 wbt

@wbt Thanks for reporting this. Currently working on a pull request that will fix this.

alextsg avatar Nov 13 '18 21:11 alextsg

Might the issue be reopened, then, as a reminder that there's still something to work on? I just reinstalled with the latest stable release 6.1.0, and the problem seems to be getting worse rather than better.
image

wbt avatar Feb 21 '19 18:02 wbt

comment peux t on consulter notre solde

damster7 avatar Feb 22 '19 06:02 damster7

comment peux t on consulter notre solde

Translation from French: "how can we see our balance"

What I thought might be an answer / possible workaround for that purpose: Go into Settings, and change Primary Currency between "ETH" and "Fiat" and then look at the smaller text underneath. That doesn't work either, though:

image

If you right-click on the number you wish to see, then choose Inspect, you should be able to see the full value there: image

Definitely not a non-developer-user-friendly workaround!

wbt avatar Feb 22 '19 17:02 wbt

Not sure if this issue is still live, but @rachelcope is redoing our number precision soon, so this is already addressed in another issue. Rachel can you link to that issue here?

danfinlay avatar Jan 19 '21 20:01 danfinlay

We are actively improving on showing the exact amount values and truncating the width with an ellipsis, designs are in progress.

tmashuang avatar Mar 16 '21 17:03 tmashuang

Easiest way to repro this for a developer is to just inspect the code and edit the html through the dev tools.

Screenshot from 2021-10-18 07-48-54

danjm avatar Oct 18 '21 10:10 danjm

@danjm related to the display of the amount we need design decision how to show the large numbers. In the ticket there are two suggestions:

  1. A very large balance may be elided (e.g. "904...465.124 ETH") similar to how the address is (and you should be able to click on it to copy the full value, as with the address) OR
  2. The balance may use fewer significant digits in scientific notation.

Also, is this one resolved or partially resolved in the issue: #5570

EHaracic avatar Oct 19 '21 09:10 EHaracic

Marking this as blocked for now. I will adjust it after it gets some design attention

danjm avatar Oct 22 '21 18:10 danjm

This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 45 days if there is no further activity. The MetaMask team intends on reviewing this issue before close, and removing the stale label if it is still a bug. We welcome new comments on this issue. We do not intend on closing issues if they report bugs that are still reproducible. Thank you for your contributions.

github-actions[bot] avatar Sep 04 '23 18:09 github-actions[bot]

I don't think this should be closed without an improvement.

wbt avatar Oct 09 '23 17:10 wbt

This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 45 days if there is no further activity. The MetaMask team intends on reviewing this issue before close, and removing the stale label if it is still a bug. We welcome new comments on this issue. We do not intend on closing issues if they report bugs that are still reproducible. Thank you for your contributions.

github-actions[bot] avatar Jan 07 '24 20:01 github-actions[bot]

I still don't think this should be closed without an improvement.

wbt avatar Jan 18 '24 03:01 wbt

This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 45 days if there is no further activity. The MetaMask team intends on reviewing this issue before close, and removing the stale label if it is still a bug. We welcome new comments on this issue. We do not intend on closing issues if they report bugs that are still reproducible. Thank you for your contributions.

github-actions[bot] avatar May 09 '24 14:05 github-actions[bot]

I don't think this should be closed without an improvement.

wbt avatar Jun 10 '24 14:06 wbt