sidebar: allow turning off tags view by default

With #37065, there is no way to hide the tags from the sidebar
by default when they are not relevant or redundant (e.g. the tab
may already show the file's tags). This can be annyoing especially
when the file has many tags. This patch adds an option to hide
the tags from the sidebar by default (the user can still open
the tags tab manually).

This also reduces one request when opening the sidebar when the
tags are turned off, since all tags don't need to be fetched
anymore.

Signed-off-by: Varun Patil <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>

Author: pulsejet

apps/files/src/sidebar.js

@@ -59,4 +59,5 @@ window.addEventListener('DOMContentLoaded', function() {
 	window.OCA.Files.Sidebar.open = AppSidebar.open
 	window.OCA.Files.Sidebar.close = AppSidebar.close
 	window.OCA.Files.Sidebar.setFullScreenMode = AppSidebar.setFullScreenMode
+	window.OCA.Files.Sidebar.setShowTagsDefault = AppSidebar.setShowTagsDefault
 })

apps/files/src/views/Sidebar.vue

@@ -36,7 +36,7 @@
 		<!-- TODO: create a standard to allow multiple elements here? -->
 		<template v-if="fileInfo" #description>
 			<div class="sidebar__description">
-				<SystemTags v-if="isSystemTagsEnabled"
+				<SystemTags v-if="isSystemTagsEnabled && showTagsDefault"
 					v-show="showTags"
 					:file-id="fileInfo.id"
 					@has-tags="value => showTags = value" />
@@ -138,6 +138,7 @@ export default {
 			// reactive state
 			Sidebar: OCA.Files.Sidebar.state,
 			showTags: false,
+			showTagsDefault: true,
 			error: null,
 			loading: true,
 			fileInfo: null,
@@ -455,7 +456,7 @@ export default {
 		 * Toggle the tags selector
 		 */
 		toggleTags() {
-			this.showTags = !this.showTags
+			this.showTagsDefault = this.showTags = !this.showTags
 		},
 
 		/**
@@ -529,6 +530,15 @@ export default {
 			}
 		},
 
+		/**
+		 * Allow to set whether tags should be shown by default from OCA.Files.Sidebar
+		 *
+		 * @param {boolean} showTagsDefault - Whether or not to show the tags by default.
+		 */
+		setShowTagsDefault(showTagsDefault) {
+			this.showTagsDefault = showTagsDefault
+		},
+
 		/**
 		 * Emit SideBar events.
 		 */