From 29a41260ed4e15eb08b7b705cd6c4b0abf9567be Mon Sep 17 00:00:00 2001
From: "Grigorii K. Shartsev" <me@shgk.me>
Date: Sat, 21 Oct 2023 14:46:54 +0200
Subject: [PATCH] fix: use button for show settings instead of nav item

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
---
 src/Photos.vue | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/src/Photos.vue b/src/Photos.vue
index 07e21b5e0..15b7dc5aa 100644
--- a/src/Photos.vue
+++ b/src/Photos.vue
@@ -74,13 +74,21 @@
 				</NcAppNavigationItem>
 			</template>
 			<template #footer>
-				<NcAppNavigationItem :name="t('photos', 'Photos settings')" @click="showSettings">
-					<Cog slot="icon" :size="20" />
-				</NcAppNavigationItem>
+				<div class="app-navigation__footer">
+					<NcButton type="tertiary"
+						alignment="start"
+						wide
+						@click="showSettings">
+						<template #icon>
+							<Cog slot="icon" :size="20" />
+						</template>
+						{{ t('photos', 'Photos settings') }}
+					</NcButton>
+				</div>
 			</template>
 		</NcAppNavigation>
 		<NcAppContent :page-heading="pageHeading">
-			<router-view />
+			<RouterView />
 
 			<!-- svg img loading placeholder (linked to the File component) -->
 			<!-- eslint-disable-next-line vue/no-v-html (because it's an SVG file) -->
@@ -116,7 +124,7 @@ import ShareVariant from 'vue-material-design-icons/ShareVariant.vue'
 import AccountBoxMultipleOutline from 'vue-material-design-icons/AccountBoxMultipleOutline.vue'
 import Cog from 'vue-material-design-icons/Cog.vue'
 
-import { NcContent, NcAppContent, NcAppNavigation, NcAppNavigationItem } from '@nextcloud/vue'
+import { NcContent, NcAppContent, NcAppNavigation, NcAppNavigationItem, NcButton } from '@nextcloud/vue'
 
 import SettingsDialog from './components/Settings/SettingsDialog.vue'
 
@@ -148,6 +156,7 @@ export default {
 		NcAppContent,
 		NcAppNavigation,
 		NcAppNavigationItem,
+		NcButton,
 		NcContent,
 		SettingsDialog,
 	},
@@ -223,7 +232,11 @@ export default {
 	align-content: space-between;
 }
 
-.app-navigation__photos:deep .app-navigation-entry-icon.icon-photos {
+.app-navigation__photos :deep(.app-navigation-entry-icon.icon-photos) {
 	background-size: 20px;
 }
+
+.app-navigation__footer {
+	padding: calc(var(--default-grid-baseline) * 2);
+}
 </style>