HEEDLS-542 Replace break-word with break-all #462
Conversation
|
I've added the IE only fix using the same pattern we use elsewhere in the css. Since there was only one instance of each of the other css classes with word-break being used, I've instead added the .word-break css class to the html elements and removed it from the specific css classes so that we don't have to define the IE fix multiple times. |
| .nhsuk-summary-list__value { | ||
| @include govuk-media-query($until: tablet) { | ||
| word-break: break-word; | ||
| word-break: break-all; |
There was a problem hiding this comment.
Ah - forgot to change here. Also, does the hack work ok on other main browsers?
There was a problem hiding this comment.
Yes, this adds an extra declaration for IE only, other browsers only see break-word. IE gets both and ignores the one it doesn't support.
There was a problem hiding this comment.
There are too many instances of this to add a css class to each of them so I've defined it using a mixin instead
3 participants
Changed all usages of word-break: break-word to word-break: break-all
IE screens the ticket was raised for:
