Closes #5449: Regression in scroll behavior of AJAX views with exposed filters

[bberndt-uaz]

Description

Adds a patch to temporarily resolve this Drupal core issue:

• Views with exposed filter and ajax causes unexpected scroll position change

Specifically, the patch reverts the changes from this Drupal core issue made to ajax.js in 10.6.x and 11.x:

• Modernize Drupal.ajaxCommands.scrollTop()

This pull request should also be backported to Quickstart 2.x.

Related issues

• Regression in scroll behavior of AJAX views with exposed filters #5449

How to test

1. To observe the existing issue, visit the Calendar page on the demo site and observe the scrolling behavior when using the filters in the right sidebar.
2. Also, visit the News Views page and observe the same behavior when using the category filter for the Grid View with Filter view.
3. Now visit the Calendar page on the Tugboat review site and confirm that no scrolling occurs when you use the filters.
4. Also, visit the News Views page on the Tugboat site and confirm that no scrolling occurs when you use the category dropdown for the Grid View with Filter view.
5. Test other views with exposed forms as needed. The Finder views are not affected since Quickstart explicitly disables scrolling for those views.

Types of changes

Arizona Quickstart (install profile, custom modules, custom theme)

• Patch release changes
  • Bug fix
  • Accessibility, performance, or security improvement
  • Critical institutional link or brand change
  • Adding experimental module
  • Update experimental module
• Minor release changes
  • New feature
  • Breaking or visual change to existing behavior
  • Upgrade experimental module to stable
  • Enable existing module by default or database update
  • Non-critical brand change
  • New internal API or API improvement with backwards compatibility
  • Risky or disruptive cleanup to comply with coding standards
  • High-risk or disruptive change (requires upgrade path, risks regression, etc.)
• Other or unknown
  • Other or unknown

Drupal core

• Patch release changes
  • Security update
  • Patch level release (non-security bug-fix release)
  • Patch removal that's no longer necessary
• Minor release changes
  • Major or minor level update
• Other or unknown
  • Other or unknown

Drupal contrib projects

• Patch release changes
  • Security update
  • Patch or minor level update
  • Add new module
  • Patch removal that's no longer necessary
• Minor release changes
  • Major level update
• Other or unknown
  • Other or unknown

Checklist

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.
• My change requires release notes.

[djcelaya]

I tested the patch locally and it seems to do the trick! I don't have a strong opinion on where the patch should be housed, however.

[tadean]

  - Applying patches for drupal/core
    ...
    https://gist.githubusercontent.com/bberndt-uaz/7183bc9aee173b19176955ba1cb178f9/raw/c7baa3a68d0325b1f459860831b075cd0819571d/3564586-11.x.patch (Views with exposed filter and ajax causes unexpected scroll position change (3564586))

Views exposed filters no longer seem to visually jump on use in my testing with this patch.