From 934953801d0e9dcddc9ae0efaeaac2e0f38cfffb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Raimund=20Schl=C3=BC=C3=9Fler?=
 <raimund.schluessler@mailbox.org>
Date: Thu, 16 Mar 2023 17:11:33 +0100
Subject: [PATCH] Sort tabs after registering a tab
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Raimund Schlüßler <raimund.schluessler@mailbox.org>
---
 .../NcAppSidebar/NcAppSidebarTabs.vue         | 20 +++++++++----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/src/components/NcAppSidebar/NcAppSidebarTabs.vue b/src/components/NcAppSidebar/NcAppSidebarTabs.vue
index a18e8cec84..1925636a35 100644
--- a/src/components/NcAppSidebar/NcAppSidebarTabs.vue
+++ b/src/components/NcAppSidebar/NcAppSidebarTabs.vue
@@ -36,7 +36,7 @@
 			@keydown.33.exact.prevent="focusFirstTab"
 			@keydown.34.exact.prevent="focusLastTab">
 			<ul>
-				<li v-for="tab in sortedTabs" :key="tab.id" class="app-sidebar-tabs__tab">
+				<li v-for="tab in tabs" :key="tab.id" class="app-sidebar-tabs__tab">
 					<a :id="tab.id"
 						:aria-controls="`tab-${tab.id}`"
 						:aria-selected="activeTab === tab.id"
@@ -117,16 +117,6 @@ export default {
 		currentTabIndex() {
 			return this.tabs.findIndex(tab => tab.id === this.activeTab)
 		},
-		sortedTabs() {
-			return [...this.tabs].sort((a, b) => {
-				const orderA = a.order || 0
-				const orderB = b.order || 0
-				if (orderA === orderB) {
-					return OC.Util.naturalSortCompare(a.name, b.name)
-				}
-				return orderA - orderB
-			})
-		},
 	},
 
 	watch: {
@@ -228,6 +218,14 @@ export default {
 		 */
 		registerTab(tab) {
 			this.tabs.push(tab)
+			this.tabs.sort((a, b) => {
+				const orderA = a.order || 0
+				const orderB = b.order || 0
+				if (orderA === orderB) {
+					return OC.Util.naturalSortCompare(a.name, b.name)
+				}
+				return orderA - orderB
+			})
 			this.updateActive()
 		},