-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
No support for currentcolor
#170
Comments
Sorry - I misunderstood this request. I can only think of one use-case that would actually work, if you have Text container B nested in Text container A, if B had a colour set by default and you wanted to unset it and inherit the text colour, you could use However, at the same time, I think people would expect this to work across the board - i.e. borders, backgrounds, non-text elements etc. and that's not possible without layering a more sophisticated CSS runtime, which is explicitly out of scope for this library I'll leave this open because we technically can achieve some of this request - but I'm undecided if we should |
@jacobp100 no worries, I definitely could've provided more context! And yes, instead of throwing an error when the value is |
I was wondering if adding an extra check for
node.value === 'currentcolor'
would cause large issues in thematchColor
function.The text was updated successfully, but these errors were encountered: