Merge pull request #39609 from nextcloud/fix/focus-outline-favorites-…

…view fix(files): Remove duplicated outline on file name of vue file list
nextcloud · Aug 2, 2023 · 6ec35e3 · 6ec35e3
2 parents 0ec6efd + 88bbf38
commit 6ec35e3
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/apps/files/src/components/FileEntry.vue b/apps/files/src/components/FileEntry.vue
@@ -78,6 +78,7 @@
 			<a v-show="!isRenaming"
 				ref="basename"
 				:aria-hidden="isRenaming"
+				class="files-list__row-name-link"
 				v-bind="linkTo"
 				@click="execDefaultAction">
 				<!-- File name -->

diff --git a/apps/files/src/components/FilesListVirtual.vue b/apps/files/src/components/FilesListVirtual.vue
@@ -310,6 +310,11 @@ export default Vue.extend({
 				// Necessary for flex grow to work
 				min-width: 0;
 
+				// Already added to the inner text, see rule below
+				&:focus-visible {
+					outline: none;
+				}
+
 				// Keyboard indicator a11y
 				&:focus .files-list__row-name-text,
 				&:focus-visible .files-list__row-name-text {

diff --git a/dist/files-main.js b/dist/files-main.js
diff --git a/dist/files-main.js.map b/dist/files-main.js.map