The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- @murz opened merge request.
- last update
over 1 year ago 29,453 pass, 2 fail - last update
over 1 year ago 29,453 pass, 2 fail - last update
over 1 year ago 29,454 pass - Merge request !4476Issue #3300426: Views viewsScrollTop (ScrollTopCommand AJAX command) doesn't work inside Drupal Off canvas dialogs → (Open) created by murz
- last update
over 1 year ago 29,882 pass - Status changed to Needs review
over 1 year ago 4:25am 26 July 2023 - 🇦🇲Armenia murz Yerevan, Armenia
In Drupal 10.1 the scrollTop was been reworked by https://www.drupal.org/project/drupal/issues/3300481 ✨ Move ScrollTopCommand views Ajax command to the Drupal Core Fixed and also added a wrapper to Off Canvas dialog.
So, I've reworked the fix for Drupal 11.x - now all works well. Please review. - last update
over 1 year ago 29,877 pass, 1 fail - last update
over 1 year ago 29,882 pass - last update
over 1 year ago 29,882 pass - 🇦🇲Armenia murz Yerevan, Armenia
Attaching a static patch file with those changes, to make it possible to use in composer until 🐛 GitLab Merge Requests Unable to Generate Incremental Patch Files Active is resolved.
- Status changed to RTBC
over 1 year ago 4:40pm 7 August 2023 - last update
over 1 year ago 29,954 pass - last update
over 1 year ago 29,959 pass - last update
over 1 year ago 29,955 pass, 2 fail The last submitted patch, 24: drupal-views-scroll-top-in-dialogs-3300426-24.patch, failed testing. View results →
- 🇦🇲Armenia murz Yerevan, Armenia
Hide the patch and revert back to "Reviewed & tested by the community", cuz the patch is outdated.
- last update
about 1 year ago 30,372 pass - Status changed to Needs work
about 1 year ago 8:47am 4 October 2023 - 🇫🇮Finland lauriii Finland
Left one comment in the MR. The fix itself makes sense but the code could be made a bit easier to read. I had to use debugger to understand what is going on.
- last update
about 1 year ago Custom Commands Failed - last update
about 1 year ago 30,378 pass - Status changed to Needs review
about 1 year ago 5:41am 6 October 2023 - Status changed to RTBC
about 1 year ago 5:50pm 6 October 2023 - 🇺🇸United States smustgrave
Additional comments requested seem fine.
I tried following the git history and made it all the way to https://github.com/drupal/drupal/commit/a626abb24faa51ac140f73779a89e1ad... where 7.x-3.x was moved into core. So -10 was there for a very very long time
- last update
about 1 year ago 30,383 pass - last update
about 1 year ago 30,385 pass - last update
about 1 year ago 30,395 pass - last update
about 1 year ago 30,398 pass - last update
about 1 year ago 30,412 pass - Status changed to Needs work
about 1 year ago 4:03pm 17 October 2023 The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇨🇦Canada b_sharpe
Related? ✨ Modernize Drupal.ajaxCommands.scrollTop() Active
- 🇺🇸United States smustgrave
smustgrave → changed the visibility of the branch 3300426-D10 to hidden.
- First commit to issue fork.
- 🇮🇳India mrinalini9 New Delhi
Fixed PHP Coding Standard pipeline issues, please review it.
Thanks! The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.