Related to task T2177, bg color needs to be added to ant-select-selector in EnhancedInputs.js.
Details
Details
- Reviewers
bytesofman - Group Reviewers
Restricted Project - Commits
- rABC45d0c6fd9ea8: [Cashtab] Add bg color ant select-selector
cd web/cashtab
npm start
ensure that drop down menus in Send.js and Configure.js have an appropriate bg color and that content is legible
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| web/cashtab/src/components/Common/EnhancedInputs.js | ||
|---|---|---|
| 52 ↗ | (On Diff #31846) | This changes the color from white to grey. This diff should keep the color constant. Is there a reason you are not using props.theme.forms.selectionBackground (which already seems to be applied to this component)? What color are you trying to control here? |
Comment Actions
Responding to review feedback, changed reference in EnhancedInputs.js at line 52 to selectionBackground. Added inline comment to give better understanding of what color I am trying to change. Updated snapshots.
| web/cashtab/src/components/Common/EnhancedInputs.js | ||
|---|---|---|
| 52 ↗ | (On Diff #31846) | This comment lead me to realizing that my monitor was not callibrated right because I could not see the difference in the two colors. I am trying to change the background and foreground color of the following input:  Without the added reference here, all of the other inputs change color, but not this one. |