@aquera/nile-elements 0.1.32-beta-1.6 → 0.1.33-beta-1.0

package/package.json

@@ -3,7 +3,7 @@
   "description": "Webcomponent nile-elements following open-wc recommendations",
   "license": "MIT",
   "author": "nile-elements",
-  "version": "0.1.32-beta-1.6",
+  "version": "0.1.33-beta-1.0",
   "main": "dist/src/index.js",
   "type": "module",
   "module": "dist/src/index.js",

package/src/nile-menu/nile-menu.ts

@@ -67,21 +67,26 @@ export class NileMenu extends NileElement {
   }
 
   private handleKeyDown(event: KeyboardEvent) {
+    const target = event.target as HTMLElement;
+
+    // Only handle keyboard shortcuts if NOT inside input, textarea, or contenteditable
+    if (target.closest('nile-input') || target.tagName === 'INPUT' || target.tagName === 'TEXTAREA' || target.isContentEditable) {
+      return; // Ignore handling space, arrows, etc.
+    }
+
     // Make a selection when pressing enter
     if (event.key === 'Enter') {
       const item = this.getCurrentItem();
       event.preventDefault();
-
-      // Simulate a click to support @click handlers on menu items that also work with the keyboard
       item?.click();
     }
 
-    // Prevent scrolling when space is pressed
+    // Prevent scrolling when space is pressed (ONLY if not typing)
     if (event.key === ' ') {
       event.preventDefault();
     }
 
-    // Move the selection when pressing down or up
+    // Move selection
     if (['ArrowDown', 'ArrowUp', 'Home', 'End'].includes(event.key)) {
       const items = this.getAllItems();
       const activeItem = this.getCurrentItem();
@@ -90,28 +95,20 @@ export class NileMenu extends NileElement {
       if (items.length > 0) {
         event.preventDefault();
 
-        if (event.key === 'ArrowDown') {
-          index++;
-        } else if (event.key === 'ArrowUp') {
-          index--;
-        } else if (event.key === 'Home') {
-          index = 0;
-        } else if (event.key === 'End') {
-          index = items.length - 1;
-        }
+        if (event.key === 'ArrowDown') index++;
+        else if (event.key === 'ArrowUp') index--;
+        else if (event.key === 'Home') index = 0;
+        else if (event.key === 'End') index = items.length - 1;
 
-        if (index < 0) {
-          index = items.length - 1;
-        }
-        if (index > items.length - 1) {
-          index = 0;
-        }
+        if (index < 0) index = items.length - 1;
+        if (index > items.length - 1) index = 0;
 
         this.setCurrentItem(items[index]);
         items[index].focus();
       }
     }
-  }
+}
+
 
   private handleMouseDown(event: MouseEvent) {
     const target = event.target as HTMLElement;