From 5bffb899a823e05cf446c549ca7887f45db9c154 Mon Sep 17 00:00:00 2001
From: Volker E <volker.e@wikimedia.org>
Date: Thu, 27 Aug 2020 01:23:33 -0700
Subject: [PATCH] [styles] Remove Internet Explorer 8 specific hacks and
 workarounds

Bug: T261378
Change-Id: If51f20137a001276800fafb97ed4ceb53617ae79
---
 resources/skins.vector.styles/SearchBox.less  |  2 --
 .../TabWatchstarLink-ie8.less                 | 32 -------------------
 .../skins.vector.styles/skin-legacy.less      |  4 ---
 resources/skins.vector.styles/skin.less       |  4 ---
 4 files changed, 42 deletions(-)
 delete mode 100644 resources/skins.vector.styles/TabWatchstarLink-ie8.less

diff --git a/resources/skins.vector.styles/SearchBox.less b/resources/skins.vector.styles/SearchBox.less
index 573505798..21147e20d 100644
--- a/resources/skins.vector.styles/SearchBox.less
+++ b/resources/skins.vector.styles/SearchBox.less
@@ -23,8 +23,6 @@
 
 // The search input.
 #searchInput {
-	// Support IE6-8: Fallback for browsers, which don't support `rgba()`.
-	background-color: @background-color-base;
 	background-color: rgba( 255, 255, 255, 0.5 );
 	color: @color-base--emphasized;
 	width: 100%;
diff --git a/resources/skins.vector.styles/TabWatchstarLink-ie8.less b/resources/skins.vector.styles/TabWatchstarLink-ie8.less
deleted file mode 100644
index a2ea27139..000000000
--- a/resources/skins.vector.styles/TabWatchstarLink-ie8.less
+++ /dev/null
@@ -1,32 +0,0 @@
-/*!
- * Convert watchstar to text label in IE 8.
- *
- * Hide the SVG icon and show the text label instead.
- * This is in a separate file because it uses a
- * @media query, which can't be nested, so we need to include
- * this outside the overall '@media screen'.
- */
-
-// stylelint-disable no-duplicate-selectors
-
-// Support IE 8: Show label instead (Grade C).
-// See https://stackoverflow.com/a/17699986/1696030
-// https://keithclark.co.uk/articles/moving-ie-specific-css-into-media-blocks/media-tests/
-@ie8-media-start: ~'@media \0screen {';
-@ie8-media-end: ~'} after';
-@{ie8-media-start}
-.vector-menu-tabs .mw-watchlink.icon a {
-	width: auto;
-	height: auto;
-	// Copied from 'MenuTabs.less'.
-	padding: 1.25em 8px 0 8px;
-}
-
-// LESS nesting doesn't work in this hack.
-.vector-menu-tabs .mw-watchlink.icon a:before {
-	display: none;
-}
-// Bogus extra rule for LESS compiler to render `@media` ending bracket.
-@{ie8-media-end} {
-	/* This comment makes the block non-empty. */
-}
diff --git a/resources/skins.vector.styles/skin-legacy.less b/resources/skins.vector.styles/skin-legacy.less
index 7fecf946b..efd9bc644 100644
--- a/resources/skins.vector.styles/skin-legacy.less
+++ b/resources/skins.vector.styles/skin-legacy.less
@@ -22,10 +22,6 @@
 	@import 'Footer.less';
 }
 
-// Support: IE8
-// Media query hack required.
-@import 'TabWatchstarLink-ie8.less';
-
 @media print {
 	@import 'common/print.less';
 }
diff --git a/resources/skins.vector.styles/skin.less b/resources/skins.vector.styles/skin.less
index 2eec30322..105a870f7 100644
--- a/resources/skins.vector.styles/skin.less
+++ b/resources/skins.vector.styles/skin.less
@@ -27,10 +27,6 @@
 	@import 'Logo.less';
 }
 
-// Support: IE8
-// Media query hack required.
-@import 'TabWatchstarLink-ie8.less';
-
 @media print {
 	@import 'common/print.less';
 	@import 'layout-print.less';