@primer/view-components 0.2.0-rc.e70004c0 → 0.2.0-rc.e85603cb

package/app/components/primer/alpha/action_menu/action_menu_element.js

@@ -116,6 +116,19 @@ let ActionMenuElement = class ActionMenuElement extends HTMLElement {
                 __classPrivateFieldGet(this, _ActionMenuElement_instances, "a", _ActionMenuElement_firstItem_get).focus();
         }
         else if (__classPrivateFieldGet(this, _ActionMenuElement_instances, "m", _ActionMenuElement_isActivationKeydown).call(this, event) || (event instanceof MouseEvent && event.type === 'click')) {
+            // Hide popover after current event loop to prevent changes in focus from
+            // altering the target of the event. Not doing this specifically affects
+            // <a> tags. It causes the event to be sent to the currently focused element
+            // instead of the anchor, which effectively prevents navigation, i.e. it
+            // appears as if hitting enter does nothing. Curiously, clicking instead
+            // works fine.
+            if (this.selectVariant !== 'multiple') {
+                setTimeout(() => { var _a; return (_a = this.popoverElement) === null || _a === void 0 ? void 0 : _a.hidePopover(); });
+            }
+            // The rest of the code below deals with single/multiple selection behavior, and should not
+            // interfere with events fired by menu items whose behavior is specified outside the library.
+            if (this.selectVariant !== 'multiple' && this.selectVariant !== 'single')
+                return;
             const item = event.target.closest(menuItemSelectors.join(','));
             if (!item)
                 return;
@@ -135,15 +148,6 @@ let ActionMenuElement = class ActionMenuElement extends HTMLElement {
                 // prevent buttons from being clicked twice
                 event.preventDefault();
             }
-            // Hide popover after current event loop to prevent changes in focus from
-            // altering the target of the event. Not doing this specifically affects
-            // <a> tags. It causes the event to be sent to the currently focused element
-            // instead of the anchor, which effectively prevents navigation, i.e. it
-            // appears as if hitting enter does nothing. Curiously, clicking instead
-            // works fine.
-            if (this.selectVariant !== 'multiple') {
-                setTimeout(() => { var _a; return (_a = this.popoverElement) === null || _a === void 0 ? void 0 : _a.hidePopover(); });
-            }
         }
     }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@primer/view-components",
-  "version": "0.2.0-rc.e70004c0",
+  "version": "0.2.0-rc.e85603cb",
   "description": "ViewComponents for the Primer Design System",
   "main": "app/assets/javascripts/primer_view_components.js",
   "module": "app/components/primer/primer.js",