fix(a11y): close button z-index, reduced-motion coverage, aria-expanded sync, info row roles, Enter double-activation

Author: konradmichalik

Resources/Public/Css/FrontendEdit.css

@@ -842,7 +842,7 @@
     position: absolute;
     top: 8px;
     right: 8px;
-    z-index: 2;
+    z-index: 10006;
     width: 32px;
     height: 32px;
     display: flex;
@@ -868,11 +868,17 @@
     .frontend-edit__sidebar--open,
     .frontend-edit__sidebar-backdrop,
     .frontend-edit__sidebar-backdrop--visible,
+    .frontend-edit__sidebar-loader,
+    .frontend-edit__sidebar-iframe,
     .frontend-edit__overlay,
     .frontend-edit__overlay--active,
     .frontend-edit__tooltip,
     .frontend-edit__tooltip--visible {
         transition-duration: 0.01ms !important;
         animation-duration: 0.01ms !important;
     }
+
+    .frontend-edit__sidebar-spinner {
+        animation: none !important;
+    }
 }

Resources/Public/JavaScript/frontend_edit.js

@@ -664,15 +664,14 @@
         if (action.type === 'divider') {
           el.className = 'frontend-edit__divider';
           el.setAttribute('role', 'separator');
+        } else if (action.type === 'info') {
+          el.className = 'frontend-edit__info';
+          el.setAttribute('role', 'presentation');
         } else {
           el.setAttribute('role', 'menuitem');
           el.setAttribute('tabindex', '-1');
         }
 
-        if (action.type === 'info') {
-          el.className = 'frontend-edit__info';
-        }
-
         const safeName = this.escapeClassName(name);
         if (safeName) {
           el.classList.add(safeName);
@@ -714,6 +713,7 @@
           e.preventDefault();
           items[items.length - 1]?.focus();
         } else if (e.key === 'Enter') {
+          e.preventDefault();
           if (document.activeElement && document.activeElement.click) {
             document.activeElement.click();
           }
@@ -972,7 +972,7 @@
         if (toolbar?.contains(e.relatedTarget)) return;
 
         OverlayManager.setActive(targetElement, false);
-        if (dropdown) dropdown.style.display = 'none';
+        if (dropdown) Dropdown.closeAll();
       });
 
       // Keep active when hovering toolbar
@@ -987,7 +987,7 @@
           if (dropdown?.contains(e.relatedTarget)) return;
 
           OverlayManager.setActive(targetElement, false);
-          if (dropdown) dropdown.style.display = 'none';
+          if (dropdown) Dropdown.closeAll();
         });
       }
 
@@ -996,7 +996,7 @@
           if (targetElement.contains(e.relatedTarget)) return;
           if (toolbar?.contains(e.relatedTarget)) return;
 
-          dropdown.style.display = 'none';
+          Dropdown.closeAll();
           OverlayManager.setActive(targetElement, false);
         });
       }