From c6960ce8a77ffbd0a23a1a2b7b5b20f1546dd34d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bartosz=20Dziewo=C5=84ski?= <matma.rex@gmail.com>
Date: Mon, 17 Sep 2018 23:29:52 +0200
Subject: [PATCH] Remove 'mobile.editor.ve' styles (move to MobileFrontend)

They no longer depend on Minerva's variables and can be moved there.
See I964b23bf9363233e5fc927cfaf1e8bdbc3435de1.

Bug: T202978
Change-Id: I6ce2156c29dcf8a95015ffdf59d02ecc5a03b74f
---
 skin.json                                |  3 --
 skinStyles/mobile.editor.ve/minerva.less | 50 ------------------------
 2 files changed, 53 deletions(-)
 delete mode 100644 skinStyles/mobile.editor.ve/minerva.less

diff --git a/skin.json b/skin.json
index 5c4c41e58..c92d8251c 100644
--- a/skin.json
+++ b/skin.json
@@ -146,9 +146,6 @@
 			"mobile.special.mobilemenu.styles": [
 				"skinStyles/mobile.special.mobilemenu.styles/minerva.less"
 			],
-			"mobile.editor.ve": [
-				"skinStyles/mobile.editor.ve/minerva.less"
-			],
 			"mobile.notifications.overlay": [
 				"skinStyles/mobile.notifications.overlay/minerva.less"
 			]
diff --git a/skinStyles/mobile.editor.ve/minerva.less b/skinStyles/mobile.editor.ve/minerva.less
deleted file mode 100644
index 58bad798b..000000000
--- a/skinStyles/mobile.editor.ve/minerva.less
+++ /dev/null
@@ -1,50 +0,0 @@
-@import 'mediawiki.mixins.less';
-@import '../../minerva.less/minerva.variables';
-
-.editor-overlay-ve {
-	.content {
-		// We don't need display:table as there are no edit/collapse buttons
-		// Also display:table causes CE issues in Mobile Safari (T202991)
-		h1, h2, h3, h4, h5 {
-			display: block;
-		}
-	}
-
-	.overlay-content {
-		.surface {
-			overflow: hidden;
-		}
-	}
-
-	// Toolbar
-	@targetIconSize: 20px;
-	.overlay-header-container {
-		box-shadow: 0 1px 1px rgba( 0, 0, 0, 0.1 );
-
-		.ve-init-mw-mobileArticleTarget-toolbar {
-			// The border will only be visible on screens >1000px due to max-width set elsewhere.
-			// On smallers screens, the negative margin will cause it to be outside of viewport.
-			border-color: @colorGray12;
-			border-style: solid;
-			border-width: 0 1px;
-			margin: 0 -1px;
-
-			.oo-ui-iconElement-icon {
-				/* Overwrite `contain` value here, as we're 16px base sized */
-				.background-size( @targetIconSize, @targetIconSize );
-			}
-
-			.oo-ui-toolbar-bar {
-				border: 0;
-				box-shadow: none;
-			}
-		}
-	}
-
-	.overlay-header {
-		/* Reset unwanted header styles */
-		border-collapse: separate;
-		border: 0;
-		height: 3em;
-	}
-}