eliminate horizontal scroll caused by footer#9669
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9669 +/- ##
=========================================
Coverage ? 42.25%
=========================================
Files ? 587
Lines ? 77987
Branches ? 0
=========================================
Hits ? 32956
Misses ? 40991
Partials ? 4040Continue to review full report at Codecov.
|
| .container { | ||
| width: 100vw !important; | ||
| padding: 0 0.5rem; | ||
| max-width: calc(100vw - 1rem) !important; |
There was a problem hiding this comment.
Back to !important are we, isn't it? 🤣
There was a problem hiding this comment.
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)