@rails/activestorage 7.0.806 → 7.0.1000

package/README.md

@@ -145,7 +145,7 @@ Active Storage, with its included JavaScript library, supports uploading directl
 1. Include the Active Storage JavaScript in your application's JavaScript bundle or reference it directly.
 
     Requiring directly without bundling through the asset pipeline in the application html with autostart:
-    ```html
+    ```erb
     <%= javascript_include_tag "activestorage" %>
     ```
     Requiring via importmap-rails without bundling through the asset pipeline in the application html without autostart as ESM:

package/app/assets/javascripts/activestorage.esm.js

@@ -767,9 +767,9 @@ function start() {
 }
 
 function didClick(event) {
-  const {target: target} = event;
-  if ((target.tagName == "INPUT" || target.tagName == "BUTTON") && target.type == "submit" && target.form) {
-    submitButtonsByForm.set(target.form, target);
+  const button = event.target.closest("button, input");
+  if (button && button.type === "submit" && button.form) {
+    submitButtonsByForm.set(button.form, button);
   }
 }
 

package/app/assets/javascripts/activestorage.js

@@ -750,9 +750,9 @@
     }
   }
   function didClick(event) {
-    const {target: target} = event;
-    if ((target.tagName == "INPUT" || target.tagName == "BUTTON") && target.type == "submit" && target.form) {
-      submitButtonsByForm.set(target.form, target);
+    const button = event.target.closest("button, input");
+    if (button && button.type === "submit" && button.form) {
+      submitButtonsByForm.set(button.form, button);
     }
   }
   function didSubmitForm(event) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rails/activestorage",
-  "version": "7.0.806",
+  "version": "7.0.1000",
   "description": "Attach cloud and local files in Rails applications",
   "module": "app/assets/javascripts/activestorage.esm.js",
   "main": "app/assets/javascripts/activestorage.js",

package/src/ujs.js

@@ -15,9 +15,9 @@ export function start() {
 }
 
 function didClick(event) {
-  const { target } = event
-  if ((target.tagName == "INPUT" || target.tagName == "BUTTON") && target.type == "submit" && target.form) {
-    submitButtonsByForm.set(target.form, target)
+  const button = event.target.closest("button, input")
+  if (button && button.type === "submit" && button.form) {
+    submitButtonsByForm.set(button.form, button)
   }
 }