-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(linear-gradient): use transparent color instead of keyword for Safari fix #4859
fix(linear-gradient): use transparent color instead of keyword for Safari fix #4859
Conversation
Deploy preview for the-carbon-components ready! Built with commit 283cfe0 https://deploy-preview-4859--the-carbon-components.netlify.com |
Deploy preview for carbon-elements ready! Built with commit 283cfe0 |
Deploy preview for carbon-components-react ready! Built with commit 283cfe0 https://deploy-preview-4859--carbon-components-react.netlify.com |
Deploy preview for carbon-components-react ready! Built with commit 9882ca5 https://deploy-preview-4859--carbon-components-react.netlify.com |
Deploy preview for carbon-elements ready! Built with commit 9882ca5 |
Deploy preview for the-carbon-components ready! Built with commit 9882ca5 https://deploy-preview-4859--the-carbon-components.netlify.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! Thanks Jen!
Modal
in this PR, where the same situation applies to the content overflow linear-gradientSafari interprets the
transparent
keyword (which maps torgba(0, 0, 0, 0)
) differently, which causes the visual issue outlined in #4769. In short: this visual issue causes linear gradients used in theCodeSnippet
&Modal
components to appear darker, like a transparent black, when viewed in Safari.Here a couple links explaining this:
Changelog
Changed
rgba()
color function to make primary gradient color transparent, rather than usingtransparent
keyword, in both theCodeSnippet
andModal
components