From da2aafa5cfa4d6e331b299fe866766f6021392fa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Mon, 26 Feb 2018 13:02:07 +0100
Subject: [PATCH] Fixed icons padding
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 apps/files_external/css/settings.scss      | 27 ++++++++++++++--------
 apps/files_external/templates/settings.php | 13 +++--------
 2 files changed, 21 insertions(+), 19 deletions(-)

diff --git a/apps/files_external/css/settings.scss b/apps/files_external/css/settings.scss
index c781cd9bd17..4f46ccb97e2 100644
--- a/apps/files_external/css/settings.scss
+++ b/apps/files_external/css/settings.scss
@@ -32,15 +32,24 @@ td.mountPoint, td.backend { width:160px; }
 #addMountPoint>td.applicable { visibility:hidden; }
 #addMountPoint>td.hidden { visibility:hidden; }
 
-#externalStorage td.mountOptionsToggle {
-	position: relative;
-	padding: 0 !important;
-	width: 44px;
-}
-#externalStorage .icon-settings {
-	padding: 14px;
-	vertical-align: text-bottom;
-	cursor: pointer;
+#externalStorage td {
+	&.mountOptionsToggle,
+	&.remove,
+	&.save {
+		position: relative;
+		padding: 0 !important;
+		width: 44px;
+		opacity: 0.5;
+		&:hover {
+			opacity: 1;
+		}
+		[class^='icon-'],
+		[class*=' icon-'] {
+			padding: 14px;
+			vertical-align: text-bottom;
+			cursor: pointer;
+		}
+	}
 }
 
 #selectBackend {
diff --git a/apps/files_external/templates/settings.php b/apps/files_external/templates/settings.php
index 1fb3245c6d6..b6bce5b30ba 100644
--- a/apps/files_external/templates/settings.php
+++ b/apps/files_external/templates/settings.php
@@ -143,21 +143,14 @@
 					</td>
 				<?php endif; ?>
 				<td class="mountOptionsToggle hidden">
-					<div class="icon-settings" title="<?php p($l->t('Advanced settings')); ?>"></div>
+					<div class="icon-settings-dark" title="<?php p($l->t('Advanced settings')); ?>"></div>
 					<input type="hidden" class="mountOptions" value="" />
 				</td>
 				<td class="remove hidden">
-					<img class="svg"
-						alt="<?php p($l->t('Delete')); ?>"
-						title="<?php p($l->t('Delete')); ?>"
-						src="<?php print_unescaped(image_path('core', 'actions/delete.svg')); ?>"
-					/>
+					<div class="icon-delete" title="<?php p($l->t('Delete')); ?>"></div>
 				</td>
 				<td class="save hidden">
-					<img alt="<?php p($l->t('Save')); ?>"
-						 title="<?php p($l->t('Save')); ?>"
-						 src="<?php print_unescaped(image_path('core', 'actions/checkmark.svg')); ?>"
-					/>
+					<div class="icon-checkmark" title="<?php p($l->t('Save')); ?>"></div>
 				</td>
 			</tr>
 		</tbody>