Make Views AJAX scroll to top optional

#4 patch works for me on 10.1.1. This is a small but necessary option.

This is great - thank you @skyredwang

That's cool, thanks @skyreddwang,
Tested on D9.5.8 /PHP 8.1/MariaDB 10.3

Hi @skyredwang
I applied the patch #4. Works well.
Thanks

I think it's reviewed and tested by community, so I think we can change status of this issue

Thanks for working on this, the patch should be moved to a Merge Request so that tests can run. Talking about tests, a test should be added per #7 to make sure we don't break that feature down the line.

Thanks!

sahana _n → made their first commit to this issue’s fork.

Hi

I tried to reproduce the issue issue is replaceable in 11. x.

And I created the MR for 11. x and tests are passed.

RTBC++

Please let me know if I missed anything happy to take suggestions.

Previously tagged for tests which are still needed.

Did not review.

Added unit test to prove new functionality.

Attaching patch simply for composer.

So I imagine with this config change we will need a schema update for the new setting.

Existing view config will have to probably be updated.

Also imagine there will have to be a post_update hook to set the new key for existing views (should be batch)

A unit test is in place. Test-only test fails as it should. Removing 'Needs tests' tag. If further test coverage is needed, please update.