Skip to content

Commit 0ad26f7

Browse files
Merge pull request rails#48290 from marckohlbrugge/patch-1
Support nested elements inside <button>
2 parents e1a0f6f + a1504f4 commit 0ad26f7

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

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

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

activestorage/app/assets/javascripts/activestorage.js

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

activestorage/app/javascript/activestorage/ujs.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ export function start() {
1515
}
1616

1717
function didClick(event) {
18-
const { target } = event
19-
if ((target.tagName == "INPUT" || target.tagName == "BUTTON") && target.type == "submit" && target.form) {
20-
submitButtonsByForm.set(target.form, target)
18+
const submitButton = event.target.closest(":is(button, input)[type='submit']")
19+
if (submitButton && submitButton.form) {
20+
submitButtonsByForm.set(submitButton.form, submitButton)
2121
}
2222
}
2323

0 commit comments

Comments
 (0)