Changed chained parent calls to .closest as suggested

ctrlo · Oct 11, 2023 · e78679b · e78679b
1 parent eaf2379
commit e78679b
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/frontend/components/button/lib/component.js b/src/frontend/components/button/lib/component.js
@@ -189,9 +189,8 @@ class ButtonComponent extends Component {
     const $button = $(ev.target).closest('button');
     const $form = $button.closest("form")
     const $requiredHiddenRecordDependentFields = $form.find(".form-group[data-has-dependency='1'][style*='display: none'] *[aria-required]")
-    //This is awful, but if I use a find on all parents to find the .modal-body, the "normal" submit button crashes the browser with an (almost) infinite loop
-    const $parent = $button.parent().parent().parent().parent().parent().parent().parent().parent().parent().parent().parent();
-
+    const $parent = $button.closest('.modal-body')
+
     if (!this.requiredHiddenRecordDependentFieldsCleared) {
       ev.preventDefault()