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
[docs] Add CSS variables documentation for Material UI #33958
Conversation
…/css-variables
…/css-variables
I think "CSS theme variables" (lowercase t) is a good term for this. 👍 |
Great job @siriwatknp! I'll follow up with suggestions for style and grammar but overall the content and structure look great. |
docs/data/material/experimental-api/css-theme-variables/customization.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/customization.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/customization.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/overview.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/overview.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/usage.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/usage.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/usage.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/usage.md
Outdated
Show resolved
Hide resolved
docs/data/material/experimental-api/css-theme-variables/usage.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Sam Sycamore <71297412+samuelsycamore@users.noreply.github.com> Signed-off-by: Siriwat K <siriwatkunaporn@gmail.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.
I think this is ready to go aside from the last couple tiny things I commented on here!
close #33417
Preview: https://deploy-preview-33958--material-ui.netlify.app/material-ui/experimental-api/css-theme-variables/overview/
The page structure follows the RFC
I think that the word
CSS variables
is too broad which refers to CSS rather than the feature that we implement.What Material UI offers is related to the CSS variables generated from the theme, so I think using
Theme CSS variables
sounds better. (another option could beCSS Theme variables
) cc @samuelsycamoreWhat's next
There are other advanced topics that I haven't covered but I think it is better to move them into a separate PR.
:where
selector