- ๐บ๐ธUnited States recrit
attached is a backport patch for 6.0.x for anyone that has not updated to 7.0.x yet.
- ๐บ๐ธUnited States recrit
MR 105 created with the following:
- Maintains the same intended solution as the original - find the element with "data-bef-auto-submit-delay" and attach listeners to it.
- Finds the correct parent with "[data-bef-auto-submit-full-form]" even when there are intermediate elements in between "[data-bef-auto-submit]" and "[data-bef-auto-submit-full-form]".
- Changed the "$form" variable to "$needsAutoSubmit" since not all elements found with the selectors are forms. - @recrit opened merge request.
- ๐บ๐ธUnited States recrit
The change can cause JS errors for
$form is null
when there is no delay found on the element or parent. This can happen when the front end theme has an extra element in between the `.views-exposed-form` top level DIV and the child FORM element.- let $form = $(e); + const $delay = $(e).data('bef-auto-submit-delay') ? $(e) : $(e).parent().data('bef-auto-submit-delay'); + const $form= $delay ? $(e).parent() : null; when there is no delay set, the $form is null here causing errors.
- ๐บ๐ธUnited States smustgrave
Change is small enough don't mind committing as is.
Fixed up some merge conflict and eslint issue.
-
smustgrave โ
committed 0cfd93f8 on 7.0.x authored by
andre.bonon โ
Issue #3468358 by andre.bonon: Text fields are auto-submitted twice when...
-
smustgrave โ
committed 0cfd93f8 on 7.0.x authored by
andre.bonon โ
- First commit to issue fork.