[Cashtab] Better label for show seed switch
ClosedPublic
Actions

Authored by bytesofman on Apr 4 2024, 12:24.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC29de4acd6a4c: [Cashtab] Better label for show seed switch

Summary

The whole "i understand" thing does not make sense when the switch is on. hiding the label when it is on looks dumb. So, just simplify it.

Test Plan

proofread, npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

switch-display-improve

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 28318
Build 56180: Build Diff	cashtab-tests
Build 56179: arc lint + arc unit

Event Timeline

bytesofman created this revision.Apr 4 2024, 12:24

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 4 2024, 12:24

bytesofman published this revision for review.Apr 4 2024, 12:29

Harbormaster completed remote builds in B28318: Diff 46821.Apr 4 2024, 12:42

Fabien accepted this revision.Apr 4 2024, 18:38

This revision is now accepted and ready to land.Apr 4 2024, 18:38

Closed by commit rABC29de4acd6a4c: [Cashtab] Better label for show seed switch (authored by bytesofman). · Explain WhyApr 4 2024, 22:32

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC29de4acd6a4c: [Cashtab] Better label for show seed switch.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

components/

BackupWallet/

BackupWallet.js

2 lines

Diff 46821

View Options

cashtab/src/components/BackupWallet/BackupWallet.js

[Cashtab] Better label for show seed switchClosedPublicActions