Merge pull request #2020 from nextcloud/bugfix/noid/navigation

Fix navigation issue with leftover nodes

src/components/navigation/AppNavigation.vue

@@ -21,61 +21,45 @@
   -->
 
 <template>
-	<div id="app-navigation" :class="{'icon-loading': loading}">
+	<AppNavigationVue :class="{'icon-loading': loading}">
-		<AppNavigationVue>
+		<ul>
-			<ul>
+			<AppNavigationBoardCategory
-				<AppNavigationBoardCategory
+				id="deck-navigation-all"
-					id="deck-navigation-all"
+				to="/board"
-					to="/board"
+				:text="t('deck', 'All boards')"
-					:text="t('deck', 'All boards')"
+				:boards="noneArchivedBoards"
-					:boards="noneArchivedBoards"
+				:open-on-add-boards="true"
-					:open-on-add-boards="true"
+				icon="icon-deck" />
-					icon="icon-deck" />
+			<AppNavigationBoardCategory
-				<AppNavigationBoardCategory
+				id="deck-navigation-archived"
-					id="deck-navigation-archived"
+				to="/board/archived"
-					to="/board/archived"
+				:text="t('deck', 'Archived boards')"
-					:text="t('deck', 'Archived boards')"
+				:boards="archivedBoards"
-					:boards="archivedBoards"
+				icon="icon-archive" />
-					icon="icon-archive" />
+			<AppNavigationBoardCategory
-				<AppNavigationBoardCategory
+				id="deck-navigation-shared"
-					id="deck-navigation-shared"
+				to="/board/shared"
-					to="/board/shared"
+				:text="t('deck', 'Shared with you')"
-					:text="t('deck', 'Shared with you')"
+				:boards="sharedBoards"
-					:boards="sharedBoards"
+				icon="icon-shared" />
-					icon="icon-shared" />
-				<AppNavigationAddBoard v-if="canCreate" />
-			</ul>
-			<AppNavigationSettings>
-				<div>
-					<Multiselect v-model="groupLimit"
-						:class="{'icon-loading-small': groupLimitDisabled}"
-						open-direction="bottom"
-						:options="groups"
-						:multiple="true"
-						:disabled="groupLimitDisabled"
-						:placeholder="t('deck', 'Limit deck usage of groups')"
-						label="displayname"
-						track-by="id"
-						@input="updateConfig" />
-					<p>{{ t('deck', 'Limiting Deck will block users not part of those groups from creating their own boards. Users will still be able to work on boards that have been shared with them.') }}</p>
-				</div>
-			</AppNavigationSettings>
-		</AppNavigationVue>
-
-		<ul id="deck-navigation">
 			<AppNavigationAddBoard v-if="canCreate" />
 		</ul>
-		<div v-if="isAdmin"
+		<AppNavigationSettings>
-			id="app-settings"
+			<div>
-			v-click-outside="closeMenu"
+				<Multiselect v-model="groupLimit"
-			:class="{open: opened}">
+					:class="{'icon-loading-small': groupLimitDisabled}"
-			<div id="app-settings-header">
+					open-direction="bottom"
-				<button class="settings-button" @click="toggleMenu">
+					:options="groups"
-					{{ t('deck', 'Settings') }}
+					:multiple="true"
-				</button>
+					:disabled="groupLimitDisabled"
+					:placeholder="t('deck', 'Limit deck usage of groups')"
+					label="displayname"
+					track-by="id"
+					@input="updateConfig" />
+				<p>{{ t('deck', 'Limiting Deck will block users not part of those groups from creating their own boards. Users will still be able to work on boards that have been shared with them.') }}</p>
 			</div>
-		</div>
+		</AppNavigationSettings>
-	</div>
+	</AppNavigationVue>
 </template>
 
 <script>
@@ -151,12 +135,6 @@ export default {
 		}
 	},
 	methods: {
-		toggleMenu() {
-			this.opened = !this.opened
-		},
-		closeMenu() {
-			this.opened = false
-		},
 		updateConfig() {
 			this.groupLimitDisabled = true
 			axios.post(generateUrl('apps/deck/config/groupLimit'), {