Skip to content
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

With Pigment CSS, it's unclear how to manually switch palette mode (light / dark) #44384

Open
aleksei-berezkin opened this issue Nov 11, 2024 · 2 comments
Assignees
Labels
package: material-ui Specific to @mui/material package: pigment-css Specific to @pigment-css/* support: docs-feedback Feedback from documentation page v6.x migration

Comments

@aleksei-berezkin
Copy link

aleksei-berezkin commented Nov 11, 2024

Steps to reproduce

Link to live example: https://github.com/aleksei-berezkin/mui-pigment-example/

Steps:

  1. Try to find some info in MUI migration guide — no info
  2. According to Pigment CSS Readme, try to add getSelector somewhere in theme created in next.config.ts

Current behavior

  • Not possible to add getSelector into an object passed to createTheme() — doesn't compile
  • Changing createTheme() to extendTheme() (from '@pigment-css/nextjs-plugin') doesn't work — theme.spacing is now undefined in page.tsx
  • Changing to extendTheme({...createTheme()}) doesn't compile

Expected behavior

There should be a documented way to switch themes manually

Context

No response

Your environment

System:
  OS: macOS 14.0
Binaries:
  Node: 20.14.0 - /opt/homebrew/opt/node@20/bin/node
  npm: 10.7.0 - /opt/homebrew/opt/node@20/bin/npm
  pnpm: 9.11.0 - /opt/homebrew/opt/node@20/bin/pnpm
Browsers:
  Chrome: 130.0.6723.117
  Edge: Not Found
  Safari: 17.0
npmPackages:
  @emotion/react: ^11.13.3 => 11.13.3 
  @emotion/styled: ^11.13.0 => 11.13.0 
  @mui/core-downloads-tracker:  6.1.6 
  @mui/material: ^6.1.6 => 6.1.6 
  @mui/material-pigment-css: ^6.1.6 => 6.1.6 
  @mui/private-theming:  6.1.6 
  @mui/styled-engine:  6.1.6 
  @mui/system:  6.1.6 
  @mui/types:  7.2.19 
  @mui/utils:  6.1.6 
  @pigment-css/nextjs-plugin: ^0.0.26 => 0.0.26 
  @pigment-css/react:  0.0.25 
  @pigment-css/unplugin:  0.0.26 
  @types/react: ^18.3.1 => 18.3.12 
  react: ^18.3.1 => 18.3.1 
  react-dom: ^18.3.1 => 18.3.1 
  typescript: ^5.6.3 => 5.6.3 

Search keywords: Pigment, material-pigment-css, palette mode

@aleksei-berezkin aleksei-berezkin added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Nov 11, 2024
@siriwatknp siriwatknp added package: material-ui Specific to @mui/material support: docs-feedback Feedback from documentation page package: pigment-css Specific to @pigment-css/* v6.x migration and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Nov 12, 2024
@siriwatknp
Copy link
Member

I am putting this issue to high priority.

@siriwatknp siriwatknp moved this to Selected in Material UI Nov 12, 2024
@oliviertassinari
Copy link
Member

oliviertassinari commented Nov 13, 2024

I guess we should transfer this issue to https://github.com/mui/pigment-css? I mean why should it be any different in Material UI compared to in Pigment CSS https://github.com/mui/pigment-css?tab=readme-ov-file#switching-color-schemes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: material-ui Specific to @mui/material package: pigment-css Specific to @pigment-css/* support: docs-feedback Feedback from documentation page v6.x migration
Projects
Status: Selected
Development

No branches or pull requests

4 participants