terrier-engine 4.52.0 → 4.52.3

package/data-dive/dives/dive-editor.ts

@@ -82,6 +82,7 @@ export default class DiveEditor extends ContentPart<DiveEditorState> {
         this.queries = new Map()
         for (const query of this.state.dive.query_data?.queries || []) {
             this.queries.set(query.id, query)
+            this.queryOrder.push(query.id)
             this.addQueryTab(query)
         }
 
@@ -92,7 +93,7 @@ export default class DiveEditor extends ContentPart<DiveEditorState> {
         })
 
         // Reorder queries in the list when the tab sort order is updated.
-        this.listenMessage(this.queryTabs.tabReorderedKey, m => {
+        this.listenMessage(this.queryTabs.tabsModifiedKey, m => {
             const { newOrder } = m.data
             this.queryOrder = newOrder
         })

package/package.json

@@ -4,7 +4,7 @@
   "files": [
     "*"
   ],
-  "version": "4.52.0",
+  "version": "4.52.3",
   "repository": {
     "type": "git",
     "url": "https://github.com/Terrier-Tech/terrier-engine"
@@ -35,4 +35,4 @@
     "vite-plugin-ruby": "^5.1.0",
     "vitest": "^2.1.5"
   }
-}
+}

package/terrier/tabs.ts

@@ -41,10 +41,10 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
     private tabs = {} as Record<string, TabDefinition>
     changeTabKey = Messages.typedKey<{ tabKey: string }>()
     changeSideKey = Messages.typedKey<{ side: TabSide }>()
-    tabReorderedKey = Messages.typedKey<{ newOrder: string[] }>()
+    tabsModifiedKey = Messages.typedKey<{ newOrder: string[] }>()
 
     async init() {
-        Object.assign(this.state, { reorderable: false }, this.state)
+        this.state = Object.assign({ reorderable: false }, this.state)
 
         this.onClick(this.changeTabKey, m => {
             log.info(`Clicked on tab ${m.data.tabKey}`)
@@ -62,17 +62,17 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
                 zoneClass: 'tt-tab-list',
                 targetClass: 'tab',
                 onSorted: (_, evt) => {
-                    this.renumberTabs(evt.toChildren)
+                    this.onTabsModified(evt.toChildren)
                 }
             })
         }
     }
 
-    renumberTabs(tabElementsMaybe?: HTMLElement[]) {
+    onTabsModified(tabElementsMaybe?: HTMLElement[]) {
         const tabElements = tabElementsMaybe ??
             Array.from(this.element?.querySelectorAll('.tt-tab-list') ?? [])
         const newOrder = tabElements.map(tabElement => tabElement.dataset.key)
-        this.emitMessage(this.tabReorderedKey, { newOrder })
+        this.emitMessage(this.tabsModifiedKey, { newOrder })
     }
 
     /**
@@ -90,7 +90,7 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
         this.tabs[tab.key] = Object.assign(existingTab, {
             state: 'enabled',
         }, tab)
-        this.renumberTabs()
+        this.onTabsModified()
         const part = this.makePart(constructor, state)
         existingTab.part = part
         this.dirty()
@@ -118,8 +118,8 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
 
         log.info(`Removing tab ${key}`, tab)
         delete this.tabs[key]
-        this.renumberTabs()
         this.removeChild(tab.part)
+        this.onTabsModified()
         this.state.currentTab = undefined
         this.dirty()
     }