@rettangoli/ui 0.1.8 → 0.1.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rettangoli/ui",
-  "version": "0.1.8",
+  "version": "0.1.10",
   "description": "A UI component library for building web interfaces.",
   "main": "dist/rettangoli-esm.min.js",
   "type": "module",

package/src/components/select/select.handlers.js

@@ -38,6 +38,7 @@ export const handleOnUpdate = (deps, payload) => {
 export const handleButtonClick = (deps, payload) => {
   const { store, render, getRefIds, props } = deps;
   const event = payload._event;
+  event.stopPropagation();
 
   const button = getRefIds()['select-button'].elm;
 
@@ -73,6 +74,7 @@ export const handleClickOptionsPopoverOverlay = (deps, payload) => {
 export const handleOptionClick = (deps, payload) => {
   const { render, dispatchEvent, props, store } = deps;
   const event = payload._event;
+  event.stopPropagation();
   const id = event.currentTarget.id.replace('option-', '');
 
   const option = props.options[id];
@@ -145,7 +147,8 @@ export const handleClearClick = (deps, payload) => {
 
 export const handleAddOptionClick = (deps, payload) => {
   const { store, render, dispatchEvent } = deps;
-
+  const { _event: event } = payload;
+  event.stopPropagation();
   // Close the popover
   store.closeOptionsPopover();
 

package/src/components/sidebar/sidebar.store.js

@@ -12,7 +12,7 @@ function flattenItems(items, selectedItemId = null) {
   for (const item of items) {
     const itemId = item.id || item.href || item.path;
     const isSelected = selectedItemId === itemId;
-    
+
     // Add the parent item if it's not just a group label
     result.push({
       id: itemId,
@@ -31,7 +31,7 @@ function flattenItems(items, selectedItemId = null) {
       for (const subItem of item.items) {
         const subItemId = subItem.id || subItem.href || subItem.path;
         const isSubSelected = selectedItemId === subItemId;
-        
+
         result.push({
           id: subItemId,
           title: subItem.title,
@@ -139,4 +139,4 @@ export const selectItem = ({ state, props, attrs }, id) => {
 
 export const setState = (state) => {
   // State management if needed
-}
+}

package/src/primitives/popover.js

@@ -11,36 +11,41 @@ class RettangoliPopoverElement extends HTMLElement {
           display: contents;
         }
 
-        .popover-overlay {
+        dialog {
+          padding: 0;
+          border: none;
+          background: transparent;
+          margin: 0;
+          overflow: visible;
+          color: inherit;
+          scrollbar-width: none;
+          outline: none;
           position: fixed;
           top: 0;
           left: 0;
           width: 100vw;
           height: 100vh;
-          z-index: 999;
-          display: none;
+          /* Prevent dialog from being focused */
+          pointer-events: none;
+        }
+
+        dialog::backdrop {
+          background-color: transparent;
+          /* Allow backdrop to receive clicks */
+          pointer-events: auto;
         }
 
         .popover-container {
           position: fixed;
           z-index: 1000;
-          display: none;
           outline: none;
-        }
-
-        :host([open]:not([no-overlay])) .popover-overlay {
-          display: block;
+          pointer-events: auto;
         }
 
         :host([open]) .popover-container {
           display: block;
           visibility: hidden;
         }
-        
-        /* For no-overlay mode, make the container non-interactive */
-        :host([no-overlay]) .popover-container {
-          pointer-events: none;
-        }
 
         :host([open][positioned]) .popover-container {
           visibility: visible;
@@ -65,20 +70,42 @@ class RettangoliPopoverElement extends HTMLElement {
     this.shadow = this.attachShadow({ mode: "open" });
     this.shadow.adoptedStyleSheets = [RettangoliPopoverElement.styleSheet];
 
-    // Create overlay
-    this._popoverOverlay = document.createElement('div');
-    this._popoverOverlay.className = 'popover-overlay';
-    this.shadow.appendChild(this._popoverOverlay);
+    // Create dialog element
+    this._dialogElement = document.createElement('dialog');
+    this.shadow.appendChild(this._dialogElement);
+
+    // Handle dialog backdrop clicks to close popover
+    this._dialogElement.addEventListener('click', (e) => {
+      e.stopPropagation();
+      // Close on backdrop clicks (when clicking outside the popover content)
+      const path = e.composedPath();
+      const clickedOnBackdrop = path[0] === this._dialogElement ||
+        (path[0].nodeName === 'DIALOG' && path[0] === this._dialogElement);
+
+      if (clickedOnBackdrop) {
+        this.dispatchEvent(new CustomEvent('close', {
+          detail: {}
+        }));
+      }
+    });
 
-    // Handle overlay clicks to close popover
-    this._popoverOverlay.addEventListener('click', () => {
-      this.dispatchEvent(new CustomEvent('close', {
-        detail: {}
-      }));
+    // Handle right-click on backdrop to close popover
+    this._dialogElement.addEventListener('contextmenu', (e) => {
+      // Close on backdrop right-clicks
+      const path = e.composedPath();
+      const clickedOnBackdrop = path[0] === this._dialogElement ||
+        (path[0].nodeName === 'DIALOG' && path[0] === this._dialogElement);
+
+      if (clickedOnBackdrop) {
+        e.preventDefault();
+        this.dispatchEvent(new CustomEvent('close', {
+          detail: {}
+        }));
+      }
     });
 
-    // Handle right-click on overlay to close popover
-    this._popoverOverlay.addEventListener('contextmenu', (e) => {
+    // Handle ESC key - prevent native close and emit custom event
+    this._dialogElement.addEventListener('cancel', (e) => {
       e.preventDefault();
       this.dispatchEvent(new CustomEvent('close', {
         detail: {}
@@ -88,20 +115,17 @@ class RettangoliPopoverElement extends HTMLElement {
     // Create popover container
     this._popoverContainer = document.createElement('div');
     this._popoverContainer.className = 'popover-container';
-    this.shadow.appendChild(this._popoverContainer);
+    this._dialogElement.appendChild(this._popoverContainer);
 
     // Store reference for content slot
     this._slotElement = null;
 
     // Track if we're open
     this._isOpen = false;
-
-    // Bind event handlers
-    this._handleEscKey = this._handleEscKey.bind(this);
   }
 
   static get observedAttributes() {
-    return ["open", "x", "y", "placement", "no-overlay"];
+    return ["open", "x", "y", "placement"];
   }
 
   connectedCallback() {
@@ -112,14 +136,19 @@ class RettangoliPopoverElement extends HTMLElement {
   }
 
   disconnectedCallback() {
-    // Clean up event listeners
-    this._removeGlobalListeners();
+    // Clean up dialog if it's open
+    if (this._isOpen && this._dialogElement.open) {
+      this._dialogElement.close();
+    }
   }
 
   attributeChangedCallback(name, oldValue, newValue) {
     if (name === 'open') {
       if (newValue !== null && !this._isOpen) {
-        this._show();
+        // Only show if element is connected to DOM
+        if (this.isConnected) {
+          this._show();
+        }
       } else if (newValue === null && this._isOpen) {
         this._hide();
       }
@@ -138,8 +167,20 @@ class RettangoliPopoverElement extends HTMLElement {
       }
 
       this._isOpen = true;
-      this._updatePosition();
-      this._addGlobalListeners();
+
+      // Show the dialog using setTimeout to ensure it's in the DOM
+      if (!this._dialogElement.open) {
+        setTimeout(() => {
+          if (this._dialogElement && !this._dialogElement.open) {
+            this._dialogElement.showModal();
+          }
+        }, 0);
+      }
+
+      // Update position after dialog is shown
+      requestAnimationFrame(() => {
+        this._updatePosition();
+      });
     }
   }
 
@@ -147,13 +188,16 @@ class RettangoliPopoverElement extends HTMLElement {
     if (this._isOpen) {
       this._isOpen = false;
 
+      // Close the dialog
+      if (this._dialogElement.open) {
+        this._dialogElement.close();
+      }
+
       // Remove slot to unmount content
       if (this._slotElement) {
         this._popoverContainer.removeChild(this._slotElement);
         this._slotElement = null;
       }
-
-      this._removeGlobalListeners();
     }
   }
 
@@ -246,25 +290,6 @@ class RettangoliPopoverElement extends HTMLElement {
     return { left, top };
   }
 
-  _addGlobalListeners() {
-    // Use setTimeout to avoid immediate triggering
-    setTimeout(() => {
-      document.addEventListener('keydown', this._handleEscKey);
-    }, 0);
-  }
-
-  _removeGlobalListeners() {
-    document.removeEventListener('keydown', this._handleEscKey);
-  }
-
-
-  _handleEscKey(e) {
-    if (e.key === 'Escape') {
-      this.dispatchEvent(new CustomEvent('close', {
-        detail: {}
-      }));
-    }
-  }
 
   // Expose popover container for advanced usage
   get popover() {