Question

i want to loop trough all my delicious.com bookmarks and wrap a link arround them…

here’s my testsite:
http://dev.thomasveit.com/json.html

$(document).ready(function(){               
        $.ajax({
            url: "http://feeds.delicious.com/v2/json/tommyholiday",
            dataType: "jsonp",
            success: function(data){
                var bookmarks = $.map(data, function(bookmark){
                    return {
                        title: bookmark.d,
                        link: bookmark.u
                    }
                });                 

                var html = "<ul>",
                    m;

                for (i=0; i<bookmarks.length; i++){
                    m = bookmarks[i];

                    html += "<li><a href="+m.link+">"+m.title+"</a></li>";
                }
                html += "</ul>";

                $("#delicious").html(html);

            }                       
        });
});

funnily enough only the third link is getting wrapped by a link… the others not.

what am i doing wrong?

Answer 1

a href attribute need quotes.

html += "<li><a href="+m.link+">"+m.title+"</a></li>";

should be

html += "<li><a href='"+m.link+"'>"+m.title+"</a></li>";