Add ability to disable AJAX for Views exposed form submit

The patch #7 for 8.4.x works for me, thank you! Using Drupal core 8.4.5.

We should add a test of this new functionality. Also when changing javascript files in core you need to change the es6 versions and then build the .js versions. See https://www.drupal.org/node/2815083 for more on how to do that.

Patch from #11 works for me in 8.5.3.

Testing #11 on 8.5.3. Patch wotks fine for exposed form. But if the exposed form is displayed as a block then the patch doesn't work anymore. Because the data attribute is set then at the block attributes level which wrap the form.

Re-roll of #14 for 8.6.1.

Hello,

Thanks for this, it's a great functionality to have!

This didn't work for me at first, it seems like removing the contents of attachExposedFormAjax function doesn't stop the ajax from firing. However, if I realized if I comment out the line above attachExposedFormAjax where it's being applied to the form, I can successfully stop the ajax. So for now I just did this:

if(!$('[data-views-ajax-submit-disabled]')){
      this.$exposed_form.once('exposed-form').each($.proxy(this.attachExposedFormAjax, this));
    };

It's not the best obviously because it stops all ajax forms even if there is 1 disabled instance of exposed filter on the page, but wanted to share my findings.

I'm using an ajax enabled view with exposed filter as a block on the view page. It includes a text search by search API with autocomplete enabled.

Thanks again, this was very helpful!

Sia

reroll for D8.8

+1 RTBC #18 worked fine on a 8.7.X core Thanks for the hard work!

Thank you so much for this patch.

Working Excellent in Drupal 8.8.2.

I was using Search API Module and an Exposed form is created via Views for search.
For Pagination I was using Views Infinite Scroll module with Ajax.
I was unable to get the keyword from URL since once the search page is reached every request from the exposed form is Ajax driven.
Now, with this patch I could disable the Ajax on the Exposed Form.

Thank You.

1. +++ b/core/modules/views/src/Plugin/views/exposed_form/ExposedFormPluginBase.php
   @@ -47,6 +49,13 @@ public function buildOptionsForm(&$form, FormStateInterface $form_state) {
   +    $form['disable_ajax_submit'] = [
   

   Any way to only show this when ajax is actually enabled?

2. +++ b/core/modules/views/src/Plugin/views/exposed_form/ExposedFormPluginBase.php
   @@ -269,6 +278,14 @@ public function exposedFormAlter(&$form, FormStateInterface $form_state) {
   +    if (!empty($this->options['disable_ajax_submit'])) {
   

   Should we also only add this when ajax is actually enabled?

Per #10, this needs tests. Also this needs an upgrade path because we are adding new config and a test for that upgrade.

#18 worked fine to disable the ajax for the exposed form, but can we able to disable the ajax submit for the facets ?

Example: Am using the facets block with check-box, when i clicked on any facets , ajax loading is happening, i need the page load , so likes to disable the ajax for the facets block but not to the search view. So is there any HOOK or way to handle this?

Note: using the Facet blocks module for the facets block display.

The patch on D9 doesn't work.
It isn't just that doesn't apply, I've reroll-ed to get it applied but it does nothing: even if I configure the options in the view configuration, the views keep to refresh via ajax if I submit the exposed filters form.

Here is a working patch for 9.15

Patch #26 unfortunately didn't provided an interdiff, but seems to work in Drupal 9.1.x

Adding a related issue that changes the same line this patch is changing

It could be important to note that actions should be performed in exact order to make this patch work:
- Add exposed filters
- Enable 'Disable AJAX form submit' option in exposed form settings
- Add pager
- And only as last action enable 'Use AJAX' option
At least for me, it worked only this way.

Patch #26 is not working on 9.3.x so here is a new patch

Tested #33 in combination with infinite scroll, which does require Ajax. Works correct.

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request as a guide.

Did not test

But a never of additional changes needed per #22

Also could use an issue summary update following the default template.

Applied the updated patch with Drupal 10 compatibility. Tested for our project and it works charm.

@n_nelson350, thanks for the D10 patch.
#39 patch applied with the latest Drupal 10.1.1.

Rerolled MR5923 for Drupal Core 10.5.1