HomePhabricator

[Cashtab] Move all colors to theme

Description

[Cashtab] Move all colors to theme

Summary:
Move css styles to theme for easier control of app styles.

This is a prereq for features like dark mode. It will also make a rebrand design much more straightforward.

It looks like a bunch of changes, but it's only replacing css colors in styled components with variables. The key file to review is themes.js. Snapshots and unit tests had to be updated to support the new ThemeProvider component.

Test Plan: npm start and compare to cashtabapp.com; styling should be identical except some dark gray text becomes black

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D9322

Details

Provenance
Joey King <joseph.roy.king@gmail.com>Authored on Mar 12 2021, 20:58
bytesofmanPushed on Mar 15 2021, 13:13
Reviewer
Restricted Project
Differential Revision
D9322: [Cashtab] Move all colors to theme
Parents
rABC9a8815856a28: [avalanche] discourage ava messages if avalanche is disabled
Branches
Unknown
Tags
Unknown

Event Timeline

Joey King <joseph.roy.king@gmail.com> committed rABC4f5b2fe83259: [Cashtab] Move all colors to theme (authored by Joey King <joseph.roy.king@gmail.com>).Mar 15 2021, 13:13