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

Firefox/IE doesn't support word-break: break-word; #855

Closed
jen-huang opened this issue May 18, 2018 · 4 comments · Fixed by #962
Closed

Firefox/IE doesn't support word-break: break-word; #855

jen-huang opened this issue May 18, 2018 · 4 comments · Fixed by #962
Assignees
Labels

Comments

@jen-huang
Copy link
Contributor

Relates to elastic/kibana#19215

Firefox and IE (inc. Edge) doesn't support word-break: break-word;:
https://developer.mozilla.org/en-US/docs/Web/CSS/word-break#Browser_compatibility

This property is being used a couple of times for tables to prevent content from overflowing. Works in Chrome:

screen shot 2018-05-18 at 10 59 52 am

Doesn't work in Firefox:

screen shot 2018-05-18 at 10 59 37 am

Or IE:

field

Consider using overflow-wrap instead (called word-wrap in IE): https://developer.mozilla.org/en-US/docs/Web/CSS/overflow-wrap

@peteharverson
Copy link

Also ran into this issue with the ML anomalies table, where fields like job IDs cannot contain spaces, so long values will overflow the cells in Firefox and Edge.

Screenshot from Edge:
image

@snide snide assigned snide and unassigned snide May 22, 2018
@snide
Copy link
Contributor

snide commented May 22, 2018

@cchaos do you have time for this one?

@cchaos
Copy link
Contributor

cchaos commented May 22, 2018

Yeah, I can take it.

@cchaos cchaos self-assigned this May 22, 2018
cchaos pushed a commit to cchaos/eui that referenced this issue May 22, 2018
Add fallback to just break-all in FF and IE
@jen-huang
Copy link
Contributor Author

jen-huang commented Jun 29, 2018

@cchaos Reopening - fallback styles in #864 doesn't seem to have been applied to the word-break property on table cells (screenshot on IE11):

screen shot 2018-06-29 at 2 57 21 pm

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

Successfully merging a pull request may close this issue.

4 participants