Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
As part of #3094468: [plan] Update core JavaScript (and CSS) dependencies prior to 9.0.0-beta1 we need to update SortableJS to 10.0.1
Proposed resolution
update it
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#8 | interdiff.txt | 84 KB | lauriii |
#4 | interdiff_2-4.txt | 480 bytes | bnjmnm |
#4 | 3101661-4.patch | 84.63 KB | bnjmnm |
#2 | 3101661-2.patch | 84.46 KB | bnjmnm |
Comments
Comment #2
bnjmnmThe differences between 10.0.0 and 10.0.1 are described here: https://github.com/SortableJS/Sortable/releases/tag/1.10.1
Neither of the two changes impact core's use, so this should uneventful.
Comment #3
tedbowIn most other libraries we link to the license on the tag we are using. We should probably also update this to
https://github.com/SortableJS/Sortable/tree/1.10.1#mit-license
Comment #4
bnjmnm#3👌
Comment #5
tedbowDid some manual testing Layout Builder, Media Library and CKEditor which are the 3 places I see sortable being used. they all worked fine with no JS errors 🎉
Comment #6
Gábor HojtsyVersion number typo was misleading.
Comment #8
lauriiiI tried to md5 compare this file to the original file from Github but it didn't match. This was because there was a new line at the end of the file. After removing the new line md5 checksums matched.
Committed 7ef9a12 and pushed to 9.0.x and 8.9.x. Thanks!