Hygiene: Remove FIXMEs now enough time has passed

Routine maintenance - all FIXME statements suggest these can
be removed. W00t!

Depends-On: Iee059a6605cf12a073954378f49ddc721a7e016a
Change-Id: I645dca1fa4d9e80d506e907b0be78a5aef0e2f12
This commit is contained in:
jdlrobson 2017-08-28 15:47:05 -05:00 committed by Jdlrobson
parent 2e50c1b9d8
commit 79fd8bd799
5 changed files with 1 additions and 40 deletions

View File

@ -31,11 +31,6 @@ footer {
position: absolute;
right: -1em;
// FIXME: Remove pseudo selector rule in 1 weeks time [T145479]
&:before {
.transform( rotate( -90deg ) );
}
.client-nojs & {
// Certain browsers e.g. Opera Mini do not support css transforms
// We assume that we do not run JS on these browsers so hide the chevron
@ -70,11 +65,6 @@ footer {
}
}
// FIXME: Remove pseudo selector rule in 1 weeks time [T145479]
.rtl footer .indicator:before {
.transform( rotate( 90deg ) );
}
@media ( min-width: @deviceWidthTablet ) {
footer {
.last-modified-bar {

View File

@ -150,21 +150,6 @@ body {
}
}
// Make search input more visible for users on small screens.
// Opera Mini doesn't support placeholders.
// FIXME: As soon as HeaderV2 feature flag is removed this entire block can disappear as a search icon
// will now be shown on Opera Mini instead.
@media all and ( max-width: @wgMFDeviceWidthMobileSmall ) {
.header {
.search {
border: 1px solid @grayLight;
// Remove the space for the search icon inside the search bar and increase height to size of button
padding: 0.5em 0.1em;
background: none; // so that the icon doesn't overlap with the placeholder
}
}
}
#searchInput {
cursor: text;
}
@ -177,8 +162,6 @@ body {
}
.search-box {
// FIXME: remove when micro.tap in stable and rule from common-js.less too
-webkit-tap-highlight-color: rgba( 255, 255, 255, 0 );
.search {
@searchIconSize: 20px;

View File

@ -1,10 +1,3 @@
@import 'mediawiki.mixins.less';
@import 'minerva.variables.less';
@import 'minerva.mixins.less';
// FIXME: Delete this in 7 days time (T42062)
@import '../../skinStyles/mediawiki.hlist/minerva.less';
.content {
ul {
list-style: square inside;

View File

@ -15,9 +15,7 @@
$toc = $( '#toc' ),
enableToc = mw.config.get( 'wgMinervaTocEnabled' );
if ( enableToc ||
// Fallback for old cached HTML, added 26 June, 2014
( enableToc === null && sections.length > 0 && !page.isMainPage() ) ) {
if ( enableToc ) {
toc = new TableOfContents( {
sections: sections
} );

View File

@ -1,7 +1,4 @@
// Additional Styles for no-js editing mode
// FIXME: `mobile.` LESS modules should not import `minerva.` LESS modules.
// Move Minerva specific rules into skinStyles.
@import 'minerva.mixins';
.editHelp, .cancelLink, .mw-editButtons-pipe-separator {