ansaurus

Question

Using Jquery each loop to display six list items per ul inside a div pulled from a JSON object

Answer 1

A:

Hi,

First of all, I suggest that you clean up your code and remove any unnecessary stuff, you'll make it easier to read and hopefully to understand and fix.

I think your problem comes from the way you handle your if block if (i == 0 || i % 6 == 0) (btw, if (i % 6 == 0) also works): the proNameLink variable is filled in this block, which explains why you only get the 1st item of every 6 found. I guess that you wanted to do the following:

if (i % 6 == 0)
{
    // Create a new list for 6 next items
    $listReference = $('<ul></ul>');
    // Create a container div for your list
    var containerDiv = $('<div class="hasImage"></div>')
    // append the div and list to the page DOM
    div.append($listReference);
    $('#ajax-returned-content').append(div);
    // Note that if you want to set an ID to the ul and the div, you have to
    // make it UNIQUE. Your code creates multiple DOM elements with the same ID.
    // A kitten dies every time you do that.
}
// Create a list item, OUTSIDE of the if block
var proName = item.pro_name;
var proNameLink = $('<li><a class="pro-name-link'+i+'" href="#">'+proName+'</a></li>');
// Append list item to current list
$listReference.append(proNameLink);

ybo 2009-11-02 09:05:27

I think that's where I went wrong...the proNameLink within the if block. But I tried out your solution and now I'm not getting any output at all...am I missing something?

loganlee 2009-11-02 18:45:00

ansaurus

tags:

views:

answers:

Using Jquery each loop to display six list items per ul inside a div pulled from a JSON object

related questions