Patch (to be ported)
Project:
Views Fluid Grid - jQuery Masonry
Version:
7.x-1.x-dev
Component:
Miscellaneous
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Jan 2011 at 12:47 UTC
Updated:
20 Jun 2014 at 20:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
stoatoffear commentedMine too!
Comment #2
erfunath commentedHas anyone come up with a solution to this? My images are stacking properly on Firefox, but Chrome/Safari are piling up vertically like in the image posted above by stoatoffear.
Comment #3
iaminawe commentedI am experiencing the same.
I think it may have to do with this http://masonry.desandro.com/docs/intro.html#imagesloaded_plugin
I should mention I am using the D7 version and other than this it works great.
Any thoughts?
Comment #4
Marko B commentedI am having the same problem, even this doesnt help.
http://drupal.org/node/1113602 in Chrome i keep getting images stacked one over the other as in attachment.
Comment #5
technikh commentedI am having same issue. ie, safari, chrome all show stacked images when I press F5(Refresh) but when I press enter key in address bar the page looks good with unstacked images.
Firefox version 8 shows unstacked images all the time
Comment #6
technikh commentedmarking as bug
Comment #7
technikh commentedeven though I enable the below option, it's not fixing this issue
Reload items on window load
Call the reload method on $(window).load(), this helps to deal with external media such as images or @font-face kits.
I am using the D7 version at http://drupal.org/node/1031402#comment-4890184
Comment #8
technikh commentedI fixed it by adding imagesLoaded functionality as suggested by iaminawe http://masonry.desandro.com/docs/intro.html#imagesloaded_plugin
.imagesLoaded( function(){
in init.js
Comment #9
technikh commentedChanging status to needs review
Comment #10
Marko B commentedTechNikh could you post this also for D6 version, i am not good with jquery so i could transform this to be comaptible with d6 JS code.
Comment #11
technikh commentedI didn't test it it, but the above code should work for D6 version too.
Comment #12
Marko B commentedOk but where to even put it? there is no init.js in D6 version. I'll test if I can install this code properly, please help on this part
Comment #13
technikh commentedI guess in views_fluidgrid.module put "imagesLoaded" before
Comment #14
Marko B commentedHmmm there is no such code in "views_fluidgrid.module" but i guess you ment to put imageloaded somehow in part where masonry is configured?
Comment #15
technikh commentedI don't remember where I saw that, but Yes! put imageloaded in part where masonry is configured
Comment #16
Marko B commentedFirst of, you need to upload a file "jquery.imagesloaded.min.js" somewhere and add it to drupal. Then add its path in views_fluidgrid.module somewhere here
then add settings somewhere think around the comment that says "this looks like shit" :-)
Comment #17
Marko B commentedHere is complete solution for drupal 6, http://www.drupaldump.com/views-fluid-grid-jquery-masonry-fix-images
Works great and as it should, asking maintainer to update this module :-)
Comment #19
giorgio79 commentedI am seeing this n 7x dev. A fix would be great in the module code, not in a blog :)
Comment #20
zilla commentedi think that this should be marked RTBC and moved into the dev version of this module for D7 and 6, and perhaps as a prep for a D7 alpha of some kind (not sure what the roadmap is for this module, though it seems quite popular)
Comment #21
arturs.v commentedCreated a quick patch from comment #8
Comment #22
sjancich commentedPatch in #21 works for me.