Show wallet address after hovering on wallet name

tahowallet / extension

Taho, the community owned and operated Web3 wallet.

https://taho.xyz

GNU General Public License v3.0

3.12k stars 393 forks source link

Show wallet address after hovering on wallet name #3713

Closed xpaczka closed 1 month ago

xpaczka commented 11 months ago

Resolves #3171

What has been done

Show address to be copies in "Copy address" tooltip
In filters on account list hover, show the truncated address (instead of ens/uns name)

Testing

[ ] Hover over item on accounts list in filters - address shows

Zrzut ekranu 2023-12-19 o 11 29 19

Latest build: extension-builds-3713 (as of Thu, 21 Dec 2023 09:31:19 GMT).

jagodarybacka commented 11 months ago

Oh I think that it doesn't make sense to show the tooltip with the actual address here - it makes UI less readable IMO compared to having copy address CTA. Other places that were mentioned in the issue make more sense. We also have a component that will allow to copy address on click - like here:

I think we should be using this component wherever possible and show ENS/UNS instead of the short address - it makes the UI more user friendly and easier to work with.

Shadowfiend commented 1 month ago

Not relevant atm, let's clean it up. We can restore later if needed.