- Issue created by @andre.bonon
- Merge request !95Issue #3468358 by andre.bonon: Text fields are auto-submitted twice when... β (Closed) created by andre.bonon
- Merge request !96Issue #3468358 by andre.bonon: Text fields are auto-submitted twice when... β (Merged) created by andre.bonon
- First commit to issue fork.
-
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 β
- Status changed to Fixed
3 months ago 10:33pm 2 September 2024 - πΊπΈUnited States smustgrave
Change is small enough don't mind committing as is.
Fixed up some merge conflict and eslint issue.
- Status changed to Needs work
3 months ago 1:38pm 5 September 2024 - πΊπΈ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.
- Merge request !105Issue #3468358: UPDATE 1 - Text fields are auto-submitted twice when filters are exposed as block β (Merged) created by recrit
- Status changed to Needs review
3 months ago 2:07pm 7 September 2024 - πΊπΈ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. - πΊπΈUnited States recrit
attached is a backport patch for 6.0.x for anyone that has not updated to 7.0.x yet.
-
smustgrave β
committed cf28ce80 on 7.0.x authored by
recrit β
Issue #3468358: UPDATE 1 - Text fields are auto-submitted twice when...
-
smustgrave β
committed cf28ce80 on 7.0.x authored by
recrit β
- Status changed to Fixed
2 months ago 2:46pm 11 September 2024 Automatically closed - issue fixed for 2 weeks with no activity.