-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fix text overflow bug in Firefox #1764
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fix seems to cover browsers well. I suggest linking this to an issue in govuk-frontend
to keep track of it.
db9a775
to
4fa82d4
Compare
630bc53
to
dcb01ed
Compare
After almost completely forgetting about this PR, I've applied a far less complex fix. I've tested it in a ton of browsers and it seems to have solved the problem properly this time, but happy for further discussion. |
dcb01ed
to
91d1f77
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a simple solution and it fixes the issue 💯
The only thing I think I should note is that changing the list items' display
property to inline-block
does slightly affect the overall spacing on the footer.
Screen.Recording.2021-01-14.at.15.05.35.mov
I don't think it's a significant enough issue to hold this back so I'm happy to approve.
What
Fix a layout bug in Firefox that was causing single column list items to column wrap into the next column. Wasn't occurring in any other browsers, because apparently only Firefox doesn't support the
break-inside
property, so this is a bit of a workaround, but including it for other browsers just in case.Tested in Firefox, Chrome, Edge, everything back to IE8.
Why
The layout footer component is now being used live, in Accounts 😱
Visual Changes
Before:
After: