From fada4ee6c90c7dfa0299d8afed579ab156496c9d Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Wed, 11 Dec 2024 16:51:11 +0100
Subject: [PATCH] fix(AdminSettings): allow to retry server checks

- rename icons
- add titles
- replace colors to NC palette

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 .../AdminSettings/RecordingServer.vue         | 31 +++++++++++-----
 .../AdminSettings/SignalingServer.vue         | 37 +++++++++++++------
 2 files changed, 47 insertions(+), 21 deletions(-)

diff --git a/src/components/AdminSettings/RecordingServer.vue b/src/components/AdminSettings/RecordingServer.vue
index e43555075db..44f20b7bd0d 100644
--- a/src/components/AdminSettings/RecordingServer.vue
+++ b/src/components/AdminSettings/RecordingServer.vue
@@ -23,26 +23,38 @@
 
 		<NcButton v-show="!loading"
 			type="tertiary"
+			:title="t('spreed', 'Delete this server')"
 			:aria-label="t('spreed', 'Delete this server')"
 			@click="removeServer">
 			<template #icon>
-				<Delete :size="20" />
+				<IconDelete :size="20" />
 			</template>
 		</NcButton>
 
 		<span v-if="server" class="test-connection">
 			<NcLoadingIcon v-if="!checked" :size="20" />
-			<AlertCircle v-else-if="errorMessage" :size="20" :fill-color="'#E9322D'" />
-			<Check v-else :size="20" :fill-color="'#46BA61'" />
+			<IconAlertCircle v-else-if="errorMessage" :size="20" fill-color="var(--color-error)" />
+			<IconCheck v-else :size="20" fill-color="var(--color-success)" />
 			{{ connectionState }}
 		</span>
+
+		<NcButton v-if="server && checked"
+			type="tertiary"
+			:title="t('spreed', 'Test this server')"
+			:aria-label="t('spreed', 'Test this server')"
+			@click="checkServerVersion">
+			<template #icon>
+				<IconReload :size="20" />
+			</template>
+		</NcButton>
 	</li>
 </template>
 
 <script>
-import AlertCircle from 'vue-material-design-icons/AlertCircle.vue'
-import Check from 'vue-material-design-icons/Check.vue'
-import Delete from 'vue-material-design-icons/Delete.vue'
+import IconAlertCircle from 'vue-material-design-icons/AlertCircle.vue'
+import IconCheck from 'vue-material-design-icons/Check.vue'
+import IconDelete from 'vue-material-design-icons/Delete.vue'
+import IconReload from 'vue-material-design-icons/Reload.vue'
 
 import { t } from '@nextcloud/l10n'
 
@@ -57,9 +69,10 @@ export default {
 	name: 'RecordingServer',
 
 	components: {
-		AlertCircle,
-		Check,
-		Delete,
+		IconAlertCircle,
+		IconCheck,
+		IconDelete,
+		IconReload,
 		NcButton,
 		NcCheckboxRadioSwitch,
 		NcLoadingIcon,
diff --git a/src/components/AdminSettings/SignalingServer.vue b/src/components/AdminSettings/SignalingServer.vue
index 417f49e35ce..0dc8b0832ae 100644
--- a/src/components/AdminSettings/SignalingServer.vue
+++ b/src/components/AdminSettings/SignalingServer.vue
@@ -22,28 +22,40 @@
 
 		<NcButton v-show="!loading"
 			type="tertiary"
+			:title="t('spreed', 'Delete this server')"
 			:aria-label="t('spreed', 'Delete this server')"
 			@click="removeServer">
 			<template #icon>
-				<Delete :size="20" />
+				<IconDelete :size="20" />
 			</template>
 		</NcButton>
 
 		<span v-if="server" class="test-connection">
 			<NcLoadingIcon v-if="!checked" :size="20" />
-			<AlertCircle v-else-if="errorMessage" :size="20" :fill-color="'#D91812'" />
-			<AlertCircleOutline v-else-if="warningMessage" :size="20" :fill-color="'#C28900'" />
-			<Check v-else :size="20" :fill-color="'#2D7B41'" />
+			<IconAlertCircle v-else-if="errorMessage" :size="20" fill-color="var(--color-error)" />
+			<IconAlertCircleOutline v-else-if="warningMessage" :size="20" fill-color="var(--color-warning)" />
+			<IconCheck v-else :size="20" fill-color="var(--color-success)" />
 			{{ connectionState }}
 		</span>
+
+		<NcButton v-if="server && checked"
+			type="tertiary"
+			:title="t('spreed', 'Test this server')"
+			:aria-label="t('spreed', 'Test this server')"
+			@click="checkServerVersion">
+			<template #icon>
+				<IconReload :size="20" />
+			</template>
+		</NcButton>
 	</li>
 </template>
 
 <script>
-import AlertCircle from 'vue-material-design-icons/AlertCircle.vue'
-import AlertCircleOutline from 'vue-material-design-icons/AlertCircleOutline.vue'
-import Check from 'vue-material-design-icons/Check.vue'
-import Delete from 'vue-material-design-icons/Delete.vue'
+import IconAlertCircle from 'vue-material-design-icons/AlertCircle.vue'
+import IconAlertCircleOutline from 'vue-material-design-icons/AlertCircleOutline.vue'
+import IconCheck from 'vue-material-design-icons/Check.vue'
+import IconDelete from 'vue-material-design-icons/Delete.vue'
+import IconReload from 'vue-material-design-icons/Reload.vue'
 
 import { t } from '@nextcloud/l10n'
 
@@ -58,10 +70,11 @@ export default {
 	name: 'SignalingServer',
 
 	components: {
-		AlertCircle,
-		AlertCircleOutline,
-		Check,
-		Delete,
+		IconAlertCircle,
+		IconAlertCircleOutline,
+		IconCheck,
+		IconDelete,
+		IconReload,
 		NcButton,
 		NcCheckboxRadioSwitch,
 		NcLoadingIcon,