Paths

Table of Contentst

Differential D17034

[Cashtab] Add margin to info boxes
ClosedPublic
Actions

Authored by bytesofman on Tue, Oct 29, 13:17.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC70d99c651d29: [Cashtab] Add margin to info boxes

Summary

These will sometimes appear next to each other on the new token landing page. Make sure they are not touching.

This should apply to them everywhere. You can't really tell a difference at the moment, since this only happens on the WalletInfo page.

Test Plan

npm test, confirm it's a style change related to spacing of info boxes

still looks ok

Diff Detail

Repository

rABC Bitcoin ABC

Branch

style-portability

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 30898
Build 61299: Build Diff	cashtab-tests
Build 61298: arc lint + arc unit

Event Timeline

bytesofman created this revision.Tue, Oct 29, 13:17

Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Oct 29, 13:17

bytesofman edited the test plan for this revision. (Show Details)Tue, Oct 29, 13:18

bytesofman added a child revision: D17035: [Cashtab] Remove repeated and unnecessary specialized styled component.Tue, Oct 29, 13:24

Harbormaster completed remote builds in B30898: Diff 50521.Tue, Oct 29, 13:31

bytesofman published this revision for review.Tue, Oct 29, 14:04

emack accepted this revision.Wed, Oct 30, 01:39

This revision is now accepted and ready to land.Wed, Oct 30, 01:39

Closed by commit rABC70d99c651d29: [Cashtab] Add margin to info boxes (authored by bytesofman). · Explain WhyWed, Oct 30, 02:11

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC70d99c651d29: [Cashtab] Add margin to info boxes.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

components/

Common/

Atoms.js

2 lines

Diff 50521

View Options

cashtab/src/components/Common/Atoms.js

[Cashtab] Add margin to info boxesClosedPublicActions