v1.5.0 - don't close dialog elements with ESC key

Author: mrwweb

clicky-menus.css

@@ -1,5 +1,5 @@
 /**
- * Clicky Menus v1.4.0
+ * Clicky Menus v1.5.0
  */
 
 /**

clicky-menus.js

@@ -1,5 +1,5 @@
 /**
- * Clicky Menus v1.4.0
+ * Clicky Menus v1.5.0
  */
 
 ( function() {
@@ -67,11 +67,11 @@
 				if ( null !== e.target.closest( 'ul[aria-hidden="false"]' ) ) {
 					currentMenuItem.focus();
 					toggleSubmenu( currentMenuItem );
-					e.stopPropagation();
+					e.preventDefault();
 				// we're on a parent item
 				} else if ( 'true' === e.target.getAttribute( 'aria-expanded' ) ) {
 					toggleSubmenu( currentMenuItem );
-					e.stopPropagation();
+					e.preventDefault();
 				}
 			}
 		}
@@ -105,7 +105,7 @@
 
 					// bind event listener to button
 					button.addEventListener( 'click', toggleOnMenuClick );
-					menu.addEventListener( 'keyup', closeOnEscKey );
+					menu.addEventListener( 'keydown', closeOnEscKey );
 				}
 			} );
 		}

readme.md

@@ -1,6 +1,6 @@
 # Clicky Menus
 
-Version 1.4.0
+Version 1.5.0
 
 Jump to: [About](#about), [Features](#features), [Setup & Configuration](#setup--configuration), [Browser Support](#browser-support) [Changelog](#changelog)
 
@@ -157,6 +157,10 @@ Internet Explorer 11 support is possible if you include polyfills for [`closest`
 
 ## Changelog
 
+### 1.5.0 (May 27, 2025)
+
+- Change `ESC` key event handler to fire on `keydown` instead of `keyup` and use `.preventDefault()` instead of `.stopPropogation()` when focus is inside a submenu or parent of open submenu. This will prevent the `ESC` key from closing `dialog` elements when a clicky-menu is inside one.
+
 ### 1.4.0 (April 29, 2025)
 
 - Further differentiate automatically-generated submenu IDs by including a unique ID for each clicky-menu.