terrier-engine 4.52.3 → 4.52.5

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

@@ -49,8 +49,6 @@ export default class DiveEditor extends ContentPart<DiveEditorState> {
     static readonly diveChangedKey = Messages.untypedKey()
 
     queries = new Map<string, Query>()
-    // Array of map keys to give the queries a sort order.
-    queryOrder = new Array<string>()
 
     async init() {
         this.queryTabs = this.makePart(TabContainerPart, { side: 'top', reorderable: true })
@@ -82,7 +80,6 @@ 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,12 +89,6 @@ export default class DiveEditor extends ContentPart<DiveEditorState> {
             this.queryTabs.updateTab({ key: query.id, title: query.name })
         })
 
-        // Reorder queries in the list when the tab sort order is updated.
-        this.listenMessage(this.queryTabs.tabsModifiedKey, m => {
-            const { newOrder } = m.data
-            this.queryOrder = newOrder
-        })
-
         this.onClick(this.newQueryKey, _ => {
             this.app.showModal(NewQueryModal, { editor: this as DiveEditor, schema: this.state.schema })
         })
@@ -188,7 +179,7 @@ export default class DiveEditor extends ContentPart<DiveEditorState> {
 
         return {
             ...this.state.dive,
-            query_data: { queries: this.queryOrder.map(id => queries.get(id)!) }
+            query_data: { queries: this.queryTabs.getTabOrder().map(id => queries.get(id)!) }
         }
     }
 

package/package.json

@@ -4,7 +4,7 @@
   "files": [
     "*"
   ],
-  "version": "4.52.3",
+  "version": "4.52.5",
   "repository": {
     "type": "git",
     "url": "https://github.com/Terrier-Tech/terrier-engine"

package/terrier/gen/hub-icons.ts

@@ -1,4 +1,4 @@
-// This file was automatically generated on 2024-10-30 08:24:15 -0500, DO NOT EDIT IT MANUALLY!
+// This file was automatically generated on 2025-05-30 09:45:22 -0500, DO NOT EDIT IT MANUALLY!
 
 import {ColorName} from "../theme"
 import {PartTag} from "tuff-core/parts"
@@ -341,6 +341,10 @@ import StatusRaw from '../images/icons/status.svg?raw'
 // @ts-ignore
 import StatusSrc from '../images/icons/status.svg'
 // @ts-ignore
+import StepDemoRaw from '../images/icons/step_demo.svg?raw'
+// @ts-ignore
+import StepDemoSrc from '../images/icons/step_demo.svg'
+// @ts-ignore
 import StepDeployRaw from '../images/icons/step_deploy.svg?raw'
 // @ts-ignore
 import StepDeploySrc from '../images/icons/step_deploy.svg'
@@ -754,6 +758,10 @@ export const IconDefs: Record<HubIconName,{ raw: string, src: string }> = {
         raw: StatusRaw,
         src: StatusSrc,
     },
+    "hub-step_demo": {
+        raw: StepDemoRaw,
+        src: StepDemoSrc,
+    },
     "hub-step_deploy": {
         raw: StepDeployRaw,
         src: StepDeploySrc,
@@ -833,7 +841,7 @@ export const IconDefs: Record<HubIconName,{ raw: string, src: string }> = {
 }
 
 const Names = [
-    'hub-active', 'hub-admin', 'hub-archive', 'hub-arrow_down', 'hub-arrow_left', 'hub-arrow_right', 'hub-arrow_up', 'hub-assign', 'hub-attachment', 'hub-back', 'hub-badge', 'hub-board', 'hub-bookmark', 'hub-bookmark_add', 'hub-bookmark_remove', 'hub-bookmarked', 'hub-bookmarks', 'hub-branch', 'hub-bug', 'hub-calculator', 'hub-checkmark', 'hub-close', 'hub-clypboard', 'hub-comment', 'hub-complete', 'hub-dashboard', 'hub-data_pull', 'hub-data_update', 'hub-database', 'hub-day', 'hub-delete', 'hub-documentation', 'hub-edit', 'hub-existing_child', 'hub-existing_parent', 'hub-feature', 'hub-flex', 'hub-forward', 'hub-github', 'hub-history', 'hub-home', 'hub-image', 'hub-inbox', 'hub-info', 'hub-internal', 'hub-issue', 'hub-lane', 'hub-lane_asap', 'hub-lane_days', 'hub-lane_hours', 'hub-lane_weeks', 'hub-lanes_board', 'hub-level_complete', 'hub-level_highway', 'hub-level_on_ramp', 'hub-level_parking', 'hub-link', 'hub-list_view', 'hub-metrics', 'hub-minus', 'hub-new_child', 'hub-new_parent', 'hub-night', 'hub-origin', 'hub-pending', 'hub-plot', 'hub-plus', 'hub-post', 'hub-posts', 'hub-pr_closed', 'hub-pr_merged', 'hub-pr_open', 'hub-prioritized', 'hub-project', 'hub-question', 'hub-reaction', 'hub-recent', 'hub-refresh', 'hub-related_posts', 'hub-releases', 'hub-request', 'hub-settings', 'hub-split_view', 'hub-status', 'hub-step_deploy', 'hub-step_develop', 'hub-step_investigate', 'hub-step_review', 'hub-step_test', 'hub-steps', 'hub-steps_board', 'hub-subscribe', 'hub-support', 'hub-team', 'hub-terrier', 'hub-thumbs_up', 'hub-type', 'hub-unprioritized', 'hub-upload', 'hub-user', 'hub-users', 'hub-website', 'hub-week'
+    'hub-active', 'hub-admin', 'hub-archive', 'hub-arrow_down', 'hub-arrow_left', 'hub-arrow_right', 'hub-arrow_up', 'hub-assign', 'hub-attachment', 'hub-back', 'hub-badge', 'hub-board', 'hub-bookmark', 'hub-bookmark_add', 'hub-bookmark_remove', 'hub-bookmarked', 'hub-bookmarks', 'hub-branch', 'hub-bug', 'hub-calculator', 'hub-checkmark', 'hub-close', 'hub-clypboard', 'hub-comment', 'hub-complete', 'hub-dashboard', 'hub-data_pull', 'hub-data_update', 'hub-database', 'hub-day', 'hub-delete', 'hub-documentation', 'hub-edit', 'hub-existing_child', 'hub-existing_parent', 'hub-feature', 'hub-flex', 'hub-forward', 'hub-github', 'hub-history', 'hub-home', 'hub-image', 'hub-inbox', 'hub-info', 'hub-internal', 'hub-issue', 'hub-lane', 'hub-lane_asap', 'hub-lane_days', 'hub-lane_hours', 'hub-lane_weeks', 'hub-lanes_board', 'hub-level_complete', 'hub-level_highway', 'hub-level_on_ramp', 'hub-level_parking', 'hub-link', 'hub-list_view', 'hub-metrics', 'hub-minus', 'hub-new_child', 'hub-new_parent', 'hub-night', 'hub-origin', 'hub-pending', 'hub-plot', 'hub-plus', 'hub-post', 'hub-posts', 'hub-pr_closed', 'hub-pr_merged', 'hub-pr_open', 'hub-prioritized', 'hub-project', 'hub-question', 'hub-reaction', 'hub-recent', 'hub-refresh', 'hub-related_posts', 'hub-releases', 'hub-request', 'hub-settings', 'hub-split_view', 'hub-status', 'hub-step_demo', 'hub-step_deploy', 'hub-step_develop', 'hub-step_investigate', 'hub-step_review', 'hub-step_test', 'hub-steps', 'hub-steps_board', 'hub-subscribe', 'hub-support', 'hub-team', 'hub-terrier', 'hub-thumbs_up', 'hub-type', 'hub-unprioritized', 'hub-upload', 'hub-user', 'hub-users', 'hub-website', 'hub-week'
 ] as const
 
 export type HubIconName = typeof Names[number]

package/terrier/images/icons/step_demo.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="32" height="32" viewBox="0 0 32 32"><g fill="currentColor" fill-rule="evenodd"><path fill-opacity="0.33" stroke="currentColor" stroke-linejoin="round" stroke-width="2" d="M5 7h22v14H5z"/><path d="M14 21h4v3h-4z"/><rect width="8" height="2" x="12" y="24" rx="1"/><path d="M13 9.902v8.196a.5.5 0 0 0 .765.424l6.557-4.098a.5.5 0 0 0 0-.848l-6.557-4.098a.5.5 0 0 0-.765.424Z"/></g></svg>

package/terrier/images/optimized/icon-step_demo.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="32" height="32" viewBox="0 0 32 32"><g fill="#000" fill-rule="evenodd"><path fill-opacity=".25" stroke="#000" stroke-linejoin="round" stroke-width="2" d="M5 7h22v14H5z"/><path d="M14 21h4v3h-4z"/><rect width="8" height="2" x="12" y="24" rx="1"/><path d="M13 9.902v8.196a.5.5 0 0 0 .765.424l6.557-4.098a.5.5 0 0 0 0-.848l-6.557-4.098a.5.5 0 0 0-.765.424Z"/></g></svg>

package/terrier/images/raw/icon-step_demo.svg

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<svg width="32px" height="32px" viewBox="0 0 32 32" version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
+    <title>icon-step_demo</title>
+    <g id="icon-step_demo" stroke="none" fill="none" fill-rule="evenodd">
+        <rect id="Rectangle" stroke="#000000" stroke-width="2" fill-opacity="0.25" fill="#000000" stroke-linejoin="round" x="5" y="7" width="22" height="14"></rect>
+        <rect id="Rectangle" fill="#000000" x="14" y="21" width="4" height="3"></rect>
+        <rect id="Rectangle" fill="#000000" x="12" y="24" width="8" height="2" rx="1"></rect>
+        <path d="M13,9.90212382 L13,18.0978762 C13,18.3740186 13.2238576,18.5978762 13.5,18.5978762 C13.5937099,18.5978762 13.6855337,18.5715415 13.7649995,18.5218753 L20.3216014,14.4239992 C20.5557696,14.277644 20.6269562,13.9691688 20.480601,13.7350005 C20.4403607,13.670616 20.3859859,13.6162412 20.3216014,13.5760008 L13.7649995,9.47812467 C13.5308312,9.3317695 13.222356,9.40295609 13.0760008,9.63712435 C13.0263347,9.71659015 13,9.80841396 13,9.90212382 Z" id="Path" fill="#000000"></path>
+    </g>
+</svg>

package/terrier/plugins/load-on-scroll-plugin.ts

@@ -21,7 +21,7 @@ export type LoadOnScrollOptions<TState> = {
     loadNextStates: (existingStates: TState[]) => Promise<TState[] | undefined>
     // The root element whose viewport is scrolling. If not provided, the document viewport is used.
     // Usually not required, but if using intersectThreshold on a scrollable element, might be necessary.
-    intersectRootSelector: string
+    intersectRootSelector?: string
     // Percentage of the last item in the collection that must be visible before the next state is loaded.
     // If not provided, a default value of 25% is used. Must be between 0 and 1.
     intersectThreshold?: number

package/terrier/tabs.ts

@@ -38,10 +38,11 @@ export type TabContainerState = {
 
 export class TabContainerPart extends TerrierPart<TabContainerState> {
 
-    private tabs = {} as Record<string, TabDefinition>
+    private tabs = new Map as Map<string, TabDefinition>
+    private tabOrder = [] as string[]
     changeTabKey = Messages.typedKey<{ tabKey: string }>()
     changeSideKey = Messages.typedKey<{ side: TabSide }>()
-    tabsModifiedKey = Messages.typedKey<{ newOrder: string[] }>()
+    tabsModifiedKey = Messages.untypedKey()
 
     async init() {
         this.state = Object.assign({ reorderable: false }, this.state)
@@ -59,22 +60,29 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
 
         if (this.state.reorderable) {
             this.makePlugin(SortablePlugin, {
-                zoneClass: 'tt-tab-list',
+                zoneClass: `tablist-${this.id}`,
                 targetClass: 'tab',
-                onSorted: (_, evt) => {
-                    this.onTabsModified(evt.toChildren)
+                onSorted: () => {
+                    // Get only our tab container, as other tab contains may exist inside this one.
+                    const ourTabList = this.element?.getElementsByClassName(`tablist-${this.id}`)[0]
+                    const tabElements = Array.from(ourTabList?.getElementsByClassName('tab') || []) as HTMLElement[]
+                    this.tabOrder = tabElements.map(tabElement => tabElement.dataset?.key!)
+                    this.#onTabsModified()
+                    this.dirty()
                 }
             })
         }
     }
 
-    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.tabsModifiedKey, { newOrder })
+    #onTabsModified() {
+        this.emitMessage(this.tabsModifiedKey, null)
     }
 
+    /**
+     * Gets the current tab order as an array of keys.
+     */
+    getTabOrder = () => [...this.tabOrder]
+
     /**
      * Adds or overwrites an existing tab.
      * @param tab initial parameters for the tab
@@ -86,13 +94,15 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
         constructor: { new(p: PartParent, id: string, state: PartStateType): PartType; },
         state: InferredPartStateType
     ): PartType {
-        const existingTab = this.tabs[tab.key] ?? {}
-        this.tabs[tab.key] = Object.assign(existingTab, {
-            state: 'enabled',
-        }, tab)
-        this.onTabsModified()
+        const existingTab = this.tabs.get(tab.key) ?? {}
         const part = this.makePart(constructor, state)
-        existingTab.part = part
+        this.tabs.set(tab.key, Object.assign(existingTab, {
+            state: 'enabled',
+            part
+        }, tab))
+        if (!this.tabOrder.includes(tab.key))
+            this.tabOrder.push(tab.key)
+        this.#onTabsModified()
         this.dirty()
         return part
     }
@@ -102,7 +112,7 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
      * @param tab
      */
     updateTab(tab: TabParams): void {
-        const existingTab = this.tabs[tab.key]
+        const existingTab = this.tabs.get(tab.key)
         if (!existingTab) throw `Tab with key '${tab.key}' does not exist!`
         Object.assign(existingTab, tab)
         this.dirty()
@@ -113,13 +123,14 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
      * @param key
      */
     removeTab(key: string) {
-        const tab = this.tabs[key]
+        const tab = this.tabs.get(key)
         if (!tab) return log.warn(`No tab ${key} to remove!`)
 
         log.info(`Removing tab ${key}`, tab)
-        delete this.tabs[key]
+        this.tabs.delete(key)
+        this.tabOrder.splice(this.tabOrder.indexOf(key), 1)
         this.removeChild(tab.part)
-        this.onTabsModified()
+        this.#onTabsModified()
         this.state.currentTab = undefined
         this.dirty()
     }
@@ -129,10 +140,10 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
      * @param tabKey
      */
     showTab(tabKey: string) {
-        if (!(tabKey in this.tabs)) {
+        if (!this.tabs.has(tabKey)) {
             throw `Unknown tab key ${tabKey}`
         }
-        if (this.tabs[tabKey].state != 'enabled') return // tab exists but is not enabled
+        if (this.tabs.get(tabKey)?.state != 'enabled') return // tab exists but is not enabled
         if (this.state.currentTab === tabKey) return // tab is already selected
 
         this.state.currentTab = tabKey
@@ -143,7 +154,7 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
      * Gets the current tab key.
      */
     get currentTagKey(): string | undefined {
-        return this.state.currentTab || Object.keys(this.tabs)[0]
+        return this.state.currentTab || this.tabOrder[0]
     }
 
 
@@ -161,20 +172,20 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
         this.dirty()
     }
 
-
-
     render(parent: PartTag) {
         let currentTabKey = this.state.currentTab
         if (!currentTabKey) {
             log.debug("no current tab specified, selecting first enabled tab")
-            currentTabKey = Object.values(this.tabs).find(t => t.state == 'enabled')?.key
+            currentTabKey = this.tabs.values().find(t => t.state == 'enabled')?.key
         }
         parent.div('tt-tab-container', this.state.side, container => {
             container.div('.tt-flex.tt-tab-list', tabList => {
+                tabList.class(`tablist-${this.id}`)
                 if (this._beforeActions.length) {
                     this.theme.renderActions(tabList, this._beforeActions, { defaultClass: 'action' })
                 }
-                for (const tab of Object.values(this.tabs)) {
+                for (const tabKey of this.tabOrder) {
+                    const tab = this.tabs.get(tabKey)!
                     if (tab.state == 'hidden') continue
 
                     tabList.a('.tab', a => {
@@ -195,7 +206,7 @@ export class TabContainerPart extends TerrierPart<TabContainerState> {
             })
 
             if (currentTabKey) {
-                const currentTab = this.tabs[currentTabKey]
+                const currentTab = this.tabs.get(currentTabKey)!
                 container.div('.tt-tab-content', panel => {
                     if (currentTab.classes?.length) panel.class(...currentTab.classes)
                     panel.part(currentTab.part as StatelessPart)