Revise margin collapsing rules with new layout

Follow up to 97783e5

Bug: T311283
Change-Id: Ia0581713fd182b9236e398a2453efd06ce8b9f1f
This commit is contained in:
Jon Robson 2022-06-23 16:32:00 -07:00 committed by Jdlrobson
parent 6e96a8e936
commit ec6b6ebd9a
1 changed files with 5 additions and 8 deletions

View File

@ -95,7 +95,12 @@ body {
}
.mw-body-subheader {
// avoid margin collapsing by setting a height and margin-top as this will sometimes
// be empty if no indicators.
// The total of margin-top and height should be 16px. Since we're using relative units 1em=14px
// We need to set both height and margin so that the margin is not ignored.
margin-top: 1em;
height: 2px;
}
.mw-header {
@ -130,14 +135,6 @@ body {
z-index: @z-index-base;
}
// Prevents margin collapsing on first element (typically a <p>) of content.
// Custom because .mixin-clearfix() automatically appends :after to the selector
// and doesn't prevent margin collapsing.
.vector-body:before {
content: '';
display: table;
}
/* Navigation Containers */
.vector-layout-legacy .mw-article-toolbar-container {
// Clear the floats on #left-navigation and #right-navigation.