v3.7.0

changelog.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 2025-12-29 - 3.7.0 - feat(dees-contextmenu,dees-appui-tabs,test)
+Prevent double-destruction of context menus, await window layer teardown, update destroyAll behavior, remove tabs content slot, and adjust tests
+
+- Add isDestroying guard in DeesContextmenu.destroy to avoid double-destruction.
+- Await windowLayer.destroy() to ensure the window layer is fully torn down before continuing.
+- Ensure submenu timeouts are cleared and submenu.destroy() is awaited during teardown.
+- Change destroyAll to locate the root/top-level menu and destroy from the root to cascade teardown reliably.
+- Remove the .content wrapper and the <slot> output from dees-appui-tabs (demo updated to render content outside the component) — this is a breaking change to the tabs API (slotted content no longer rendered).
+- Increase test timeout in test.contextmenu-nested-close.browser.ts to 600ms to account for ~300ms windowLayer destruction + ~100ms context menu delay.
+
+## 2025-12-29 - 3.6.1 - fix(readme)
+document new App UI APIs to control main/secondary menu and content tabs visibility
+
+- Added docs for setMainMenuCollapsed(), setMainMenuVisible(), setSecondaryMenuCollapsed(), setSecondaryMenuVisible(), and setContentTabsVisible() programmatic APIs.
+- Included a TypeScript example showing how to hide secondary menu, hide content tabs, and collapse the main menu in a view's onActivate hook.
+
 ## 2025-12-29 - 3.6.0 - feat(dees-appui)
 add visibility toggles for main/secondary menus and ability to show/hide content tabs; expose corresponding setters and appconfig entries
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@design.estate/dees-catalog",
-  "version": "3.6.0",
+  "version": "3.7.0",
   "private": false,
   "description": "A comprehensive library that provides dynamic web components for building sophisticated and modern web applications using JavaScript and TypeScript.",
   "main": "dist_ts_web/index.js",

readme.md

@@ -54,7 +54,7 @@ For developers working on this library, please refer to the [UI Components Playb
 |----------|------------|
 | **Core UI** | [`DeesButton`](#deesbutton), [`DeesButtonExit`](#deesbuttonexit), [`DeesButtonGroup`](#deesbuttongroup), [`DeesBadge`](#deesbadge), [`DeesChips`](#deeschips), [`DeesHeading`](#deesheading), [`DeesHint`](#deeshint), [`DeesIcon`](#deesicon), [`DeesLabel`](#deeslabel), [`DeesPanel`](#deespanel), [`DeesSearchbar`](#deessearchbar), [`DeesSpinner`](#deesspinner), [`DeesToast`](#deestoast), [`DeesWindowcontrols`](#deeswindowcontrols) |
 | **Forms** | [`DeesForm`](#deesform), [`DeesInputText`](#deesinputtext), [`DeesInputCheckbox`](#deesinputcheckbox), [`DeesInputDropdown`](#deesinputdropdown), [`DeesInputRadiogroup`](#deesinputradiogroup), [`DeesInputFileupload`](#deesinputfileupload), [`DeesInputIban`](#deesinputiban), [`DeesInputPhone`](#deesinputphone), [`DeesInputQuantitySelector`](#deesinputquantityselector), [`DeesInputMultitoggle`](#deesinputmultitoggle), [`DeesInputTags`](#deesinputtags), [`DeesInputTypelist`](#deesinputtypelist), [`DeesInputRichtext`](#deesinputrichtext), [`DeesInputWysiwyg`](#deesinputwysiwyg), [`DeesInputDatepicker`](#deesinputdatepicker), [`DeesInputSearchselect`](#deesinputsearchselect), [`DeesFormSubmit`](#deesformsubmit) |
-| **Layout** | [`DeesAppuiBase`](#deesappuibase), [`DeesAppuiMainmenu`](#deesappuimainmenu), [`DeesAppuiSecondarymenu`](#deesappuisecondarymenu), [`DeesAppuiMaincontent`](#deesappuimaincontent), [`DeesAppuiAppbar`](#deesappuiappbar), [`DeesAppuiActivitylog`](#deesappuiactivitylog), [`DeesAppuiProfiledropdown`](#deesappuiprofiledropdown), [`DeesAppuiTabs`](#deesappuitabs), [`DeesAppuiView`](#deesappuiview), [`DeesMobileNavigation`](#deesmobilenavigation), [`DeesDashboardGrid`](#deesdashboardgrid) |
+| **Layout** | [`DeesAppuiBase`](#deesappuibase), [`DeesAppuiMainmenu`](#deesappuimainmenu), [`DeesAppuiSecondarymenu`](#deesappuisecondarymenu), [`DeesAppuiMaincontent`](#deesappuimaincontent), [`DeesAppuiAppbar`](#deesappuiappbar), [`DeesAppuiActivitylog`](#deesappuiactivitylog), [`DeesAppuiProfiledropdown`](#deesappuiprofiledropdown), [`DeesAppuiTabs`](#deesappuitabs), [`DeesMobileNavigation`](#deesmobilenavigation), [`DeesDashboardGrid`](#deesdashboardgrid) |
 | **Data Display** | [`DeesTable`](#deestable), [`DeesDataviewCodebox`](#deesdataviewcodebox), [`DeesDataviewStatusobject`](#deesdataviewstatusobject), [`DeesPdf`](#deespdf), [`DeesStatsGrid`](#deesstatsgrid), [`DeesPagination`](#deespagination) |
 | **Visualization** | [`DeesChartArea`](#deeschartarea), [`DeesChartLog`](#deeschartlog) |
 | **Dialogs & Overlays** | [`DeesModal`](#deesmodal), [`DeesContextmenu`](#deescontextmenu), [`DeesSpeechbubble`](#deesspeechbubble), [`DeesWindowlayer`](#deeswindowlayer) |
@@ -666,9 +666,26 @@ class MyApp extends DeesElement {
 - `navigateToView(viewId, params?)` - Navigate between views
 - `setAppBarMenus()`, `setBreadcrumbs()`, `setUser()` - Control the app bar
 - `setMainMenu()`, `setMainMenuSelection()`, `setMainMenuBadge()` - Control main navigation
-- `setSecondaryMenu()`, `setContentTabs()` - Control view-specific UI
+- `setMainMenuCollapsed()`, `setMainMenuVisible()` - Control main menu visibility
+- `setSecondaryMenu()`, `setSecondaryMenuCollapsed()`, `setSecondaryMenuVisible()` - Control secondary menu
+- `setContentTabs()`, `setContentTabsVisible()` - Control view-specific UI
 - `activityLog.add()`, `activityLog.addMany()`, `activityLog.clear()` - Manage activity entries
 
+**View Visibility Control:**
+```typescript
+// In your view's onActivate hook
+onActivate(context: IViewActivationContext) {
+  // Hide secondary menu for a fullscreen view
+  context.appui.setSecondaryMenuVisible(false);
+
+  // Hide content tabs
+  context.appui.setContentTabsVisible(false);
+
+  // Collapse main menu to give more space
+  context.appui.setMainMenuCollapsed(true);
+}
+```
+
 #### `DeesAppuiMainmenu`
 Main navigation menu component for application-wide navigation.
 

test/test.contextmenu-nested-close.browser.ts

@@ -76,8 +76,8 @@ tap.test('should close all parent menus when clicking action in nested submenu',
   expect(childItem).toBeTruthy();
   childItem!.click();
   
-  // Wait for menus to close
-  await new Promise(resolve => setTimeout(resolve, 200));
+  // Wait for menus to close (windowLayer destruction takes 300ms + context menu 100ms)
+  await new Promise(resolve => setTimeout(resolve, 600));
   
   // Verify action was called
   expect(actionCalled).toEqual(true);

ts_web/00_commitinfo_data.ts

@@ -3,6 +3,6 @@
  */
 export const commitinfo = {
   name: '@design.estate/dees-catalog',
-  version: '3.6.0',
+  version: '3.7.0',
   description: 'A comprehensive library that provides dynamic web components for building sophisticated and modern web applications using JavaScript and TypeScript.'
 }

ts_web/elements/00group-appui/dees-appui-tabs/dees-appui-tabs.demo.ts

@@ -67,9 +67,8 @@ export const demoFunc = () => {
     <div class="demo-container">
       <div class="section">
         <div class="section-title">Horizontal Tabs with Animated Indicator</div>
-        <dees-appui-tabs .tabs=${horizontalTabs}>
-          ${demoContent('Select a tab to see the smooth sliding animation of the indicator. The indicator automatically adjusts its width to match the tab content with minimal padding.')}
-        </dees-appui-tabs>
+        <dees-appui-tabs .tabs=${horizontalTabs}></dees-appui-tabs>
+        ${demoContent('Select a tab to see the smooth sliding animation of the indicator. The indicator automatically adjusts its width to match the tab content with minimal padding.')}
       </div>
 
       <div class="section">
@@ -82,9 +81,8 @@ export const demoFunc = () => {
 
       <div class="section">
         <div class="section-title">Without Indicator</div>
-        <dees-appui-tabs .showTabIndicator=${false} .tabs=${noIndicatorTabs}>
-          ${demoContent('Tabs can also be used without the animated indicator by setting showTabIndicator to false.')}
-        </dees-appui-tabs>
+        <dees-appui-tabs .showTabIndicator=${false} .tabs=${noIndicatorTabs}></dees-appui-tabs>
+        ${demoContent('Tabs can also be used without the animated indicator by setting showTabIndicator to false.')}
       </div>
     </div>
   `;

ts_web/elements/00group-appui/dees-appui-tabs/dees-appui-tabs.ts

@@ -198,19 +198,12 @@ export class DeesAppuiTabs extends DeesElement {
         z-index: 1;
         box-shadow: 0 1px 3px rgba(0, 0, 0, 0.08);
       }
-
-      .content {
-        padding: 32px 24px;
-      }
     `,
   ];
 
   public render(): TemplateResult {
     return html`
       ${this.renderTabsWrapper()}
-      <div class="content">
-        <slot></slot>
-      </div>
     `;
   }
 

ts_web/elements/dees-contextmenu/dees-contextmenu.ts

@@ -134,6 +134,7 @@ export class DeesContextmenu extends DeesElement {
   private submenu: DeesContextmenu | null = null;
   private submenuTimeout: any = null;
   private parentMenu: DeesContextmenu | null = null;
+  private isDestroying: boolean = false;
 
   constructor() {
     super();
@@ -416,27 +417,33 @@ export class DeesContextmenu extends DeesElement {
   }
 
   public async destroy() {
+    // Guard against double-destruction
+    if (this.isDestroying) {
+      return;
+    }
+    this.isDestroying = true;
+
     // Clear timeout
     if (this.submenuTimeout) {
       clearTimeout(this.submenuTimeout);
       this.submenuTimeout = null;
     }
-    
+
     // Destroy submenu first
     if (this.submenu) {
       await this.submenu.destroy();
       this.submenu = null;
     }
-    
+
     // Only destroy window layer if this is not a submenu
     if (this.windowLayer && !this.parentMenu) {
-      this.windowLayer.destroy();
+      await this.windowLayer.destroy();
     }
-    
+
     this.style.opacity = '0';
     this.style.transform = 'scale(0.95) translateY(-10px)';
     await domtools.plugins.smartdelay.delayFor(100);
-    
+
     if (this.parentElement) {
       this.parentElement.removeChild(this);
     }
@@ -446,13 +453,14 @@ export class DeesContextmenu extends DeesElement {
    * Destroys this menu and all parent menus in the chain
    */
   public async destroyAll() {
-    // First destroy parent menus if they exist
-    if (this.parentMenu) {
-      await this.parentMenu.destroyAll();
-    } else {
-      // If we're at the top level, just destroy this menu
-      await this.destroy();
+    // Find the root menu (top-level parent)
+    let rootMenu: DeesContextmenu = this;
+    while (rootMenu.parentMenu) {
+      rootMenu = rootMenu.parentMenu;
     }
+
+    // Destroy from the root - this will cascade through all submenus
+    await rootMenu.destroy();
   }
 }