v3.7.1

changelog.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## 2025-12-29 - 3.7.1 - fix(dees-appui-maincontent)
+migrate main content layout to CSS Grid and enable topbar collapse transitions
+
+- Replace absolute positioning with CSS Grid on :host and .maincontainer to enable natural document flow
+- Make .topbar a grid and animate collapse via grid-template-rows; switch :host([notabs]) to grid-template-rows: 0fr instead of display:none to allow transitions
+- Set .maincontainer to display:contents and add min-height:0 on content areas and topbar children to fix overflow/scrolling and flex/grid height issues
+- Remove positional styles (position:absolute/top/left/right/bottom) so content scrolls correctly and layout is more robust
+
+## 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
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@design.estate/dees-catalog",
-  "version": "3.6.1",
+  "version": "3.7.1",
   "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",

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.1',
+  version: '3.7.1',
   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-maincontent/dees-appui-maincontent.ts

@@ -53,41 +53,36 @@ export class DeesAppuiMaincontent extends DeesElement {
       /* TODO: Migrate hardcoded values to --dees-* CSS variables */
       :host {
         color: ${cssManager.bdTheme('#333', '#fff')};
-        display: block;
+        display: grid;
+        grid-template-rows: auto 1fr;
         width: 100%;
         height: 100%;
-        position: relative;
         background: ${cssManager.bdTheme('#ffffff', '#161616')};
       }
+
       .maincontainer {
-        position: absolute;
-        height: 100%;
-        right: 0px;
-        top: 0px;
-        width: 100%;
+        display: contents;
       }
 
       .topbar {
-        position: absolute;
-        width: 100%;
+        display: grid;
+        grid-template-rows: 1fr;
+        overflow: hidden;
         user-select: none;
+        transition: grid-template-rows 0.3s ease;
+      }
+
+      .topbar > * {
+        min-height: 0;
       }
 
       .content-area {
-        position: absolute;
-        top: 60px;
-        left: 0;
-        right: 0;
-        bottom: 0;
         overflow: auto;
+        min-height: 0;
       }
 
       :host([notabs]) .topbar {
-        display: none;
-      }
-
-      :host([notabs]) .content-area {
-        top: 0;
+        grid-template-rows: 0fr;
       }
     `,
   ];

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();
   }
 }