- Issue created by @r_cheh
- Merge request !4Issue #3436460: Fix removing "scrollTop" AJAX command from response. β (Merged) created by r_cheh
- Status changed to Needs review
about 1 year ago 12:43pm 27 March 2024 - Assigned to mandclu
- Status changed to Needs work
12 months ago 1:47pm 21 April 2024 - π¨π¦Canada mandclu
It looks like the renaming of "viewsScrollTop" to "scrollTop" was fairly recent, having been released in Drupal 10.1. I don't see any harm in checking for both, which would keep the module compatible with older versions of Drupal.
- Issue was unassigned.
- Status changed to Needs review
12 months ago 1:53pm 21 April 2024 - π¨π¦Canada mandclu
Here's an approach that checks for both the old and new names, to maintain backwards compatibility.
-
mandclu β
committed 6dbc9b15 on 1.0.x authored by
r_cheh β
Issue #3436460 by r_cheh, mandclu: The "Disable view scroll" option...
-
mandclu β
committed 6dbc9b15 on 1.0.x authored by
r_cheh β
- Status changed to Fixed
9 months ago 8:09am 8 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.