Skip to content

[Docs] Updates Accordion, Bottom bar, Flex, and Flyout pages#4655

Merged
cchaos merged 8 commits intoelastic:masterfrom
cchaos:docs/update/layout_section_part1
Mar 25, 2021
Merged

[Docs] Updates Accordion, Bottom bar, Flex, and Flyout pages#4655
cchaos merged 8 commits intoelastic:masterfrom
cchaos:docs/update/layout_section_part1

Conversation

@cchaos
Copy link
Copy Markdown
Contributor

@cchaos cchaos commented Mar 19, 2021

Starting a series of PR's to fix up docs pages to make better use of the new docs layout and increase clarity

This one tackles Accordion, Bottom bar, Flex, and Flyout pages

Fixes #4139 by adding this section: https://eui.elastic.co/pr_4655/#/layout/flex#flex-items-are-also-flex

Checklist

  • Check against all themes for compatibility in both light and dark modes
  • Checked in mobile
  • Checked in Chrome, Safari, Edge, and Firefox
  • Props have proper autodocs and playground toggles
  • Added documentation
  • Checked Code Sandbox works for the any docs examples
  • [ ] Added or updated jest tests
  • [ ] Checked for breaking changes and labeled appropriately
  • [ ] Checked for accessibility including keyboard-only and screenreader modes
  • [ ] A changelog entry exists and is marked appropriately

@cchaos cchaos 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) labels Mar 19, 2021
@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_4655/

@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_4655/

@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_4655/

@cchaos cchaos requested a review from snide March 23, 2021 16:17
Copy link
Copy Markdown
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

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

Small bits I noticed. Nice work.

Co-authored-by: Dave Snider <dave.snider@gmail.com>
@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_4655/

@cchaos
Copy link
Copy Markdown
Contributor Author

cchaos commented Mar 25, 2021

I need to merge this because I have some updates to do for Flyout. Gonna merge upstream and run CI, then merge.

@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_4655/

@cchaos cchaos merged commit 29a6d24 into elastic:master Mar 25, 2021
@cchaos cchaos deleted the docs/update/layout_section_part1 branch March 25, 2021 20:13
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 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.

[EuiFlexGroup] Responsive flex layouts do not work with buttons properly

3 participants