-
Notifications
You must be signed in to change notification settings - Fork 839
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
[WIP] Emotion & Propagate #3760
Conversation
Preview documentation changes for this PR: https://eui.elastic.co/pr_3760/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_3760/ |
Test failures due to missing theme context in the jest environment. Not detrimental at the moment. |
Preview documentation changes for this PR: https://eui.elastic.co/pr_3760/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_3760/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_3760/ |
…rings similar to the padding CSS rule
Preview documentation changes for this PR: https://eui.elastic.co/pr_3760/ |
…s using EuiCallOut
…s using EuiCallOut
My official/final 😂 review and exploration of this POC and possible next steps are all documented here: https://app.simplenote.com/publish/D9dZkc The important bits are the top (explaining what we aim to gain from CSS in JS) and the bottom (my proposal for moving forward) sections. |
I'm going to close this in favor of making smaller, more targeted PRs, perhaps even cherry-picking commits from this branch. We can decide when the first context PR is ready if we want to make use of a merge-base feature branch. Follow #3912 for updates |
Makes sense to me! |
Summary
A very much draft version of integrating
emotion-js
into EUI by way of a custom cascading property library ("Propagate", for now) built for theming.Contents:
gatsby-eui-starter
testing ground: https://github.com/thompsongl/eui-emotion-gatsbyyarn build && npm pack
, then use the.tgz
as the EUI target in this repo.Checklist