dazscript-framework 1.0.23 → 1.0.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dazscript-framework",
-  "version": "1.0.23",
+  "version": "1.0.25",
   "author": "Freddy Diaz",
   "license": "MPL-2.0",
   "description": "",

package/src/dialog/basic-dialog.ts

@@ -24,10 +24,7 @@ export abstract class BasicDialog {
     run(): boolean {
         this.init()
         this.builder.restoreObjectName()
-        this.builder.traceGeometry('beforeExec')
-        let result = this.dialog.exec()
-        this.builder.traceGeometry('afterExec', { result })
-        return result
+        return this.dialog.exec()
     }
 
     ok(): boolean {

package/src/dialog/builders/dialog-builder.ts

@@ -1,6 +1,5 @@
 import { DialogProperties } from '../shared';
 import { WidgetBuilderContext, WidgetsBuilder } from './widgets-builder';
-import { debug } from '@dsf/common/log';
 
 export class DialogBuilder extends WidgetsBuilder {
     public properties: DialogProperties
@@ -17,13 +16,10 @@ export class DialogBuilder extends WidgetsBuilder {
 
     build(setup: () => void): DzBasicDialog {
         this.init()
-        this.traceGeometry('init')
         setup()
         this.restoreObjectName()
-        this.traceGeometry('afterSetup')
         this.resize()
         this.restoreObjectName()
-        this.traceGeometry('afterResize')
         return this.context.dialog
     }
 
@@ -69,64 +65,4 @@ export class DialogBuilder extends WidgetsBuilder {
                 this.context.dialog.setFixedHeight(height);
         }
     }
-
-    traceGeometry(phase: string, extra: { [key: string]: string | number | boolean | undefined } = {}) {
-        let dialog = this.context.dialog
-        let dialogWidget = dialog.getWidget()
-        let objectName = this.read(dialogWidget, 'objectName')
-        let caption = this.read(dialog, 'caption')
-        let sizeHint = this.read(dialogWidget, 'minimumSizeHint')
-        let key = objectName ?? ''
-        let fields: { [key: string]: string | number | boolean | undefined } = {
-            event: 'dialog.geometry',
-            phase,
-            title: this.title,
-            id: this.id ?? '',
-            caption,
-            objectName,
-            windowGeometryKey: key,
-            dialogX: this.read(dialog, 'x'),
-            dialogY: this.read(dialog, 'y'),
-            dialogWidth: this.read(dialog, 'width'),
-            dialogHeight: this.read(dialog, 'height'),
-            widgetX: this.read(dialogWidget, 'x'),
-            widgetY: this.read(dialogWidget, 'y'),
-            widgetWidth: this.read(dialogWidget, 'width'),
-            widgetHeight: this.read(dialogWidget, 'height'),
-            sizeHintWidth: this.read(sizeHint, 'width'),
-            sizeHintHeight: this.read(sizeHint, 'height'),
-            propertyWidth: this.properties.width,
-            propertyHeight: this.properties.height,
-            propertyMaxWidth: this.properties.maxWidth,
-            propertyMaxHeight: this.properties.maxHeight,
-            resizable: this.properties.resizable === true,
-            registrySubKey: key ? `WindowGeometries\\${key}` : ''
-        }
-
-        Object.keys(extra).forEach((name) => fields[name] = extra[name])
-        debug(`[DialogGeometry] ${this.formatGeometryTrace(fields)}`)
-    }
-
-    private read(source: any, field: string): any {
-        if (!source) return undefined
-        try {
-            return source[field]
-        }
-        catch (err) {
-            return undefined
-        }
-    }
-
-    private formatGeometryTrace(fields: { [key: string]: string | number | boolean | undefined }): string {
-        return Object.keys(fields)
-            .filter((name) => fields[name] !== undefined)
-            .map((name) => `${name}=${this.formatGeometryValue(name, fields[name])}`)
-            .join(' ')
-    }
-
-    private formatGeometryValue(name: string, value: string | number | boolean | undefined): string {
-        if (name === 'event' || name === 'phase') return String(value)
-        if (typeof value === 'string') return JSON.stringify(value)
-        return String(value)
-    }
 }

package/src/helpers/custom-action-installer-helper.ts

@@ -12,7 +12,7 @@ import { TreeNode } from '@dsf/lib/tree-node'
 import { promptKeyboardShortcut } from '@dsf/shared/set-keyboard-shortcut'
 import { readFromFile, saveToFile } from './file-helper'
 import { getCanonicalInstallerEntry, toActionKey } from './custom-action-installer-entries'
-import { canResetSetupShortcut, getDisplayedSetupShortcut, resetSetupShortcut, setSetupShortcut, updateShortcutOverrideState } from './custom-action-installer-shortcuts'
+import { canResetSetupShortcut, getDisplayedSetupShortcut, resetSetupShortcut, resetSetupShortcuts, setSetupShortcut, updateShortcutOverrideState } from './custom-action-installer-shortcuts'
 import { getScriptPath } from './script-helper'
 
 type InstallerEntry = {
@@ -397,6 +397,7 @@ class InstallerSelectionDialog extends BasicDialog {
         debug(`[Setup] context menu row resolved item="${this.safeListItemText(contextItem, 1)}" entry="${entry?.action?.text ?? ''}"`)
         const canSetShortcut = Boolean(entry)
         const canResetShortcut = Boolean(entry && canResetSetupShortcut(entry))
+        const canResetAllShortcuts = this.entries.some(canResetSetupShortcut)
 
         const items = [
             {
@@ -413,6 +414,10 @@ class InstallerSelectionDialog extends BasicDialog {
                     this.resetDefaultShortcut(entry)
                 }
             },
+            {
+                text: 'Reset All Shortcuts to Defaults',
+                activated: () => this.resetAllDefaultShortcuts()
+            },
             {},
             { text: 'Check All', activated: () => this.checkVisible(listView, true) },
             { text: 'Uncheck All', activated: () => this.checkVisible(listView, false) }
@@ -421,6 +426,7 @@ class InstallerSelectionDialog extends BasicDialog {
         return new PopupMenuBuilder(this.builder.context).items(...items).build((menu) => {
             menu.setItemEnabled(0, canSetShortcut)
             menu.setItemEnabled(1, canResetShortcut)
+            menu.setItemEnabled(2, canResetAllShortcuts)
         })
     }
 
@@ -498,6 +504,15 @@ class InstallerSelectionDialog extends BasicDialog {
         this.refreshListEvent$.trigger()
     }
 
+    private resetAllDefaultShortcuts() {
+        const resettableEntries = this.entries.filter(canResetSetupShortcut)
+        if (resettableEntries.length === 0) return
+
+        resetSetupShortcuts(resettableEntries)
+        debug(`[Setup] reset ${resettableEntries.length} shortcuts to defaults`)
+        this.refreshListEvent$.trigger()
+    }
+
     private checkVisible(listView: DzListView, onOff: boolean) {
         if (!this.keywords$.value?.trim()) {
             checkAll(listView, onOff)

package/src/helpers/custom-action-installer-shortcuts.test.ts

@@ -2,6 +2,7 @@ import { describe, expect, it } from 'vitest'
 import {
     canResetSetupShortcut,
     getDisplayedSetupShortcut,
+    resetSetupShortcuts,
     resetSetupShortcut,
     setSetupShortcut,
     updateShortcutOverrideState
@@ -80,6 +81,29 @@ describe('setup shortcut state', () => {
         expect(canResetSetupShortcut(entry)).toBe(false)
     })
 
+    it('resets all shortcut entries to their defaults', () => {
+        const installed = shortcutState({
+            installedActionName: 'installed-guid',
+            defaultShortcut: 'Ctrl+P',
+            installedShortcut: 'F12',
+            effectiveShortcut: 'F12',
+            isShortcutOverridden: true
+        })
+        const pending = shortcutState({
+            defaultShortcut: '',
+            effectiveShortcut: 'Ctrl+Alt+K'
+        })
+
+        resetSetupShortcuts([installed, pending])
+
+        expect(installed.installedShortcut).toBe('Ctrl+P')
+        expect(installed.effectiveShortcut).toBe('Ctrl+P')
+        expect(installed.isShortcutOverridden).toBe(false)
+        expect(pending.installedShortcut).toBe('')
+        expect(pending.effectiveShortcut).toBe('')
+        expect(pending.isShortcutOverridden).toBe(false)
+    })
+
     it('displays custom marker when installed shortcut was changed outside setup', () => {
         const entry = shortcutState({
             installedActionName: 'installed-guid',

package/src/helpers/custom-action-installer-shortcuts.ts

@@ -47,3 +47,7 @@ export const resetSetupShortcut = (entry: InstallerShortcutState): void => {
     entry.effectiveShortcut = defaultShortcut
     updateShortcutOverrideState(entry)
 }
+
+export const resetSetupShortcuts = (entries: InstallerShortcutState[]): void => {
+    entries.forEach(resetSetupShortcut)
+}

package/src/dialog/dialog-geometry-trace.test.ts

@@ -1,207 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-import { BasicDialog } from './basic-dialog'
-import { setLogLevel } from '../common/log'
-
-class TestDialog extends BasicDialog {
-    protected build(): void {
-        this.builder.options({ resizable: true })
-    }
-}
-
-class HeaderDialog extends BasicDialog {
-    protected build(): void {
-        this.builder.header({ text: 'Header' }).build()
-        this.builder.options({ resizable: true })
-    }
-}
-
-class NonResizableDefaultSizeDialog extends BasicDialog {
-    protected build(): void { }
-}
-
-class ExplicitFixedHeightDialog extends BasicDialog {
-    protected build(): void {
-        this.builder.options({ height: 35 })
-    }
-}
-
-describe('dialog geometry trace', () => {
-    beforeEach(() => {
-        ; (globalThis as any).App = {
-            debug: vi.fn(),
-            log: vi.fn(),
-            warning: vi.fn(),
-            flushLogBuffer: vi.fn(),
-            statusLine: vi.fn()
-        }
-        ; (globalThis as any).DzBasicDialog = class {
-            caption = ''
-            width = 300
-            height = 200
-            minHeight = 0
-            maxHeight = 0
-            maxWidth = 0
-            sizeGripEnabled = false
-            fixedHeight: number | undefined
-            fixedWidth: number | undefined
-            private widget = {
-                objectName: '',
-                x: 10,
-                y: 20,
-                width: 300,
-                height: 200,
-                minimumSizeHint: { width: 300, height: 200 }
-            }
-
-            getWidget() {
-                return this.widget
-            }
-
-            inherits(typeName: string) {
-                return typeName === 'DzWidget'
-            }
-
-            addLayout() { }
-
-            exec() {
-                if (this.sizeGripEnabled) {
-                    this.width = 640
-                    this.height = 480
-                    this.widget.width = 640
-                    this.widget.height = 480
-                    return true
-                }
-
-                this.widget.minimumSizeHint = { width: 196, height: 76 }
-                this.width = this.fixedWidth ?? this.widget.minimumSizeHint.width
-                this.height = this.fixedHeight ?? this.widget.minimumSizeHint.height
-                this.widget.width = this.width
-                this.widget.height = this.height
-                return true
-            }
-
-            setFixedHeight(value: number) {
-                this.fixedHeight = value
-                this.height = value
-            }
-
-            setFixedWidth(value: number) {
-                this.fixedWidth = value
-                this.width = value
-            }
-        }
-        const Layout = class {
-            direction = 0
-            constructor(public parent: unknown) { }
-            inherits(typeName: string) {
-                return typeName === 'DzLayout'
-            }
-            addWidget() { }
-        }
-        class Widget {
-            private _name = ''
-            height = 30
-            width = 100
-            minHeight = 0
-            maxHeight = 0
-            toolTip = ''
-            whatsThis = ''
-            constructor(private parent: any) { }
-            set name(value: string) {
-                this._name = value
-                if (this.parent?.getWidget) this.parent.getWidget().objectName = value
-            }
-            get name() {
-                return this._name
-            }
-            getWidget() {
-                return this
-            }
-            inherits(typeName: string) {
-                return typeName === 'DzWidget'
-            }
-            setFixedHeight(value: number) {
-                this.height = value
-            }
-            setFixedWidth(value: number) {
-                this.width = value
-            }
-            show() { }
-            hide() { }
-        }
-        ; (globalThis as any).DzVBoxLayout = Layout
-        ; (globalThis as any).DzHBoxLayout = Layout
-        ; (globalThis as any).DzWidget = Widget
-        ; (globalThis as any).DzLabel = Widget
-        ; (globalThis as any).DzTextBrowser = class extends Widget {
-            html = ''
-            readOnly = false
-            lineWrapMode = 0
-            wordWrapMode = 0
-        }
-        ; (globalThis as any).DzTextEdit = { WidgetWidth: 1, WordWrap: 2 }
-        ; (globalThis as any).DzSplitter = { Vertical: 1, Horizontal: 2 }
-        setLogLevel('trace')
-    })
-
-    it('logs dialog objectName and geometry before and after exec', () => {
-        new TestDialog('Power Loader', 'Vholf3D Power Loader').run()
-
-        const messages = (globalThis as any).App.debug.mock.calls.map((call: unknown[]) => String(call[0]))
-
-        expect(messages.some((message: string) =>
-            message.includes('event=dialog.geometry phase=init') &&
-            message.includes('objectName="Vholf3DPowerLoaderDlg"')
-        )).toBe(true)
-        expect(messages.some((message: string) =>
-            message.includes('event=dialog.geometry phase=beforeExec') &&
-            message.includes('dialogWidth=300')
-        )).toBe(true)
-        expect(messages.some((message: string) =>
-            message.includes('event=dialog.geometry phase=afterExec') &&
-            message.includes('dialogWidth=640') &&
-            message.includes('result=true')
-        )).toBe(true)
-    })
-
-    it('keeps the DAZ geometry objectName stable after building a header widget', () => {
-        new HeaderDialog('Power Menu - Active Tool', 'Vholf3D Power Menu - Active Tool').run()
-
-        const messages = (globalThis as any).App.debug.mock.calls.map((call: unknown[]) => String(call[0]))
-
-        expect(messages.some((message: string) =>
-            message.includes('event=dialog.geometry phase=afterSetup') &&
-            message.includes('objectName="Vholf3DPowerMenu-ActiveToolDlg"')
-        )).toBe(true)
-        expect(messages.some((message: string) =>
-            message.includes('event=dialog.geometry phase=beforeExec') &&
-            message.includes('objectName="Vholf3DPowerMenu-ActiveToolDlg"')
-        )).toBe(true)
-    })
-
-    it('does not freeze a non-resizable dialog to an early implicit size hint', () => {
-        new NonResizableDefaultSizeDialog('Store Active Camera').run()
-
-        const messages = (globalThis as any).App.debug.mock.calls.map((call: unknown[]) => String(call[0]))
-
-        expect(messages.some((message: string) =>
-            message.includes('event=dialog.geometry phase=afterExec') &&
-            message.includes('objectName="StoreActiveCameraDlg"') &&
-            message.includes('dialogHeight=76') &&
-            message.includes('sizeHintHeight=76')
-        )).toBe(true)
-    })
-
-    it('keeps explicit non-resizable dialog height fixed', () => {
-        new ExplicitFixedHeightDialog('Explicit Height').run()
-
-        const messages = (globalThis as any).App.debug.mock.calls.map((call: unknown[]) => String(call[0]))
-
-        expect(messages.some((message: string) =>
-            message.includes('event=dialog.geometry phase=afterExec') &&
-            message.includes('objectName="ExplicitHeightDlg"') &&
-            message.includes('dialogHeight=35') &&
-            message.includes('sizeHintHeight=76')
-        )).toBe(true)
-    })
-})