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

Set a max width to EuiPageBody #3375

Merged
merged 2 commits into from
Apr 22, 2020
Merged

Set a max width to EuiPageBody #3375

merged 2 commits into from
Apr 22, 2020

Conversation

snide
Copy link
Contributor

@snide snide commented Apr 22, 2020

Summary

Fixes #3373

See #3373 (comment) for more discussion

By setting a max width of EuiPageBody we can likely force inner items to not break down in various smaller screen configs.

image

Checklist

  • Check against all themes for compatibility in both light and dark modes
  • Checked in mobile
  • Checked in IE11 and Firefox
  • Props have proper autodocs
  • [ ] Added documentation 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

@kibanamachine
Copy link

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

Copy link
Contributor

@cchaos cchaos left a comment

Choose a reason for hiding this comment

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

🎉 And it almost fixes the Flex page too, but alas...
Screen Shot 2020-04-22 at 16 38 25 PM

Approved, though I didn't check any other browser but Chrome

@snide
Copy link
Contributor Author

snide commented Apr 22, 2020

@cchaos It actually works there too. It's that we have content within one of the flexs itself that break the width. Since the example is meant to show a different problem, I think it's OK to leave that page as it is.

image

@kibanamachine
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EuiTabs can break layouts
3 participants