Why is the text color being hard-coded to white here? #43539
Answered
by
jasmussen
polarstoat
asked this question in
Q&A
-
I've just had an issue of Buttons with the 'Fill' style having their text color overridden by this snippet here, what's the purpose of it? It appears to take precedence over the text color which is set on |
Beta Was this translation helpful? Give feedback.
Answered by
jasmussen
Aug 24, 2022
Replies: 1 comment 1 reply
-
I believe the color was necessary to ensure contrast against the default dark gray background that was also provided. However it seems like those colors are now provided by a back-compat theme library, and can be removed here. I've created a PR to do that #43553 — but would appreciate input there. |
Beta Was this translation helpful? Give feedback.
1 reply
Answer selected by
polarstoat
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I believe the color was necessary to ensure contrast against the default dark gray background that was also provided.
However it seems like those colors are now provided by a back-compat theme library, and can be removed here. I've created a PR to do that #43553 — but would appreciate input there.