[Cashtab] Remove legacy css
ClosedPublic
Actions

Authored by bytesofman on Mar 4 2021, 21:47.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCdbf7fdac386e: [Cashtab] Remove legacy css

Summary

A good deal of css in App.css is not relevant to the Cashtab app. This diff removes css not relevant to present Cashtab functionality. It also adds comments for dev reference on how to refactor remaining css in future diffs.

Test Plan

npm start ; confirm app still looks ok.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

css-cleanup

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15318
Build 30565: Build Diff	cashtab-tests
Build 30564: arc lint + arc unit

Event Timeline

bytesofman created this revision.Mar 4 2021, 21:47

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 4 2021, 21:47

bytesofman requested review of this revision.Mar 4 2021, 21:47

Harbormaster completed remote builds in B15318: Diff 27883.Mar 4 2021, 21:50

PiRK accepted this revision.Mar 5 2021, 07:24

This revision is now accepted and ready to land.Mar 5 2021, 07:24

Closed by commit rABCdbf7fdac386e: [Cashtab] Remove legacy css (authored by Joey King <joseph.roy.king@gmail.com>). · Explain WhyMar 5 2021, 17:29

This revision was automatically updated to reflect the committed changes.

Joey King <joseph.roy.king@gmail.com> added a commit: rABCdbf7fdac386e: [Cashtab] Remove legacy css.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

components/

App.css

217 lines

Diff 27883

View Options

web/cashtab/src/components/App.css

[Cashtab] Remove legacy cssClosedPublicActions