-
-
Notifications
You must be signed in to change notification settings - Fork 41
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
[examples] Fix vite example & add dark mode support #340
Conversation
examples/pigment-css-vite-ts/src/components/ColorSchemeProvider.tsx
Outdated
Show resolved
Hide resolved
@mnajdova @brijeshb42 Would this aim to fix mui/material-ui#43487 (comment)? This used to work, but no longer appears to be. |
Yes. This should have fixed the issue. Can you try the latest example? |
Are you trying the example from Material ui repo or Pigment CSS repo?
…On Thu, Dec 5, 2024, 3:48 PM Brandon Fitzwater ***@***.***> wrote:
Yes. This should have fixed the issue. Can you try the latest example?
I just tried the example and I'm still receiving this Screenshot.2024-12-05.at.5.17.56.AM.png
(view on web)
<https://github.com/user-attachments/assets/26bceb37-58cd-4137-b79d-7fece87c9d83>
—
Reply to this email directly, view it on GitHub
<#340 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFPF3U7SMOKIB7STVRY7CT2EASAXAVCNFSM6AAAAABTAMCKHGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJZHA3DMNZRGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I had been using the example from the Material UI repository. It had worked well until recently. I tried the Pigment CSS repository and it looks like it does render, so I guess I'll just have to find the delta between the two repositories. Any ideas off the top of your head? |
It works similar to the Material UI + Pigment CSS next.js example in terms of dark mode support.