Clean up cached HTML code

Follow-up: I59b3acd3d56cd5761e5978607634dfb9a88f60e3
Bug: T317900
Change-Id: Idfbd77c6b84c0c8541a81effc272f4f36f17b823
This commit is contained in:
bwang 2022-12-14 16:26:48 -06:00 committed by Jdrewniak
parent 6b784bf5af
commit a50654309d
2 changed files with 27 additions and 43 deletions

View file

@ -7,10 +7,8 @@
* - language variants, Actions menus
* - more menu, user menu
* - ULS button in sticky header
* FIXME: Drop .vector-menu-dropdown selector when cache clears.
*/
.vector-dropdown > .vector-menu-heading,
.vector-menu-dropdown > .vector-menu-heading {
.vector-dropdown > .vector-menu-heading {
display: flex;
&:after {
@ -20,10 +18,8 @@
/**
* Dropdown container
* FIXME: Drop .vector-menu-dropdown selector when cache clears.
*/
.vector-dropdown,
.vector-menu-dropdown {
.vector-dropdown {
> .vector-menu-content {
background-color: @background-color-base;
border: @border-width-base @border-style-base @border-color-base;
@ -35,11 +31,8 @@
/**
* Dropdown menu items.
* FIXME: Drop .vector-menu-dropdown selector when cache clears.
* TODO: Can any of these unique classes be converted to use .mw-list-item instead?
*/
.vector-dropdown .mw-list-item,
.vector-menu-dropdown > .vector-menu-content .mw-list-item {
.vector-dropdown .mw-list-item {
a {
.mixin-vector-dropdown-menu-item();
white-space: nowrap;

View file

@ -14,24 +14,17 @@
.vector-feature-page-tools-disabled {
@media ( max-width: @max-width-tablet ) {
// FIXME: Remove .vector-main-menu-contents from this selector after I59b3acd3d56cd5761e5978607634dfb9a88f60e3 has been in prod for 5 days
.vector-main-menu-contents,
.vector-main-menu {
// Makes the sidebar full screen at lower resolutions.
width: 100%;
}
}
// Hide sidebar entirely when the checkbox is disabled
// FIXME: Remove .vector-main-menu-contents from this selector after I59b3acd3d56cd5761e5978607634dfb9a88f60e3 has been in prod for 5 days
@{selector-main-menu-closed} ~ .vector-main-menu-container .vector-main-menu-contents,
@{selector-main-menu-closed} ~ .vector-main-menu-container .vector-main-menu {
display: none;
}
}
// FIXME: Remove .vector-main-menu-contents from this selector after I59b3acd3d56cd5761e5978607634dfb9a88f60e3 has been in prod for 5 days
.vector-main-menu-contents,
.vector-main-menu {
box-sizing: border-box;
@ -39,36 +32,34 @@
#p-navigation .vector-menu-heading {
display: none;
}
}
// Main menu when pinned in the sidebar
// FIXME: Remove .vector-main-menu-contents from this selector after I59b3acd3d56cd5761e5978607634dfb9a88f60e3 has been in prod for 5 days
.vector-main-menu-contents,
.vector-feature-page-tools-disabled .vector-main-menu,
.vector-main-menu-pinned-container .vector-main-menu {
// Temporary magic number, will be calculated after TOC specs are finalized
padding: 12px 19px 12px 9px;
background-color: @background-color-secondary--modern;
// Main menu when pinned in the sidebar
.vector-feature-page-tools-disabled &,
.vector-main-menu-pinned-container & {
// Temporary magic number, will be calculated after TOC specs are finalized
padding: 12px 19px 12px 9px;
background-color: @background-color-secondary--modern;
@media ( min-width: @min-width-desktop ) {
// Magic number from the edge of the main menu to the start of the main menu text.
margin-left: -28px;
}
}
// Main menu when unpinned in the dropdown
.vector-main-menu-unpinned-container .vector-main-menu {
width: 250px;
display: flex;
flex-direction: column;
.vector-menu-heading,
.vector-main-menu-group {
margin: 0;
@media ( min-width: @min-width-desktop ) {
// Magic number from the edge of the main menu to the start of the main menu text.
margin-left: -28px;
}
}
.vector-menu-content {
margin-left: 0;
// Main menu when unpinned in the dropdown
.vector-main-menu-unpinned-container & {
width: 250px;
display: flex;
flex-direction: column;
.vector-menu-heading,
.vector-main-menu-group {
margin: 0;
}
.vector-menu-content {
margin-left: 0;
}
}
}