bitbox-wallet-app icon indicating copy to clipboard operation
bitbox-wallet-app copied to clipboard

frontend: use sat spacing in all-accounts

Open thisconnect opened this issue 9 months ago • 2 comments

Branched off of https://github.com/BitBoxSwiss/bitbox-wallet-app/pull/3420 (so this needs to be merged first)

  • use sat spacing in all-accounts menu
  • simplify and remove some dry code

Please note that this needed some CSS changes, so if we want sat spacing this needs a bit more visual testing

thisconnect avatar Jun 25 '25 09:06 thisconnect

Screenshot 2025-06-25 at 11 57 34

thisconnect avatar Jun 25 '25 09:06 thisconnect

Screenshot 2025-06-25 at 11 58 01

thisconnect avatar Jun 25 '25 09:06 thisconnect

@thisconnect , I see that it gets cut off at the end (TBTC) on smaller screens .

Screenshot 2025-07-28 at 10 45 12

Previously, if the balance does not fit, it's truncated at the end

Screenshot 2025-07-28 at 10 46 51

shonsirsha avatar Jul 28 '25 08:07 shonsirsha

@shonsirsha I added a 2nd commit to improve some layout issues PTAL.

Screenshot 2025-08-19 at 15 43 08 Screenshot 2025-08-19 at 15 42 56

thisconnect avatar Aug 19 '25 13:08 thisconnect

There is one more issue with the connected keystore name, which doesn't exist on the account-summary, but I'll open a separate PR for this. Looks like this could go into a shared component

see difference between account-summary and all-account

Screenshot 2025-08-19 at 15 47 24 Screenshot 2025-08-19 at 15 47 12

thisconnect avatar Aug 19 '25 13:08 thisconnect