- 🇵🇱Poland nsavitsky
@abramm This is just wow. First, this patch saved me a lot of time and, second, this patch is just masterclass. Thank you so much. I have updated your patch for 8.x.-6.x and it works just perfect.
- 🇺🇸United States scott_earnest
Tested on:
- PHP 8.0
- Drupal core 9.5.8
- Better Exposed Filters 6.0.3
working great thank you kindly for the patch @abramm
it may be worth noting that in the view settings for the exposed form options (Exposed form options) you need to uncheck "Exclude Textfield" to set the "Delay for textfield autosubmit". The default is 500ms, so it may not seem like the patch is working as expected. You don't have to leave "Exclude Textfield" unchecked, but you do need to toggle to set that option. The view output still seems to respect that setting on the date field even though it is not there in the settings. So this patch works great with "exclude textfield" in either state, but just know to toggle to set the delay.
+1 RTBC for the 6.x branch
- 🇮🇳India krishna mishra
I found by default I was getting $form.data('bef-auto-submit-delay') = 'undefined' for very first auto-submit of form field, if multiple exposed form exists on page. And causing configurable value is not applied for first auto submit and default delay is set 500 instead of configured value.
So passed default delay using drupalSettings.
Here is patch : - 🇺🇦Ukraine abramm Lutsk
Having multiple exposed forms on the page, drupalSettings would be overwritten by the one rendering last; so I'd stick with the value from the data attribute.
- Status changed to Needs work
9 months ago 8:37pm 10 April 2024 - 🇺🇸United States smustgrave
Tested #7 and still seems to be immediately triggering.
- Status changed to Needs review
4 months ago 10:37pm 4 September 2024 - 🇺🇸United States smustgrave
slight different from #7 but how about this? Manually testing it worked.
-
smustgrave →
committed 3cb03fe7 on 7.0.x
Resolve #3299687 "Date input triggers"
-
smustgrave →
committed 3cb03fe7 on 7.0.x
- 🇺🇸United States smustgrave
2nd to last item on the 7.0.2 target list so going to go ahead and merge.
Automatically closed - issue fixed for 2 weeks with no activity.