From c94f5268d153fbe20e6a78a0a523f1ed531620a9 Mon Sep 17 00:00:00 2001
From: Mauro Mura <Mauro-Efisio.Mura@t-systems.com>
Date: Thu, 28 Nov 2024 13:04:31 +0100
Subject: [PATCH] fix lint errors

---
 src/components/DisplaySettings.vue  |  13 +--
 src/components/NcIconSvgWrapper.vue | 133 ----------------------------
 src/components/SkipActions.vue      |  54 ++++++-----
 src/js/skipactions.js               |   1 -
 4 files changed, 33 insertions(+), 168 deletions(-)
 delete mode 100644 src/components/NcIconSvgWrapper.vue

diff --git a/src/components/DisplaySettings.vue b/src/components/DisplaySettings.vue
index bc40762..4214f75 100644
--- a/src/components/DisplaySettings.vue
+++ b/src/components/DisplaySettings.vue
@@ -9,21 +9,22 @@
 			@click.stop="toggleOpen">
 			{{ t('nmctheme', 'Display settings') }}
 		</button>
-		<div id="display-settings-list" class="display-settings__list" :class="{open: isOpened}" aria-labelledby="display-settings-btn">
+		<div id="display-settings-list"
+			class="display-settings__list"
+			:class="{open: isOpened}"
+			aria-labelledby="display-settings-btn">
 			<NcCheckboxRadioSwitch v-if="textAvailable"
 				:checked="userConfig.show_folder_info"
 				:disabled="isOpened ? false : true"
 				@update:checked="setConfig('show_folder_info', $event)">
 				{{ t('nmctheme', 'Show folder info text') }}
 			</NcCheckboxRadioSwitch>
-			<NcCheckboxRadioSwitch 
-				:checked="userConfig.show_hidden"
+			<NcCheckboxRadioSwitch :checked="userConfig.show_hidden"
 				:disabled="isOpened ? false : true"
 				@update:checked="setConfig('show_hidden', $event)">
 				{{ t('files', 'Show hidden files') }}
 			</NcCheckboxRadioSwitch>
-			<NcCheckboxRadioSwitch 
-				:checked="userConfig.crop_image_previews"
+			<NcCheckboxRadioSwitch :checked="userConfig.crop_image_previews"
 				:disabled="isOpened ? false : true"
 				@update:checked="setConfig('crop_image_previews', $event)">
 				{{ t('files', 'Crop image previews') }}
@@ -92,7 +93,7 @@ export default {
 
 		&:hover {
 			color: var(--color-primary-hover);
-    		cursor: pointer;
+			cursor: pointer;
 		}
 
 		// arrow
diff --git a/src/components/NcIconSvgWrapper.vue b/src/components/NcIconSvgWrapper.vue
deleted file mode 100644
index c604079..0000000
--- a/src/components/NcIconSvgWrapper.vue
+++ /dev/null
@@ -1,133 +0,0 @@
-<template>
-	<span v-if="!cleanSvg"
-		v-bind="attributes">
-		<svg viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg">
-			<path :d="path" />
-		</svg>
-	</span>
-	<span v-else
-		v-bind="attributes"
-		v-html="cleanSvg" /> <!-- eslint-disable-line vue/no-v-html -->
-</template>
-
-<script>
-import Vue from 'vue'
-import DOMPurify from 'dompurify'
-
-export default {
-	name: 'NcIconSvgWrapper',
-
-	props: {
-		/**
-		 * Set if the icon should be used as inline content e.g. within text.
-		 * By default the icon is made a block element for use inside `icon`-slots.
-		 */
-		inline: {
-			type: Boolean,
-			default: false,
-		},
-
-		/**
-		 * Raw SVG string to render
-		 */
-		svg: {
-			type: String,
-			default: '',
-		},
-
-		/**
-		 * Label of the icon, used in aria-label
-		 */
-		name: {
-			type: String,
-			default: '',
-		},
-
-		/**
-		 * Raw SVG path to render. Takes precedence over the SVG string in the `svg` prop.
-		 */
-		path: {
-			type: String,
-			default: '',
-		},
-
-		/**
-		 * Size of the icon to show. Only use if not using within an icon slot.
-		 * Defaults to 20px which is the Nextcloud icon size for all icon slots.
-		 * @default 20
-		 */
-		size: {
-			type: [Number, String],
-			default: 20,
-			validator: (value) => typeof value === 'number' || value === 'auto',
-		},
-	},
-
-	computed: {
-		/**
-		 * Icon size used in CSS
-		 */
-		iconSize() {
-			return typeof this.size === 'number' ? `${this.size}px` : this.size
-		},
-
-		cleanSvg() {
-			if (!this.svg || this.path) {
-				return
-			}
-
-			const svg = DOMPurify.sanitize(this.svg)
-
-			const svgDocument = new DOMParser().parseFromString(svg, 'image/svg+xml')
-
-			if (svgDocument.querySelector('parsererror')) {
-				Vue.util.warn('SVG is not valid')
-				return ''
-			}
-
-			if (svgDocument.documentElement.id) {
-				svgDocument.documentElement.removeAttribute('id')
-			}
-
-			return svgDocument.documentElement.outerHTML
-		},
-		attributes() {
-			return {
-				class: ['icon-vue', { 'icon-vue--inline': this.inline }],
-				style: {
-					'--icon-size': this.iconSize,
-				},
-				role: 'img',
-				'aria-hidden': !this.name ? true : undefined,
-				'aria-label': this.name || undefined,
-			}
-		},
-	},
-}
-</script>
-
-<style lang="scss" scoped>
-.icon-vue {
-	display: flex;
-	justify-content: center;
-	align-items: center;
-	min-width: var(--default-clickable-area);
-	min-height: var(--default-clickable-area);
-	opacity: 1;
-
-	&--inline {
-		display: inline-flex;
-		min-width: fit-content;
-		min-height: fit-content;
-		vertical-align: text-bottom;
-	}
-
-	&:deep(svg) {
-		fill: currentColor;
-		width: var(--icon-size, 20px);
-		height: var(--icon-size, 20px);
-		max-width: var(--icon-size, 20px);
-		max-height: var(--icon-size, 20px);
-	}
-}
-</style>
diff --git a/src/components/SkipActions.vue b/src/components/SkipActions.vue
index 83d4794..72b6d10 100644
--- a/src/components/SkipActions.vue
+++ b/src/components/SkipActions.vue
@@ -1,36 +1,36 @@
 <template>
 	<div id="content-vue" :class="['content', `app-${appName.toLowerCase()}`]">
-        <!-- TODO: with vue3 the `selector` attribute needs to be changed to `to="#skip-actions"` -->
-        <Teleport selector="#skip-actions">
-            <div class="vue-skip-actions__container">
-                <div class="vue-skip-actions__headline">
-                    {{ t('core', 'Keyboard navigation help') }}
-                </div>
-                <div class="vue-skip-actions__buttons">
-                    <NcButton v-show="hasAppNavigation"
-                        type="tertiary"
-                        href="#app-navigation-vue"
-                        @click.prevent="openAppNavigation"
-                        @focusin="currentFocus = 'navigation'"
-                        @mouseover="currentFocus = 'navigation'">
-                        {{ t('core', 'Skip to app navigation') }}
-                    </NcButton>
-                    <NcButton type="tertiary"
-                        href="#app-content-vue"
-                        @focusin="currentFocus = 'content'"
-                        @mouseover="currentFocus = 'content'">
-                        {{ t('core', 'Skip to main content') }}
-                    </NcButton>
-                </div>
+		<!-- TODO: with vue3 the `selector` attribute needs to be changed to `to="#skip-actions"` -->
+		<Teleport selector="#skip-actions">
+			<div class="vue-skip-actions__container">
+				<div class="vue-skip-actions__headline">
+					{{ t('core', 'Keyboard navigation help') }}
+				</div>
+				<div class="vue-skip-actions__buttons">
+					<NcButton v-show="hasAppNavigation"
+						type="tertiary"
+						href="#app-navigation-vue"
+						@click.prevent="openAppNavigation"
+						@focusin="currentFocus = 'navigation'"
+						@mouseover="currentFocus = 'navigation'">
+						{{ t('core', 'Skip to app navigation') }}
+					</NcButton>
+					<NcButton type="tertiary"
+						href="#app-content-vue"
+						@focusin="currentFocus = 'content'"
+						@mouseover="currentFocus = 'content'">
+						{{ t('core', 'Skip to main content') }}
+					</NcButton>
+				</div>
 				<NcIconSvgWrapper v-show="!isMobile"
 					class="vue-skip-actions__image"
 					:svg="currentImage"
 					size="auto" />
-            </div>
+			</div>
             &nbsp;<!-- TODO: Remove with vue3! This is a bug of vue-simple-portal that does not allow a single child, ref LinusBorg/vue-simple-portal#20 -->
-        </Teleport>
-        <slot />
-    </div>
+		</Teleport>
+		<slot />
+	</div>
 </template>
 
 <script>
@@ -74,7 +74,6 @@ export default {
 	},
 	computed: {
 		currentImage() {
-            console.log(this.currentFocus)
 			if (this.currentFocus === 'navigation') {
 				return navigationSvg
 			}
@@ -101,7 +100,6 @@ export default {
 			})
 		},
 		setAppNavigation(value) {
-            console.log(value)
 			this.hasAppNavigation = value
 			// If app navigation is available and no focus was set yet, set it to navigation as it is the first button
 			if (this.currentFocus === '') {
diff --git a/src/js/skipactions.js b/src/js/skipactions.js
index dbab452..b5304d9 100644
--- a/src/js/skipactions.js
+++ b/src/js/skipactions.js
@@ -9,7 +9,6 @@ window.addEventListener('DOMContentLoaded', function() {
 	if (!skipActions) return
 	const anchor = document.createElement('div')
 
-
 	anchor.id = 'skip-actions-app'
 	skipActions.appendChild(anchor)