From 577b72a14df1e4f7f4fdd41b5dfa7090b28f2cad Mon Sep 17 00:00:00 2001
From: jdlrobson <jdlrobson@gmail.com>
Date: Thu, 7 Feb 2019 12:29:00 -0800
Subject: [PATCH] Remove backwards compatible CSS

This is no longer needed when Varnish cache has cleared (About a
week from I7e989a3d4553eb3357598a5cad3ccebf51dc9fae being deployed

Bug: T212216
Change-Id: I1afb68f68cc90d89c442907426bd046ddc68f488
---
 includes/skins/minerva.mustache               |  2 +-
 minerva.less/minerva.variables.less           |  2 --
 .../pageactions.less                          | 21 +------------------
 3 files changed, 2 insertions(+), 23 deletions(-)

diff --git a/includes/skins/minerva.mustache b/includes/skins/minerva.mustache
index 4c1b288..8d56c9f 100644
--- a/includes/skins/minerva.mustache
+++ b/includes/skins/minerva.mustache
@@ -28,7 +28,7 @@
 		<main id="content" class="mw-body">
 			{{#hasheadingholder}}
 			{{{prebodyhtml}}}
-			<div class="pre-content heading-holder heading-holder--new">
+			<div class="pre-content heading-holder">
 				{{{headinghtml}}}
 				{{{taglinehtml}}}
 				{{#tabs}}
diff --git a/minerva.less/minerva.variables.less b/minerva.less/minerva.variables.less
index eb08eb9..7526935 100644
--- a/minerva.less/minerva.variables.less
+++ b/minerva.less/minerva.variables.less
@@ -46,8 +46,6 @@
 @titleSectionSpacingBottom: 25px;
 
 // Page actions
-@pageActionsGutter: 0.5em;
-@pageActionsHeight: @pageActionFontSize + (2 * @iconGutterWidth);
 @tabBorderSize: ( 1em / 16px ) * 2;
 @taglineFontSize: 0.85em;
 
diff --git a/resources/skins.minerva.base.styles/pageactions.less b/resources/skins.minerva.base.styles/pageactions.less
index baf1c3d..b1789dc 100644
--- a/resources/skins.minerva.base.styles/pageactions.less
+++ b/resources/skins.minerva.base.styles/pageactions.less
@@ -2,28 +2,9 @@
 @import '../../minerva.less/minerva.mixins';
 
 .heading-holder {
-	@pageActionsGutter: 0.5em;
-	@pageActionsHeight: @pageActionFontSize + (2 * @iconGutterWidth);
-	// FIXME: padding-bottom can be removed when work on T212216
-	// has been deployed and varnish cache cleared.
-	padding: @titleSectionSpacingTop 0 (@pageActionsHeight + @pageActionsGutter);
+	padding: @titleSectionSpacingTop 0 0;
 	position: relative;
 
-	// FIXME: Can be made default when work on T212216
-	// has been deployed and varnish cache cleared.
-	&--new {
-		padding-bottom: 0;
-	}
-	// FIXME: Can be removed when work on T212216
-	// has been deployed and varnish cache cleared.
-	&--no-page-actions {
-		padding-bottom: @pageActionsGutter;
-
-		&--new {
-			padding-bottom: 0;
-		}
-	}
-
 	.tagline {
 		color: @colorGray5;
 		font-size: @taglineFontSize;