-
-
Notifications
You must be signed in to change notification settings - Fork 2
[color-swatch] Fix popovers with color details in <color-chart>s
#202
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for color-elements ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
5df1843 to
1342729
Compare
… Safari Safari appears to have issues with combining styles within container queries with popover styles, so we need to move them out.
1342729 to
48eee8c
Compare
Now it works not only in Chrome, but also in Safari and Firefox. Support for `popover` is much wider than support for style queries.
48eee8c to
ea4d6f2
Compare
<color-chart>s in Safari<color-chart>s
|
Safari supports style queries, why wasn't it working in Safari? |
I still don't have an answer to this question. I can't reproduce the issue. In simple cases, everything works as expected. However, in our case, we have a hierarchy of custom elements: some are within a component's light DOM, while others are in the shadow DOM. And since we rely on many edge features, some may not play well together. I don't have another explanation for now. 🤷♂️ |
Found it! And already filed a bug report. Let's consider this PR as a workaround. 😛 Btw, style queries work correctly when |
Now it works not only in Chrome, but also in Safari and Firefox. Support for
popoveris much wider than support for style queries. For example, Firefox still lacks them.Open the link in Safari/Firefox for preview: https://deploy-preview-202--color-elements.netlify.app/src/color-chart/
@LeaVerou, could you please take another look at this PR? It turned out that it solves issues not only in Safari.