-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
eliminate horizontal scroll caused by footer #9669
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9669 +/- ##
=========================================
Coverage ? 42.25%
=========================================
Files ? 587
Lines ? 77987
Branches ? 0
=========================================
Hits ? 32956
Misses ? 40991
Partials ? 4040 Continue to review full report at Codecov.
|
@@ -774,6 +774,7 @@ footer { | |||
.container { | |||
width: 100vw !important; | |||
padding: 0 0.5rem; | |||
max-width: calc(100vw - 1rem) !important; |
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.
Back to !important
are we, isn't it? 🤣
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.
Yeah, it's needed because it is overriding this style from fomantic:
.ui.container {
max-width: 100%;
}
We really do need to move to less completely... |
Please send backport to v1.11 |
Not sure I follow. Personally, I'd rather move to Sass 😉 |
I meant build our CSS directly from fomantic less by integrating with its theming system etc. so we can drop those !important |
Ah, yes. I don't think we actually need to integrate that tightly. Theoretically it's enough if our styles are loaded after the fomantic styles for them to win. Still, I prefer to "reinforce" statements via |
fixes an issue with horizontal scroll on all pages on mobile, caused by the page footer being too wide for some reason.
Before (notice horizontal scrollbar)
After (no more horizontal scrollbar)