Merge pull request #16122 from nextcloud/fix/apps/sidebar-close-icon

Remove duplicate close-icon

settings/src/components/appDetails.vue

@@ -22,7 +22,6 @@
 
 <template>
 	<div id="app-details-view" style="padding: 20px;">
-		<a class="close icon-close" href="#" v-on:click="hideAppDetails"><span class="hidden-visually">Close</span></a>
 		<h2>
 			<div v-if="!app.preview" class="icon-settings-dark"></div>
 			<svg v-if="app.previewAsIcon && app.preview" width="32" height="32" viewBox="0 0 32 32">
@@ -126,14 +125,6 @@ export default {
 			this.groupCheckedAppsData = true;
 		}
 	},
-	methods: {
-		hideAppDetails() {
-			this.$router.push({
-				name: 'apps-category',
-				params: {category: this.category}
-			});
-		},
-	},
 	computed: {
 		appstoreUrl() {
 			return `https://apps.nextcloud.com/apps/${this.app.id}`;

settings/src/views/Apps.vue

@@ -31,7 +31,7 @@
 		<AppContent class="app-settings-content" :class="{ 'icon-loading': loadingList }">
 			<AppList :category="category" :app="currentApp" :search="searchQuery" />
 		</AppContent>
-		<AppSidebar v-if="id && currentApp" >
+		<AppSidebar v-if="id && currentApp" @close="hideAppDetails">
 			<AppDetails :category="category" :app="currentApp" />
 		</AppSidebar>
 	</Content>
@@ -79,6 +79,12 @@ export default {
 		},
 		resetSearch() {
 			this.setSearch('');
+		},
+		hideAppDetails() {
+			this.$router.push({
+				name: 'apps-category',
+				params: {category: this.category}
+			})
 		}
 	},
 	beforeMount() {