Allow forms inside ResourceItems

baoagency · Aug 16, 2021 · 8e69fd4 · 8e69fd4
1 parent c858c91
commit 8e69fd4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/app/assets/javascripts/polaris.js b/app/assets/javascripts/polaris.js
@@ -16,12 +16,12 @@ function _defineProperty(obj, key, value) {
 
 class _class$2 extends Controller {
   open(event) {
-    if (this.hasLinkTarget && this.targetIsNotLink(event.target)) {
+    if (this.hasLinkTarget && this.targetNotClickable(event.target)) {
       this.linkTarget.click();
     }
   }
-  targetIsNotLink(element) {
-    return !element.closest("a") && !element.closest("button");
+  targetNotClickable(element) {
+    return !element.closest("a") && !element.closest("button") && element.nodeName !== "INPUT";
   }
 }
 

diff --git a/app/javascript/polaris/resource_item_controller.js b/app/javascript/polaris/resource_item_controller.js
@@ -4,12 +4,12 @@ export default class extends Controller {
   static targets = ['link']
 
   open(event) {
-    if (this.hasLinkTarget && this.targetIsNotLink(event.target)) {
+    if (this.hasLinkTarget && this.targetNotClickable(event.target)) {
       this.linkTarget.click()
     }
   }
 
-  targetIsNotLink(element) {
-    return !element.closest('a') && !element.closest('button')
+  targetNotClickable(element) {
+    return !element.closest('a') && !element.closest('button') && element.nodeName !== "INPUT"
   }
 }