Search code examples
javascriptjqueryjquery-masonrymasonry

Masonry why arranging in single column overlapping


I am using the appended function to append elements to an already initialized masonry instance, but all of my tiles are being laid out in a single column and many are overlapping. Can you see what I am doing wrong?

function placeNewsTiles(news){ //places news tiles 
    var length = (news.data.length > 20) ? 20 : news.data.length;

    var $container = $('#news');

    var elems ="";
    for(var i = 0; i < length; i++){
        elems += '<div class="pageNewsItem" id="'+ count + i + '">\
                <div class="textWrap">\
                    <a href="' + news.data[i]._url + '">\
                    <strong>' + news.data[i]._title + '</strong>\
                    </a>\
                    <span class="source">' + news.data[i]._source + '</span>\
                </div>\
                <div class="imageWrap"></div>\
                <div class="thumbsOverlay" style="display:none">\
                    <div class="thumbs">\
                        <div>\
                            <a href="#"" class="up"><img src="../images/Thumbs-Up-2.png" /></a>\
                            <a href="#"" class="down"><img src="../images/Thumbs-Down-2.png" /></a>\
                        </div>\
                    </div>\
                </div>\
            </div>';


        getTileImage({total: news.count, i:count + "" + i, url:news.data[i]._url});
    }
    elems = $(elems);

    $('#news').append(elems).imagesLoaded(function(){
        $('#news').masonry( 'appended', elems, true );
    });

    newsPage = 0;
    count++;
    hoverTiles();
}

$(document).ready(function() {

    newClock();

    readyLogin();

    var $container = $('#news');
    //$container.imagesLoaded( function() {
      $container.masonry({itemSelector: '.pageNewsItem'});
    //});


    obtainSearchQuery();
});

Solution

  • On this line, try triggering layout once items are appended:

    $('#news').masonry( 'appended', elems, true ).masonry('layout');
    

    I've seen examples with and without the extra .masonry() call, but it might work.