I am displaying images and image titles in views using masonry. The grid layout looks fine for admin user but for anonymous user, everything displays in one column only. Not sure if this is a bug or an incorrect configuration.

CommentFileSizeAuthor
#5 masonry_not_working_for_anonymous--2696293-5.patch830 bytesDom.
#4 masonry_not_working_for_anonymous--2696293-4.patch828 bytesDom.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cmwelding created an issue. See original summary.

Dom.’s picture

Dom. CreditAttribution: Dom. commented
Status: Active » Needs work

Acknowledged, I will have a look.

shevatvb’s picture

shevatvb CreditAttribution: shevatvb commented

Same issue here.
I'm getting this error from console

Uncaught TypeError: $container.once is not a function masonry.js?o4unuz:94

Dom.’s picture

Dom. CreditAttribution: Dom. commented
Status: Needs work » Needs review
FileSize
masonry_not_working_for_anonymous--2696293-4.patch828 bytes

Hi!

Very nice catch: this is a stupid problem ! (~50 sites uses the D8 version, you are the first to report it !). The Jquery Once library is used so widely in core and on contrib modules that the chance to not use another module that loads it is low. I guess that's why no-one noticed it. (also proof that my dev environment has too much modules now ^^)

A patch is attached; one line correction: declare the dependency on jquery once !

After patch, you need to clean cache to take effect, or to run update.php: the patch contains an update method that will clean it for your.

Dom.’s picture

Dom. CreditAttribution: Dom. commented
FileSize
masonry_not_working_for_anonymous--2696293-5.patch830 bytes

Better comment on update.php method !

shevatvb’s picture

shevatvb CreditAttribution: shevatvb commented

Hi Dom!
Thank you so much!
It works now! You were super fast!
Thanks again!

Dom.’s picture

Dom. CreditAttribution: Dom. commented
Status: Needs review » Fixed

Commited on dev-branch

Dom.’s picture

Dom. CreditAttribution: Dom. commented
Project: Masonry Views » Masonry API

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

cancerian7’s picture

cancerian7 CreditAttribution: cancerian7 commented

Not fixed in latest dev. I have to implement this patch manually.

gemalm’s picture

gemalm
Primary language Spanish
CreditAttribution: gemalm at Digitalist commented

This patch works for me in Drupal 8.

dadderley’s picture

dadderley CreditAttribution: dadderley as a volunteer and commented

I am having the same issue as the original poster.
Also I cannot get the patch to work.

Strangely enough I have another Drupal 8.8.4 site where this problem does not happen.
Really odd.

joshuautley’s picture

joshuautley CreditAttribution: joshuautley commented

#5 worked for me using Drupal Version 8.9.1

Thank you!

fabiana_matos’s picture

fabiana_matos CreditAttribution: fabiana_matos commented

I am having the same issue as drupal 9.1.0 and Masonry 8.x-1.2
I implemented the patch manually and it didn't work