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

[wiki] Add CSS variables docs to our Emotion wiki #7145

Merged
merged 2 commits into from
Sep 1, 2023

Conversation

cee-chen
Copy link
Contributor

Summary

Adds an example of how components can set CSS variables. Ideally should also clearly explain when to use CSS variables, but I'm not sure this is totally clear and my brain is a little fried right now, so would appreciate feedback.

BTW, I definitely anticipate us tweaking these docs more as we move on to converting form components to and actually have to set up CSS variables for them.

QA

General checklist

N/A, wiki/docs only

- ideally should also clearly explain when to use CSS variables, but I'm not sure this is totally clear
@cee-chen cee-chen added documentation Issues or PRs that only affect documentation - will not need changelog entries skip-changelog emotion labels Aug 30, 2023
@cee-chen cee-chen requested a review from a team August 30, 2023 17:56
Copy link
Contributor

@1Copenut 1Copenut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a handful of copy suggestions @cee-chen. LMK when you've had a chance to pick or pass, and I'll update my review to approved.

Co-authored-by: Trevor Pierce <1Copenut@users.noreply.github.com>
@cee-chen
Copy link
Contributor Author

Fantastic suggestions, thank you Trevor!

Copy link
Contributor

@1Copenut 1Copenut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢 it!

@cee-chen cee-chen merged commit 2bd110f into elastic:theme-css-variables Sep 1, 2023
@cee-chen cee-chen deleted the css-vars-wiki-doc branch September 1, 2023 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues or PRs that only affect documentation - will not need changelog entries emotion skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants