Page MenuHomePhabricator

[Cashtab] show active wallet on home
ClosedPublic

Authored by josephroyking on Tue, Feb 16, 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
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Apply active wallet label in standardized way to all screens

Fabien requested changes to this revision.Wed, Feb 17, 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.Wed, Feb 17, 09:21

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

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