Skip to content

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

Merged
cee-chen merged 2 commits intoelastic:theme-css-variablesfrom
cee-chen:css-vars-wiki-doc
Sep 1, 2023
Merged

[wiki] Add CSS variables docs to our Emotion wiki#7145
cee-chen merged 2 commits intoelastic:theme-css-variablesfrom
cee-chen:css-vars-wiki-doc

Conversation

@cee-chen
Copy link
Copy Markdown
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 Use on PRs to skip changelog requirement (Don't delete - used for automation) emotion labels Aug 30, 2023
@cee-chen cee-chen requested a review from a team August 30, 2023 17:56
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

Fantastic suggestions, thank you Trevor!

Copy link
Copy Markdown
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 Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants