From 4f2f1bb3499faadbd09677c6290d5b4fbfda0df2 Mon Sep 17 00:00:00 2001
From: jdlrobson <jdlrobson@gmail.com>
Date: Thu, 27 May 2021 14:10:06 -0700
Subject: [PATCH] Drop duplicate styles in Minerva

These styles are also applied inside MobileFrontend as they should apply
to all skins, not just Minerva.

Several styles for desktop Minerva are amended with a comment.

Bug: T253370
Change-Id: Ib23024d192f331ef76007dbd9ef54596cc1adb0c
---
 resources/skins.minerva.base.styles/ui.less | 44 +++++++++------------
 1 file changed, 19 insertions(+), 25 deletions(-)

diff --git a/resources/skins.minerva.base.styles/ui.less b/resources/skins.minerva.base.styles/ui.less
index 640ab8c9c..88c63b20b 100644
--- a/resources/skins.minerva.base.styles/ui.less
+++ b/resources/skins.minerva.base.styles/ui.less
@@ -191,36 +191,30 @@ input.search {
 		margin-bottom: @headingMargin;
 	}
 
-	.mw-parser-output > h1,
-	.mw-parser-output > h2,
-	.mw-parser-output > h3,
-	.mw-parser-output > h4,
-	.mw-parser-output > h5,
-	.section-heading,
-	.in-block {
-		// Safari needs this. @see T106347
-		display: table;
+	// The following rules are duplicated in MobileFrontend
+	// but needed by Minerva desktop
+	.mw-parser-output {
+		> h1,
+		> h2,
+		> h3,
+		> h4,
+		> h5 {
+			// Safari needs this. @see T106347
+			display: table;
 
-		.mw-headline {
-			width: 100%;
-		}
+			.mw-headline {
+				width: 100%;
+			}
 
-		> span {
-			display: table-cell;
-			vertical-align: middle;
-		}
-
-		> .mw-editsection > .mw-ui-icon-element {
-			margin-top: -0.75em;
-			margin-bottom: -0.75em;
+			> span {
+				display: table-cell;
+			}
 		}
 	}
-}
 
-// Toggling indicators are unusable without JavaScript
-.client-nojs {
-	.section-heading .indicator {
-		display: none;
+	.mw-editsection > .mw-ui-icon-element {
+		margin-top: -0.75em;
+		margin-bottom: -0.75em;
 	}
 }