[Checkbox] Add support for CSS variables #32579
Conversation
|
These images below represent the change in their hover background when I switched to using Using theme.vars Not using theme.vars Can you please explain what's going on here wrong with my code? |
| theme.palette.action.hoverOpacity, | ||
| ), | ||
| backgroundColor: theme.vars | ||
| ? `rgba(${theme.vars.palette.text.primaryChannel} / ${theme.vars.palette.action.hoverOpacity})` |
There was a problem hiding this comment.
You need to add here too the check for the ownerState.color that you have below.
There was a problem hiding this comment.
We don't have action.activeChannel yet, seems like we have to add it or find other alternatives.
There was a problem hiding this comment.
I have fixed the hover background color by applying using palette.primary.mainChannel and palette.action.hoverOpacity.
How do I do the check for ownerState.color, I didn't understand that part ?
There was a problem hiding this comment.
It's the same as in #32599 (comment), let's wait for it to be resolved there and we can fix it in this PR too.
There was a problem hiding this comment.
Note, I just merged #32599. Feel free to rebase and you will have the activeChannel in your theme.
See #32579 (comment) |
danilo-leal
added
component: checkbox
Support for CSS variables with
<CheckBox />