Page MenuHomePhabricator

[Cashtab] show active wallet on home
ClosedPublic

Authored by josephroyking on Feb 16 2021, 14:20.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC5a0f74cdd8b5: [Cashtab] show active wallet on home
Summary

Show "Wallet Name: walletname" on home screen so that user knows which wallet balance is displayed.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Apply active wallet label in standardized way to all screens

Fabien requested changes to this revision.Feb 17 2021, 09:21
Fabien added a subscriber: Fabien.

Please update the summary to reflect the current diff content

web/cashtab/src/components/Common/WalletLabel.js
5 ↗(On Diff #27712)

This is not very large when compared to other texts, but this is a pretty important one. Can it be slightly larger and/or bold ?

14 ↗(On Diff #27712)

The Wallet: prefix is not really useful imo, it is obvious enough without it and you get some display room

This revision now requires changes to proceed.Feb 17 2021, 09:21

Bold, bump font size, remove margin bottom, remove unnecessary !important flag

This revision is now accepted and ready to land.Feb 17 2021, 19:17