Remove unused css rules

* language-selector can only appear in secondary actions so only
load rule on Main page
* .no-js-only not used in MobileFrontend or Minerva
* Remove last-modified-bar rules from ui.less that are repeated
inside footer less file
* #mw-mf-last-modified id is no longer used post-footer rewrite
* CSS rules for .mw-mf-banner are defined in Zero extension
* mw-mf-image-replacement is no longer needed after we turned off
the option to disable images in the mobile site

Change-Id: I7abdbac4d2d16b931e2b110fcebf0dfc82146753
This commit is contained in:
jdlrobson 2017-12-12 16:48:04 -08:00
parent 1648c7752e
commit f9ac17d94d
6 changed files with 4 additions and 60 deletions

View File

@ -33,15 +33,6 @@
.box-sizing( border-box ); .box-sizing( border-box );
} }
.mw-mf-image-replacement {
font-style: italic;
color: @colorGray7;
text-decoration: underline;
border: dotted 1px @colorGray12;
padding: 10px;
display: inline-block;
}
textarea { textarea {
padding: 5px 0 5px 5px; padding: 5px 0 5px 5px;
width: 100%; width: 100%;
@ -57,7 +48,6 @@ textarea {
.toc, .toc,
// We also need a more specific rule for tablet non-JS users who will load skins.minerva.tablet.styles // We also need a more specific rule for tablet non-JS users who will load skins.minerva.tablet.styles
.client-nojs .toc-mobile, .client-nojs .toc-mobile,
.client-js .no-js-only,
.client-js .mw-redirectedfrom, .client-js .mw-redirectedfrom,
// FIXME: Use generic rule for print stylesheets // FIXME: Use generic rule for print stylesheets
.printfooter, .printfooter,
@ -67,7 +57,6 @@ textarea {
/* TODO: Fix specificity */ /* TODO: Fix specificity */
/* stylelint-disable no-descending-specificity */ /* stylelint-disable no-descending-specificity */
.no-js-only,
.client-js .jsonly { .client-js .jsonly {
display: inherit; display: inherit;
} }

View File

@ -41,11 +41,6 @@ body {
} }
.last-modified-bar { .last-modified-bar {
background-color: transparent;
display: block;
color: @colorGray5;
transition: background-color 0.2s ease, color 0.2s ease;
a, a,
a:visited { a:visited {
color: @colorGray5; color: @colorGray5;
@ -54,24 +49,6 @@ body {
font-weight: bold; font-weight: bold;
} }
} }
#mw-mf-last-modified {
padding-top: 5px;
padding-bottom: 5px;
background-color: @colorGray14;
}
&.active {
#mw-mf-last-modified {
background-color: @lastModifiedBarActiveBackgroundColor;
}
color: #fff;
a {
color: #fff;
}
}
} }
.header { .header {
@ -280,9 +257,6 @@ input.search {
a { a {
margin: 10px 2px 2px 0; margin: 10px 2px 2px 0;
} }
.language-selector {
margin-top: 1em;
}
} }
.truncated-text { .truncated-text {
@ -382,16 +356,9 @@ input.search {
visibility: hidden; visibility: hidden;
} }
.content .nojs-edit {
display: inline-block !important;
visibility: visible;
float: right;
}
// It may be better to express these in a single class // It may be better to express these in a single class
// or think about using extend or a mixin to stay with semantic selectors // or think about using extend or a mixin to stay with semantic selectors
// https://css-tricks.com/the-extend-concept/ // https://css-tricks.com/the-extend-concept/
.last-modified-bar #mw-mf-last-modified,
#bodyContent .panel .content, #bodyContent .panel .content,
.overlay .content-header, .overlay .content-header,
.overlay .panel, .overlay .panel,

View File

@ -49,8 +49,7 @@ a {
} }
// Generic class name needed // Generic class name needed
.return-link, .return-link {
#mw-mf-last-modified {
display: block; display: block;
font-size: 0.9em; font-size: 0.9em;
margin-top: 1.5em; margin-top: 1.5em;

View File

@ -17,9 +17,9 @@
} }
} }
.content { #page-secondary-actions {
#mw-mf-last-modified { .language-selector {
padding: 5px 0; margin-top: 1em;
} }
} }
} }

View File

@ -109,10 +109,6 @@ A file for css that optimises the Minerva skin on larger devices.
} }
} }
#mw-mf-last-modified {
padding: 5px 0;
}
.last-modified-bar { .last-modified-bar {
background-color: transparent; background-color: transparent;
padding-left: 0; padding-left: 0;
@ -122,8 +118,6 @@ A file for css that optimises the Minerva skin on larger devices.
} }
@media screen and ( min-width: @width-breakpoint-desktop ) { @media screen and ( min-width: @width-breakpoint-desktop ) {
// FIXME: Zero should use banner-container class or better - append to banner-container
#mw-mf-page-center .mw-mf-banner,
.banner-container, .banner-container,
// FIXME: Generic header class needed me-thinks! // FIXME: Generic header class needed me-thinks!
.header, .header,

View File

@ -23,9 +23,4 @@ FIXME: Review all of these hacks to see if they still apply.
float: right !important; float: right !important;
clear: right !important; clear: right !important;
} }
// banners inserted by other extensions, i.e. Zero
#mw-mf-page-center .mw-mf-banner {
// !important because Zero has an inline style
background-clip: content-box !important;
}
} }