Hey BMPanda

The module is awesome and works perfectly out of the box. Is there any way that it can play nicely with views infinite scroll? drupal.org/project/views_infinite_scroll. This would be the perfect combo to create sites like pinterest for example.

Again, thanks for the module.

UPDATE: This issue was first filed with the Masonry module, that explains why it seems to refer to itself (views infinite scroll).

Related Issue in Masonry #1808018: Make work with Views Infinite Scroll

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrmeech’s picture

I agree that making this play nice with vews_infinite_scroll would be awesome! :)

BWPanda’s picture

Title: Infinite Scroll » Support Masonry module
Project: Masonry API » Views Infinite Scroll
Category: feature » task
Status: Active » Needs review
FileSize
1.47 KB

I was waiting for this feature to be requested ;)

I tried using both Masonry and Views Infinite Scroll (VIS) on my test site and found that the main reason they didn't work together was because VIS specifies the plugin names it works with (and obviously hasn't added the Masonry Views plugin to that list).

However, as Masonry simply builds on the default 'Unformatted' plugin, integration can be achieved by simply changing the VIS plugins so that the 'default' plugin becomes the default option when no other more-specific plugins are found. I've attached a patch for VIS that makes this possible.

However, even with the attached patch the two modules still don't work together properly as Masonry needs to re-organise the newly-added items to appear below the existing items somehow. I tried to get this to work, but couldn't. I believe some code needs to be added to the VIS javascript file somewhere...

I'm therefore moving this issue to the VIS issue queue so as to:
- Get the attached patch committed to support Masonry
- Work out how to re-load Masonry when new items are added

Feel free to move this back to Masonry (or open a new issue) if it turns out that I need to patch Masonry to get this to work, but from what I've tested I think it's the VIS maintainer that needs to address this.

rickharington’s picture

Category: task » feature
Status: Needs review » Active

BWPanda

I'm not so sure that it needs to be fixed. I've just implemented your patch and it works like a charm! Tested in Firefox, Chrome and IE!

Man you are a legend byond belief!

Thanks

dwkitchen’s picture

Just had a quick try of this patch, I may have had the same issue.

The after the initial page load it looks good, scrolling down the extra content is loaded by VIS it is placed on top of the original content. On the second load of extra content it is placed below the the first set of VIS content. Subsequent VIS loaded content is placed correctly.

This may be a theme based issue.

funature’s picture

there will be an overlapping issue if the items are with image. A temporary solution is http://drupal.org/node/970870#comment-5945292. Like @Carlitus said, maybe a not elegant method.

Put after the 47 line of views_infinite_scroll.js

$(content_selector).masonry( 'reload' );

Before

load: function() {
                $('div#views_infinite_scroll-ajax-loader').remove();
                Drupal.attachBehaviors(this);
              }

After

load: function() {
           $('div#views_infinite_scroll-ajax-loader').remove();
                Drupal.attachBehaviors(this);
                $(content_selector).masonry( 'reload' );
              }
I'm sure this is dirty code, i think it must be a Drupal behaviours job and only when the masonry is active, maybe someone can improve it?

i hope the maintainer could make the change for the both masonry module.

dwkitchen’s picture

This worked to stop the first page load from overlapping. However now some subsequent items are over lapping and once the first page load happens all the other pages load as well.

funature’s picture

you are right, i'm also facing this issue. do you have any solution?

mErilainen’s picture

I would like to make this work with Views Accordion (http://drupal.org/project/views_accordion) and the patch makes it possible to load the infinite scroll JS, but after loading more results they don't get inside the accordions because the views accordion is not doing anything for the new results.

I have also applied the patch from #981368: Support views grouping to make this module work with grouping. It would be awesome to make the grouping work and also Masonry, would open a lot of nice use cases for this module!

funature’s picture

maybe this post is helpful, http://stackoverflow.com/questions/9766515/imagesloaded-method-not-worki...
maybe it should be done similar like this:

// trigger Masonry as a callback
          function( newElements ) {
            // hide new items while they are loading
            var $newElems = $( newElements ).css({ opacity: 0 });
            // ensure that images load before adding to masonry layout
            $newElems.imagesLoaded(function(){
              // show elems now they're ready
              $newElems.animate({ opacity: 1 });
              $container.masonry( 'appended', $newElems, true ); 
            });
          }

because i'm not a coder and know almost nothing about jquery, so it will be great if somebody can fix this!

logaritmisk’s picture

I would suggest to do this:

load: function() {
  $('div#views_infinite_scroll-ajax-loader').remove();
  Drupal.attachBehaviors(this);
  $(content_selector).trigger( 'change' );
}

And patch masonry_views.module on line 73 with:

}).bind("change", function() { $container.masonry("reload"); });
TelFiRE’s picture

I mean it sounds like a really simple solution and looks like it has been solved for quite a while? Why can't someone just commit it? Pretty please?

kyletaylored’s picture

I'm just converting logaritmisk's into a patch, it worked for me. I'll post the Masonry patch in their issue queue as well.

BWPanda’s picture

Status: Active » Needs review
FileSize
1.96 KB

Here's an updated patch that:

  • Includes the necessary changes from #2 above
  • Is formatted correctly (see http://drupal.org/node/707484)
  • Renames the trigger event to be less generic
BWPanda’s picture

Issue summary: View changes

Updated issue summary.

jibran’s picture

Status: Needs review » Reviewed & tested by the community
pierrelord’s picture

Works great. Big thank you :)

stupiddingo’s picture

Brilliant! Works great with Omega.

jibran’s picture

#1808018: Make work with Views Infinite Scroll is fixed yay! so can we please fix this as well :)

vinoth.3v’s picture

Here, with latest version of both Infinite scroll, Masonry and jquery 1.8: (Omega theme)

  1. New Blocks are overlapping with existing Masonry blocks
  2. Views duplicate mini pager are displaying.

any Idea?

nyleve101’s picture

I am also using the omega theme and experiencing the same issues.

vinoth.3v’s picture

Status: Reviewed & tested by the community » Active

Forget to mention, Omega version is 4.x

jibran’s picture

Status: Active » Needs work

Can you test it against core jQuery 1.4.4? If it is working fine then it is out of scope for this issue we have to create new issue for jQuery 1.8.

vinoth.3v’s picture

Priority: Normal » Major
Status: Needs work » Active
FileSize
118.88 KB

With Garland theme and Jquery Update module disabled, it works with Table, Grid display formats. But ajax loading is not even showing with Masonry grid display.

Chrome inspector show JS error.

jibran’s picture

Status: Active » Needs work

Can you attach your view as well?

If you are getting error after applying the patch that means patch need work. Usually active status mean no activity and we just started the issue. After issue got patch it will be need work, need review or RTBC. So changing the status to need work.

vinoth.3v’s picture

Status: Needs work » Reviewed & tested by the community

After applying the patch?
Oh wait. I thought the patch is already committed to git.

It works great now. Thank you for the patch.

btopro’s picture

patch in #13 works great. RTBC?

Deg’s picture

FileSize
1.98 KB

I'd recommend adding the current and next variables to the load function (as defined in the autopager.js docs at http://lagoscript.org/jquery/autopager/documentation) so this information is available in the trigger.

jpitassi’s picture

The issue that I still have is that masonry is not reloaded or appended so my next page of results just end up overlapping each other. Looking at both Masonry and Infinite Scroll module, I can not see a way for Infinite scroll to reload masonry without explicitly calling masonry. Because I am only using infinite scroll for 1 view on my site and just prototyping something I made a quick and dirty solution. I first remove the infinite scroll js from the scripts array in my theme template.php and add in a custom js file:

function THEMENAME_js_alter(&$javascript) {
  if( isset($javascript['sites/all/modules/contrib/views_infinite_scroll/js/views_infinite_scroll.js']) ) {
    unset($javascript['sites/all/modules/contrib/views_infinite_scroll/js/views_infinite_scroll.js']);
    drupal_add_js(drupal_get_path('theme', 'THEMENAME').'/js/views_infinite_scroll_custom.js');
  }
}

Then I copy the views_infinite_scroll/js/views_infinite_scroll.js to THEME/js/views_infinite_scroll_custom.js
Then add on line 45 of views_infinite_scroll_custom.js add $(content_selector).masonry( 'reload' );

...
              load: function() {
                $('div#views_infinite_scroll-ajax-loader').remove();
                Drupal.attachBehaviors(this);
                $(content_selector).trigger('views_infinite_scroll_updated');
                $(content_selector).masonry( 'reload' );
              }
            });
...

That will reload masonry after the infinite scroll has loaded the objects in the list. I am just prototyping so this will work for me, for now.

CarlosML’s picture

I'm having the same problem as previous post: even applying patches, when Views Infinite Scroll adds more images, they're overlapped with already loaded ones.

I'm by no means a programmer or an expert, but reading jQuery Masonry documentation I found it has imagesLoaded plugin built-in. So, I wrapped the views_infinite_scroll_updated as this:

 }).bind("views_infinite_scroll_updated", function () {
		$container.imagesLoaded(function(){ $container.masonry("reload"); });
	  });

And now it seems its working OK. It is necessary to make sure that images are already loaded before calling 'reload' or I'm completely wrong headed?

Thanks!

andrew smith’s picture

For total beginners with patches (like me) – on Mac using Terminal - #13 patch works, BUT NOTE: it patches two different files in different locations at once so is tricky.
Using the usual method for applying patches for Mac using Terminal, it won't work: you need to divide the patch using TextEdit (or whatever) into two separate patches. One is applicable to js/views_infinite_scroll.js the other is for views_plugin_pager_infinite_scroll.inc .

Then patch both patches in the usual Terminal way. Apologies if there is a more sensible method; I don't know it.

stupiddingo’s picture

So to clarify my earlier delight. Works brilliantly with Omega 7.x-3.1 and as #29 states there are two patches from #13 that must be applied for correct behavior.

bradallenfisher’s picture

Is there any new insight into this. I have applied patches, however when using images in a view with masonry and vis, I get overlapping.
I am using Omega 3.x.
This doesn't happen on first load.
This clears up on refresh.
Looks like more than one "page" is loaded when i start to scroll. Like it seems like two many pages are being pulled in. Could that be the cause of the overlapping?

I will test whatever you need me to. This is stopping me from using the combo of these two module in production.

Thanks.

bradallenfisher’s picture

It seems like all subsequent pages are being loaded at once on scroll... This is chrome.

Anonymous’s picture

I am experiencing the same issues. It seems to load 1-2 pages at a time on the first scroll. Sometimes there is overlap, other times not so.

dddave’s picture

Patch in #13 works fine for me in conjuncture with the latest 1x dev of masonry. No other tweaking necessary. Tested with Chrome and FF.

miik4’s picture

#27:
I didn't see this functionality attached to Masonry / Infinite Views Scroll, and tried this solution.
Works great, and doesn't cause problems when replacing views_infinite_scroll.js with views_infinite_scroll_custom.js file ;)

Thanks!

capellic’s picture

In addition to the patch in #13, I had to change the code in masonry_views.module, specifically the block of "Initialize Masonry" code starting at line 57. The problem is that the code runs the Masonry script before the images were loaded on scroll.

I wrapped it in jQuery(document).ready(function(){ ... }); and also wrapped the reload in $container.imagesLoaded(function () { ... }); Here's the whole file to save confusion:

/**
 * @file
 * Provides a Views style plugin to display content in a jQuery Masonry grid.
 */

/**
 * Implements hook_views_api().
 */
function masonry_views_views_api() {
  return array('api' => '3.0');
}

/**
 * Preprocess function for views_view_masonry.tpl.php.
 */
function template_preprocess_views_view_masonry(&$vars) {
  // Run preprocess function for unformatted style
  template_preprocess_views_view_unformatted($vars);

  if (($library = libraries_load('masonry')) && !empty($library['loaded'])) {
    $view = $vars['view'];
    $view_class = 'view-' . drupal_clean_css_identifier($view->name) . '.view-display-id-' . $view->current_display;
    $settings = $vars['options'];

    // Add default styling to make grids display properly out-of-the-box
    $css_margin = ($settings['masonry_width_unit'] == 'px') ? '10px' : '2%';
    $css_width = ($settings['masonry_width_unit'] == 'px') ? ($settings['masonry_width'] - 20) . 'px' : ($settings['masonry_width'] - 5) . '%';
    $grid_styles = '
      .' . $view_class . ' .masonry-item {
        float: left;
        margin: ' . $css_margin . ';
        width: ' . $css_width . ';
      }
    ';
    drupal_add_css($grid_styles, 'inline');
    if ($settings['masonry_center']) {
      $center_styles = '
        .' . $view_class . ' .view-content {
          margin: 0 auto;
        }
      ';
      drupal_add_css($center_styles, 'inline');
    }

    // Get column width
    if ($settings['masonry_width_unit'] == 'px') {
      $column_width = (int) $settings['masonry_width'];
    }
    else {
      $percentage = $settings['masonry_width'] / 100;
      $column_width = 'function (containerWidth) {
        return containerWidth * ' . $percentage . ';
      }';
    }

    // Initialize Masonry
    $script = 'jQuery(document).ready(function(){
      (function ($) {
        var $container = jQuery(".' . $view_class . ' .view-content");
        $container.imagesLoaded(function () {
          $container.masonry({
            itemSelector: ".masonry-item",
            columnWidth: ' . $column_width . ',
            isAnimated: ' . (int) $settings['masonry_animated'] . ',
            animationOptions: {
              duration: ' . (int) $settings['masonry_animated_duration'] . '
            },
            isResizable: ' . (int) $settings['masonry_resizable'] . ',
            isFitWidth: ' . (int) $settings['masonry_center'] . ',
            gutterWidth: ' . (int) $settings['masonry_gutter'] . ',
            isRTL: ' . (int) $settings['masonry_rtl'] . '
          });
        }).bind("views_infinite_scroll_updated", function () {
          $container.imagesLoaded(function () {
            $container.masonry("reload");
          });
        });
      })(jQuery);
    });';
    drupal_add_js($script, array(
      'type' => 'inline',
      'scope' => 'footer',
    ));
  }
}
xaqrox’s picture

I had to make the same modifications to Masonry as capellic. Gonna reference his comment in the Masonry queue so the infinite scroll patch in #13 stays in the right place.

xaqrox’s picture

Actually I think this was my problem: #1983582: Allow default row classes to be disabled

metastate’s picture

I couldn't get Views Infinite Scroll to work using the latest versions of Views (7.x-3.7), Masonry API (7.x.2.0), Masonry Views (7.x.1.0) and Views Infinite Scroll (7.x.1.1). I am using jQuery v1.7.1, jQuery Masonry v2.1.08, jQuery.autopager v1.0.0.

Here is my troubleshooting sequence (long story short, I needed the fixes in #13 and #27 above)...

  • Disabled all my custom CSS for the View for testing purposes.
  • Using the "Masonry" page style format, left all settings as default, other than adding a column width.
  • Added a width and margin for the .views-row div in my theme's CSS file.
  • Installed Views Infinite Scroll and added it to the View. Make sure jquery.autopager-1.0.0.js has been downloaded to /modules/views_infinite_scroll/js/
  • Nothing is happening, Views Infinite Scroll is not working. Time to apply patches...

Masonry API - Make work with Views Infinite Scroll - patch in #3: https://drupal.org/node/1808018#comment-6954014
This patch doesn't apply to the latest version of Masonry Views, so I skipped it.

Views Infinite Scroll - Support Masonry module - patch in #13:
https://drupal.org/node/1806628#comment-6954012
NOTE: This patch applies to two files - views_infinite_scroll.js and views_plugin_pager_infinite_scroll.inc

At this point, Infinite Scroll works now, but when the scroll loads, all of the images are in a single column on the left hand side, with some overlapping at the top.

Views Infinite Scroll - Support Masonry module - soluton in #27:
https://drupal.org/node/1806628#comment-7068082
I just modified views_infinite_scroll.js instead of creating a custom.js file.

HUZZAH!! It works.

  • Then I tweaked my CSS until everything lined up the way I wanted (I'm using images with different widths).
  • There is sometimes a little bit of image overlapping at first, but after a brief delay they align to the Masonry grid.
  • Of course be sure to clear your site AND browser cache every time you apply a patch or modify a JS/CSS file.

Phew...

nuncius’s picture

Hey metastate, thanks a lot!

I can confirm that this method described in #39 works perfectly, Finally after severeal days of trying, my infinite scroll finally works with masonry.

mohamadaliakbari’s picture

in addition to #27 its better to reload masonry only if its necessary:

....
$('div#views_infinite_scroll-ajax-loader').remove();
Drupal.attachBehaviors(this);
$(content_selector).trigger('views_infinite_scroll_updated');
if ($(content_selector).hasClass('masonry')) {
  $(content_selector).masonry('reload');
}
....

So still you can use infinite scroll without masonry.

charlie charles’s picture

would let Let me know please?
what's the command to install this patch with Drush?

Many Thanks

dixon_’s picture

dixon_’s picture

Assigned: Unassigned » dixon_
Status: Reviewed & tested by the community » Needs review
FileSize
1.93 KB

Here's an updated and slightly more generic patch inspired by @logaritmisk in #10. This patch works great with the new Masonry 7.x-2.x after applying the new patch at #1808018-12: Make work with Views Infinite Scroll.

rteijeiro’s picture

Status: Needs review » Reviewed & tested by the community

Tested #44 patch and #1808018-12: Make work with Views Infinite Scroll patch and now it works like a charm.

heddn’s picture

+1 on RTBC

Andre-B’s picture

+1 RTBC just tested both patches, works like a charm. Hope that there will be a stable/ dev release soon

Mavro’s picture

Hi Everyone,

I'm working on a project that uses both Masonry and Views Infinite Scroll and had the same overlapping issues.

After reading through the comments and trying a few of the suggested solutions, Dixon's instructions on comment #44 worked for me:

1. Applied this patch to the Masonry API module:
https://drupal.org/files/1808018-masonry2-vis-11.patch

2. Applied this patch to the Views Infinite Scroll module:
https://drupal.org/files/1806628-vis-masonry-43.patch

For those who need to learn how to apply patches, here's a useful tutorial: http://www.youtube.com/watch?v=i-oe7_qHreY

Everything appears to be working correctly now. Thank you, everyone! Hopefully, new releases of both Views Infinite Scroll and Masonry API will be released that play nice with each other.

Lastly, I don't post often, so please let me know if I'm referencing things incorrectly. I'll update my post.

bwoods’s picture

Agreed ... #44 works for me too. I should also note that since my masonry items are variable width, I checked the box in the view settings to load the images first. This way, masonry resizes properly. Thanks!

Geizt’s picture

Assigned: dixon_ » Unassigned
Category: feature » bug
Status: Reviewed & tested by the community » Needs review
FileSize
1.04 KB

After applying #1806628-vis-masonry-43.patch the switch case concerning the "list" plugin, forces the wrapper class to be set in order to create a proper content selector.

If no wrapper class is set the infinite scroll JS will not be able to attach the result of the AJAX request to the existing list being displayed.

This patch checks if a wrapper class exists and only then uses it in the creation of the content selector, otherwise a fall-back is used instead that doesn't require a wrapper class.

Geizt’s picture

Issue summary: View changes

Updated issue summary.

MHz’s picture

After applying the two patches from #48 things still don't work for me, a pager appears at the bottom of the view, and when you scroll down new items load but are layered under the old ones, e.g http://i.imgur.com/rOPHOrQ.png

Mavro’s picture

MHz, which version of modules (Infinite Scroll and Masonry) are you using? I can compare with my project.

MHz’s picture

I've managed to fix my problem.

rteijeiro’s picture

Tested #50 patch and it seems to fix the problem with item lists in views. It should be added to main patch in #44.

alesel’s picture

Has anybody problems with iOS devices?

alesel’s picture

On Android have also troubles.
I found out that it works only after zooming.

morleman’s picture

One question, when scrolling back up via JavaScript button or mouse is it possible to "reset" it again when you reach the top? So if they scroll back down they re-encounter this feature?

pacome’s picture

Issue summary: View changes

Thanks a lot !

great result with
patch on https://drupal.org/comment/7940483#comment-7940483 (this issue, #50)
patch on https://drupal.org/comment/8472445#comment-8472445 (V2 on #24)

[edit] : it seems that the javascript used inside the view is not loaded after the first page (ie : slideshows will be loaded on 1st page, not on the next ones..)
it's probably related to those other issues : https://drupal.org/node/2164235 and https://drupal.org/node/2122273

xaa’s picture

patch #44 and #50 working on desktop. thanks

spkrs’s picture

I'm removing my request as I posted it in the wrong thread. Apologies for the newbie move.

kopeboy’s picture

Are the working patches still needed for this to work or have they been included already?

Months have passed and this isn't Fixed yet.

zanix’s picture

I have patch #44 and #50 along with a patch for Masonry #2022371-23: Support for Masonry 3.0? and I have it working.

kopeboy’s picture

I successfully applied #2022371-23: Support for Masonry 3.0? to Masonry dev, but these #44 and #50 doesn't seem to apply to VIS. Is it because the latest dev version as them already? I am a noob.. sorry

(to clarify, in the first case I get "patch applied cleanly", in 2nd and 3rd I get no message on my console when trying to apply patch)

EDIT: Ok I had to use patch -p1 < in this case, but I got this error on the 1806628-vis-masonry-43.patch

patching file js/views_infinite_scroll.js
<strong>Hunk #1 FAILED at 41.</strong>
1 out of 1 hunk FAILED -- saving rejects to file js/views_infinite_scroll.js.rej
patching file views_plugin_pager_infinite_scroll.inc
Hunk #1 succeeded at 62 (offset 37 lines).
Hunk #2 succeeded at 80 with fuzz 1 (offset 38 lines).

and this when applying 1806628-50.patch

Hunk #1 FAILED at 74.
1 out of 1 hunk FAILED -- saving rejects to file views_plugin_pager_infinite_scroll.inc.rej

Why is that?

dddave’s picture

Quick guess: The latest dev changed too much (looking at the commits it seems likely). Have you an older dev saved somewhere to test the patches? (Better yet: Update the patches, but that might be a bit much for now ;))
I have it working nicely with older versions of masonry and VIS and never updated them to avoid this scenario. Nothing here has been committed so far.

kopeboy’s picture

Unfortunately I don't have old versions. I restarted downloading the dev version once more and reapplying patches, both for Masonry API and VIS.

Support for Masonry 3.0 > ok
Patch 43 here > ok

Patch 50 here > error:

patching file views_plugin_pager_infinite_scroll.inc
Hunk #1 FAILED at 74.
likewhoa’s picture

patch needs a re-roll that's all.

zanix’s picture

Re-roll and combination of patches #43 and #50 applied to 7.x-1.x-dev

newme154’s picture

Hey Guys,

Having the toughest time where to put these libraries. I have them under sites/all/libraries/imagesLoaded/imagesloaded.pkgd.min

and

sites/all/libraries/Masonry/masonry.pkgd.min with the same outcome. Nothing. In views it says "These options have been disabled as the jQuery Masonry plugin is not installed." However, I know its there.

Any ideas?

zanix’s picture

The library files need to look like this (no capitol M or L)

sites/all/libraries/imagesloaded/imagesloaded.pkgd.min.js
sites/all/libraries/masonry/masonry.pkgd.min.js
TheRealDeal’s picture

After applying dixon's patches that are mentioned in #44 the views infinite scroll and masonry were now working together in sync but then the animations that I had on the page that used the jquery zoom plugin were all broken, I have also tried applying patch in post #50 and thought that it had resolved the issue but it hasn't, after i scroll below the first 'page' and the new items load in the jquery zoom script stops working correctly, does anyone know why this is happening?

guschilds’s picture

EDIT: Forgot to attach a patch. See #72. :)

guschilds’s picture

The attached patch is the same as the one in #67, but applies to the 7.x-1.1 release. I'm adding it here so I can reference it with Drush Patchfile until the next release is made for this module. Thanks everyone!

Dom.’s picture

Hi !
This inspired by patch #67 applied to latest (as per 19/07/2015) 7.x-1.x-dev branch. This is important to get Masonry Views module (and possibly others based on the unformatted list style) compatible with VIS module.
NOTE: It is not necessary to trigger a special event "change" as per patch #67 since DrupalBehaviors are already triggered in the load method here, and that is what they are made for.
Thanks

stevieb’s picture

The patch in #72 works beautifully for me ... thanks

Dom.’s picture

Just push for the patch ! Because Masonry Views module users needs it !

2pha’s picture

#72 worked for me.
I then just needed to add this to my sites js.

  Drupal.behaviors.infiniteMasonry = {
      attach: function(context, settings){
        $('.view .masonry').change(function(e){
          $(this).masonry('reload');
        });
      }
  };

The masonry views module should probably listen for these change events and reload if necessary (should also check target of event as to not reload on change events on something inside the view).

knalstaaf’s picture

#73 works perfectly for me (no need for extra js)!

(There's a patch to hide the pager as well, which plays well with Masonry)

knalstaaf’s picture

Status: Needs review » Reviewed & tested by the community
alexit873’s picture

Hi, i'm sorry to open again this request but i think it doesnt work anymore the fix on #73,
because the view_plugin_pager_infinite_scroll.inc changed and i can't find any of the patched lines on the new file.

Here is my page with the problem:
http://vidiall.com/node/133#filter

Please help so the new items will fit like they should

errand’s picture

#73 works perfectly on 7x-1-dev version

dddave’s picture

Ugh, why was a rc1 rolled out at least a commiters review of this patch. :/

Sam152’s picture

RC1 is rolled out for 7.x-2.x. This patch can be committed to 7.x-1.x, but if it's a widely requested feature it would be nice to see it in the latest version.

szeidler’s picture

I can confirm both, that the masonry effect is working using the old 7.x-1.x branch with patch #73

  • Views Infinite Scroll 7.x-1.x-dev (2014-Oct-02) with patch #73
  • Masonry API 7.x-3.0-beta1
  • Masonry Views 7.x-3.0-beta1 and 7.x-3.x-dev (2016-Feb-16)

and the new 7.x-2.x branch

  • Views Infinite Scroll 7.x-2.0-rc1
  • Masonry API 7.x-3.0-beta1
  • Masonry Views 7.x-3.0-beta1 and 7.x-3.x-dev (2016-Feb-16)
dddave’s picture

Ugh myself for overlooking the version # and conveniently forgetting that I have it tested on 2x a while ago. Sorry.

Sam152’s picture

Status: Reviewed & tested by the community » Needs review
FileSize
494 bytes
494 bytes

I agree that #73 is on the right train of thought. Behaviors should be attached and everything should know what to do with it from there. I have not tested this patch but the fix could be in this direction. It's possible the AJAX system already calls this for us, will have to dig into this deeper.

ShaneOnABike’s picture

Patch #73 works really great with the latest Masonry and Masonry Views

Honza Pobořil’s picture

Status: Needs review » Closed (outdated)

7.x-1.x will receive no work. Migrate to 7.x-2.x.