@primer/view-components 0.18.1-rc.671a5d11 → 0.18.1-rc.e20b9eff

package/app/components/primer/dialog_helper.js

@@ -18,6 +18,7 @@ function dialogInvokerButtonHandler(event) {
     // If the user is clicking a valid dialog trigger
     let dialogId = button === null || button === void 0 ? void 0 : button.getAttribute('data-show-dialog-id');
     if (dialogId) {
+        event.stopPropagation();
         const dialog = document.getElementById(dialogId);
         if (dialog instanceof HTMLDialogElement) {
             dialog.showModal();
@@ -25,6 +26,7 @@ function dialogInvokerButtonHandler(event) {
             // If the behaviour is allowed through the dialog will be shown but then
             // quickly hidden- as if it were never shown. This prevents that.
             event.preventDefault();
+            event.stopPropagation();
         }
     }
     dialogId = button.getAttribute('data-close-dialog-id') || button.getAttribute('data-submit-dialog-id');
@@ -78,8 +80,11 @@ export class DialogHelperElement extends HTMLElement {
     handleEvent(event) {
         const target = event.target;
         const dialog = this.dialog;
-        // The click target _must_ be the dialog element itself, and not elements underneath or inside.
-        if (target !== dialog || !(dialog === null || dialog === void 0 ? void 0 : dialog.open))
+        if (!(dialog === null || dialog === void 0 ? void 0 : dialog.open))
+            return;
+        // if the target is inside the dialog, but is not the dialog itself, leave
+        // the dialog open
+        if ((target === null || target === void 0 ? void 0 : target.closest('dialog')) === dialog && target !== dialog)
             return;
         const rect = dialog.getBoundingClientRect();
         const clickWasInsideDialog = rect.top <= event.clientY &&

package/package.json

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

package/static/info_arch.json

@@ -3375,19 +3375,6 @@
             "color-contrast"
           ]
         }
-      },
-      {
-        "preview_path": "primer/alpha/dialog/dialog_inside_overlay",
-        "name": "dialog_inside_overlay",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
       }
     ],
     "subcomponents": [

package/static/previews.json

@@ -3139,19 +3139,6 @@
             "color-contrast"
           ]
         }
-      },
-      {
-        "preview_path": "primer/alpha/dialog/dialog_inside_overlay",
-        "name": "dialog_inside_overlay",
-        "snapshot": "false",
-        "skip_rules": {
-          "wont_fix": [
-            "region"
-          ],
-          "will_fix": [
-            "color-contrast"
-          ]
-        }
       }
     ]
   },