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

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

@@ -116,19 +116,6 @@ 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;
@@ -148,6 +135,15 @@ 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/app/components/primer/beta/button_group.css

@@ -1 +1 @@
-.ButtonGroup{display:inline-flex}.ButtonGroup .Button{border-radius:0;margin-inline-end:-1px;position:relative}.ButtonGroup .Button:active,.ButtonGroup .Button:focus,.ButtonGroup .Button:hover{z-index:1}.ButtonGroup>.Button-withTooltip:first-child .Button,.ButtonGroup>:first-child{border-bottom-left-radius:var(--borderRadius-medium,6px);border-top-left-radius:var(--borderRadius-medium,6px)}.ButtonGroup>.Button-withTooltip:last-child .Button,.ButtonGroup>:last-child{border-bottom-right-radius:var(--borderRadius-medium,6px);border-top-right-radius:var(--borderRadius-medium,6px)}
+.ButtonGroup{display:inline-flex}.ButtonGroup button{border-radius:0;margin-inline-end:-1px;position:relative}.ButtonGroup button:active,.ButtonGroup button:focus,.ButtonGroup button:hover{z-index:1}.ButtonGroup>.Button-withTooltip:first-child button,.ButtonGroup>:first-child{border-bottom-left-radius:var(--borderRadius-medium,6px);border-top-left-radius:var(--borderRadius-medium,6px)}.ButtonGroup>.Button-withTooltip:last-child button,.ButtonGroup>:last-child{border-bottom-right-radius:var(--borderRadius-medium,6px);border-top-right-radius:var(--borderRadius-medium,6px)}

package/app/components/primer/beta/button_group.css.json

@@ -2,13 +2,13 @@
   "name": "beta/button_group",
   "selectors": [
     ".ButtonGroup",
-    ".ButtonGroup .Button",
-    ".ButtonGroup .Button:active",
-    ".ButtonGroup .Button:focus",
-    ".ButtonGroup .Button:hover",
-    ".ButtonGroup>.Button-withTooltip:first-child .Button",
+    ".ButtonGroup button",
+    ".ButtonGroup button:active",
+    ".ButtonGroup button:focus",
+    ".ButtonGroup button:hover",
+    ".ButtonGroup>.Button-withTooltip:first-child button",
     ".ButtonGroup>:first-child",
-    ".ButtonGroup>.Button-withTooltip:last-child .Button",
+    ".ButtonGroup>.Button-withTooltip:last-child button",
     ".ButtonGroup>:last-child"
   ]
 }

package/package.json

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