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
Branch
show-active-wallet-name
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15211
Build 30360: Build Diffcashtab-tests
Build 30359: arc lint + arc unit

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

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

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