jonny
/
Vector-Local
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Restore sticky header class and update TOC styles 

Bug: T311170
Change-Id: Ib12b6f79a20b2a25b49110e5a7de59d199917dc9
This commit is contained in:
Clare Ming 2022-06-22 12:37:47 -06:00 committed by bwang
parent e1942cec4e
commit 75cc9bdcd9
2 changed files with 13 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
resources/skins.vector.styles/components/StickyHeader.less
View File

@ -136,12 +136,18 @@
// sticky header isn't visible yet but we still need scroll padding applied
// (e.g. when the user navigates to a page with a hash fragment in the URI).
scroll-padding-top: @height-sticky-header;
// T289817 Override other sticky element offsets to ensure that other
// sticky elements (i.e. table headers) appear below the sticky header.
.charts-stickyhead th {
/* stylelint-disable-next-line declaration-no-important */
top: @height-sticky-header !important;
}
}
}
.client-js .vector-sticky-header-visible {
// T289817 Override other sticky element offsets to ensure that other
// sticky elements (i.e. table headers) appear below the sticky header.
//
// @stable See the Integration notes for developers section at
// https://www.mediawiki.org/wiki/Reading/Web/Desktop_Improvements/Features/Sticky_Header
.mw-sticky-header-element,
.charts-stickyhead th {
/* stylelint-disable-next-line declaration-no-important */
top: @height-sticky-header !important;
}
}

8
resources/skins.vector.styles/layouts/screen.less
View File

@ -210,14 +210,6 @@ body {
}
}
@media ( min-width: @width-breakpoint-tablet ) {
// T289817 Override other sticky element offsets to ensure that other
// sticky elements (i.e. table headers) appear below the sticky header.
.mw-table-of-contents-container {
top: @height-sticky-header;
}
}
.parsoid-body {
padding: @padding-content;
}