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

Added euiTextBreakWord() to EuiToast header and fixed .eui-textBreakAll on Firefox #2549

Merged
merged 3 commits into from
Nov 21, 2019

Conversation

cchaos
Copy link
Contributor

@cchaos cchaos commented Nov 20, 2019

Fixes #1021 and elastic/kibana#48984

Since euiTextBreakWord() is an elegant solution for only forcing line breaks within words (if necessary) it provides an easy solution.

Before
Screen Shot 2019-11-20 at 13 30 35 PM

After
Screen Shot 2019-11-20 at 13 30 49 PM

Fixes #2288

Added @legrego 's solutions of overflow-wrap: break-word to .eui-textBreakAll.

Before
Image 2019-11-20 at 1 40 46 PM

After
Image 2019-11-20 at 1 42 11 PM

Checklist

  • [ ] Checked in dark mode
  • [ ] Checked in mobile
  • Checked in IE11 and Firefox
  • [ ] Props have proper autodocs
  • [ ] Added documentation examples I will be reverting the doc change before merge
  • [ ] 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 changed the title Added euiTextBreakWord() to EuiToast header and fix .eui-textBreakAll on Firefox Added euiTextBreakWord() to EuiToast header and fixed .eui-textBreakAll on Firefox Nov 20, 2019
Copy link
Contributor

@andreadelrio andreadelrio left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@elizabetdev elizabetdev left a comment

Choose a reason for hiding this comment

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

Awesome! 🎉

@cchaos cchaos merged commit 5109500 into elastic:master Nov 21, 2019
@cchaos cchaos deleted the toast-title-wrap branch November 21, 2019 14:00
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.

FireFox not respecting .eui-textBreakAll in all cases Toast header overflows the toast element
3 participants