Issue #3343994: Drupal 10 compatiility?

js/masonry.js

@@ -15,17 +15,20 @@
        * (internal) Sub-function for code factoring: actually build masonry.
        */
       function _buildMasonry($container, options) {
+        // Check if masonry already exists
+        let masonryExists = $container.hasClass('masonry');
+      
         // If no masonry find in container: init it...
-        if ($container.findOnce('masonry').length === 0) {
-          $container.once('masonry').addClass('masonry').masonry(options);
+        if (!masonryExists) {
+          $container.addClass('masonry').masonry(options);
           $(window).resize(function () {
-            $container.findOnce('masonry').masonry('bindResize');
+            $container.masonry('bindResize');
           });
         } else {
           // ...otherwise, simply rebuild the layout.
           $container.masonry('reloadItems').masonry('layout');
         }
-
+      
         // Hack to support lazysizes lazy loading.
         if (options.imagesLazyLoad) {
           let lastClassState = false;
@@ -33,13 +36,13 @@
             mutationsList.forEach(function(mutation) {
               if (mutation.type === 'attributes' && mutation.attributeName === 'class') {
                 let currentClassState = mutation.target.classList.contains(options.lazyloadedSelector);
-
+      
                 if (lastClassState !== currentClassState) {
                   lastClassState = currentClassState;
-
+      
                   if (currentClassState) {
                     if (options.imagesFirst) {
-                      $container.imagesLoaded(function () {
+                      imagesLoaded($container[0], function () {
                         $container.masonry('reloadItems').masonry('layout');
                       });
                     } else {
@@ -50,12 +53,15 @@
               }
             });
           });
-
-          $('img.' + options.lazyloadSelector, $container).once('imgloaded').each(function() {
-            observer.observe(this, { attributes: true });
+      
+          $('img.' + options.lazyloadSelector, $container).each(function() {
+            if (!$(this).hasClass('imgloaded')) {
+              $(this).addClass('imgloaded');
+              observer.observe(this, { attributes: true });
+            }
           });
         }
-      }
+      }      
 
       /**
        * Apply Masonry.

[Dominique CLAUSE]

Why not using core once() in replacement of jquery.once() as stated here ? https://www.drupal.org/node/3158256

[Dominique CLAUSE]

Also: should be a const here.

[Dominique CLAUSE]

Same question applies here, why not using once() ?