Apply margin-top to indicators

Bug: T335627
Change-Id: Ic384f9d532ed0a1165c29869e53d99cd0c7dc495
This commit is contained in:
Brandon Fowler 2023-04-29 14:22:30 -04:00 committed by Jdrewniak
parent bed8e2523f
commit 3433525941
2 changed files with 11 additions and 2 deletions

View file

@ -136,6 +136,9 @@
// Language button
@height-button-lang: unit( 32 / @font-size-browser, em );
// Pre-content
@margin-top-pre-content: 8px;
// Header
// T245190 T296321 For sticky header, set an explicit height. This is needed for scroll padding
// and for other sticky elements on the page. Setting the height in relative units enables

View file

@ -70,6 +70,12 @@ body {
// Contain the floating .mw-indicators, but don't use clearfix because of browser inconsistencies
// when combining 'clear' and 'margin-top' (T325391)
overflow: hidden;
// Margin is only applied to indicators inside `.vector-body-before-content` since they can
// also appear next to the page title, where margin is not needed.
.mw-indicators {
margin-top: @margin-top-pre-content;
}
}
.mw-body .mw-portlet-lang {
@ -82,7 +88,7 @@ body {
}
#siteSub {
margin-top: 8px; // T311564
margin-top: @margin-top-pre-content; // T311564
}
// Styles only applied to non-empty #contentSub to avoid extra margins when both #contentSub and #contentSub2
@ -94,7 +100,7 @@ body {
color: @color-subtle;
width: auto;
// Visually separate #contentSub and #contentSub2 (T315639)
margin: 8px 0 0;
margin: @margin-top-pre-content 0 0;
}
.parsoid-body {