Merge pull request #40198 from nextcloud/fix/40015-The_apps_view_repr…

…esents_a_list_of_elements_and_should_be_implemented_as_such
nextcloud · Sep 6, 2023 · 5273299 · 5273299
2 parents 0c64e31 + 2235d3b
commit 5273299
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 9 deletions.
diff --git a/apps/settings/src/components/AppList.vue b/apps/settings/src/components/AppList.vue
@@ -103,13 +103,13 @@
 						:category="category" />
 				</template>
 			</table>
-			<template v-if="useAppStoreView">
+			<ul v-if="useAppStoreView" class="apps-store-view">
 				<AppItem v-for="app in apps"
 					:key="app.id"
 					:app="app"
 					:category="category"
 					:list-view="false" />
-			</template>
+			</ul>
 		</div>
 
 		<div id="apps-list-search" class="apps-list installed">
@@ -295,4 +295,8 @@ export default {
 		line-height: 30px;
 		color: var(--color-text-light);
 	}
+	.apps-store-view {
+		display: flex;
+		flex-wrap: wrap;
+	}
 </style>
diff --git a/apps/settings/src/components/AppList/AppItem.vue b/apps/settings/src/components/AppList/AppItem.vue
@@ -21,7 +21,7 @@
   -->
 
 <template>
-	<component :is="listView ? `tr` : `div`"
+	<component :is="listView ? `tr` : `li`"
 		class="section"
 		:class="{ selected: isSelected }"
 		@click="showAppDetails">

diff --git a/dist/settings-apps-view-7418.js b/dist/settings-apps-view-7418.js
diff --git a/dist/settings-apps-view-7418.js.map b/dist/settings-apps-view-7418.js.map
diff --git a/dist/settings-vue-settings-apps-users-management.js b/dist/settings-vue-settings-apps-users-management.js
diff --git a/dist/settings-vue-settings-apps-users-management.js.map b/dist/settings-vue-settings-apps-users-management.js.map