-
Notifications
You must be signed in to change notification settings - Fork 842
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
Update to reflect guidelines #8297
Conversation
@ryankeairns We should also sync the updates for the new EUI+ docs 🙂 |
@ryankeairns Thanks for taking the initiative here @ryankeairns! |
Thanks for the feedback. Added a screenshot of the changes in the new site. |
Preview staging links for this PR:
|
💚 Build Succeeded
History
|
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.
Thanks for ensuring the docs are more clear! 🎉
Co-authored-by: Lene Gadewoll <[email protected]>
Closes #8295
Summary
Update docs example to follow more closely our guidelines while keeping the component as-is for non-Elastic use cases.
Change included
subdued
andwarning
subdued
andwarning
Unrelated tweaks
Preview
Current site

New site

QA
Remove or strikethrough items that do not apply to your PR.
General checklist
@default
if default values are missing) and playground toggles