fix(TheEditor): No sidebar item deactivation - don't blow up

Signed-off-by: Michał Dziekoński <[email protected]>
mainsail-crew · Sep 14, 2024 · 52de8fc · 52de8fc
1 parent e69b976
commit 52de8fc
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/components/TheEditor.vue b/src/components/TheEditor.vue
@@ -67,7 +67,7 @@
                             dense
                             :active="structureActive"
                             :open="structureOpen"
-                            item-key="line"
+                            :item-key="treeviewItemKeyProp"
                             :items="configFileStructure"
                             class="w-100"
                             @update:active="activeChanges">
@@ -189,6 +189,7 @@ export default class TheEditor extends Mixins(BaseMixin) {
     dialogConfirmChange = false
     dialogDevices = false
     fileStructureSidebar = true
+    treeviewItemKeyProp = 'line' as const
     structureActive: number[] = []
     structureOpen: number[] = []
     structureActiveChangedBySidebar: boolean = false
@@ -431,13 +432,18 @@ export default class TheEditor extends Mixins(BaseMixin) {
         this.structureActiveChangedBySidebar = true
     }
 
-    activeChanges(key: any) {
+    activeChanges(activeItems: Array<ConfigFileSection[typeof this.treeviewItemKeyProp]>) {
         if (!this.structureActiveChangedBySidebar) {
             return
         }
 
         this.structureActiveChangedBySidebar = false
-        this.editor?.gotoLine(key)
+
+        if (!activeItems.length) {
+            return
+        }
+
+        this.editor?.gotoLine(activeItems[0])
     }
 
     lineChanges(line: number) {