Page MenuHomePhabricator

[Cashtab] Add custom styling to Antd Descriptions
ClosedPublic

Authored by kieran709 on Jan 18 2022, 20:21.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Commits
rABC1c524ade67d3: [Cashtab] Add custom styling to Antd Descriptions
Summary

In order to make further progress with the dark theme, the antd Descriptions in SendToken.js need custom CSS. Currently, it is not possible to change the background and foreground colors in left most column on the table that contains the token's stats. If left unchanged the column would maintain its white background and dark text when the app is toggled to the dark theme. In order to exercise more control over the table a whole, I have included both columns (.ant-descriptions-item-lable & .ant-descriptions-item-content) in AntDescriptionsCss. Related to task T2173.

Test Plan

cd web/cashtab
npm start
navigate to send token page
ensure that content in the table is legible
In your IDE:
In theme.js, change the value of props.theme.tokenListItem.background and props.theme.tokenListItem.color
In browser:
navigate to the Send Token page.
Ensure that the colors selected in the previous step match what is rendering on the screen.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
custom-css-antd-descriptions
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17959
Build 35740: Build Diffcashtab-tests
Build 35739: arc lint + arc unit

Event Timeline

Please expand on the description of what is going on here. It looks like you are trying to put more css controls into the the theme.js object -- which is good -- but it is not clear why this is necessary. Is the issue that the current values being used are antd defaults, but we need to be able to change them? If so, please clarify. The test plan should also include changing colors in theme.js and confirming that they change on this screen.

This revision now requires changes to proceed.Jan 20 2022, 16:51
kieran709 edited the test plan for this revision. (Show Details)
kieran709 edited the test plan for this revision. (Show Details)
kieran709 edited the summary of this revision. (Show Details)
kieran709 edited the test plan for this revision. (Show Details)

Responded to review feedback and updated snapshots.

This revision is now accepted and ready to land.Jan 20 2022, 18:59