rimble-ui
rimble-ui copied to clipboard
Decrease theme colors
Audit the theme colors and find ones that can be removed.
Also, consider adding variants for action colors that do not rely on opacity but instead change the hue via a color transform function like lighten (success-light).
Acceptance criteria:
- theme should only have appropriate colors
E: 1/2D
This should also get reflected in the Figma kit.
@gesquinca - does it make sense to rename to abstract additional color names to something less tied to color and more tied to function? When I want to override the color of the button text, instead of using text I have to set white: "black" which feels weird. Should there be a new theme.colors.buttonText value?
@MikeLockz yes it does. That's why we added these values to the theme recently.
https://github.com/ConsenSys/rimble-ui/blob/master/src/theme.js#L7-L8
This doesn't apply to button colors, but it applies to text color in the page as a whole.