I think we got this solved off-list, but for completeness I'll respond here as well.
The line starting with "statusHTML += ..." should all be one line. The email broke it into several lines apparently. JavaScript doesn't like breaking string literals into several lines like some other languages allow. So, take out the linebreaks and you should be set. Also, don't forget a semi-colon at the end of that line. -Chad On Tue, Feb 3, 2009 at 11:09 AM, Plizzo - MacThemes <[email protected]> wrote: > > Thank you so much chad. Now I only have one problem. I get a syntax > error at line 14 where this_bg_color is. > > On 2 Feb, 21:52, Chad Etzel <[email protected]> wrote: >> i have added some lines (marked by //new) that should accomplish what you >> want. >> >> function twitterCallback2(obj) { >> var twitters = obj; >> var statusHTML = ""; >> var username = ""; >> >> var name = ""; //new >> var bg_arr = []; //new >> bg_arr.push("#ddd"); //new >> bg_arr.push("#bbb"); //new >> >> for (var i=0; i<twitters.length; i++){ >> >> var this_bg_color = bg_arr[i % 2]; //new >> >> username = twitters[i].user.screen_name >> >> name = twitters[i].user.name //new >> >> statusHTML += ('<li><span style="background:' + >> this_bg_color + ';">'+ name + ' said: ' + twitters[i].text+'</span> >> <a >> target="_blank" style="font-size:85%" href="http:// >> twitter.com/'+username+'/statuses/'+twitters[i].id+'">'+relative_time >> (twitters[i].created_at)+'</a></li>') //new >> } >> document.getElementById('twitter_update_list').innerHTML = >> statusHTML; >> >> } >> >> -Chad >> >> On Mon, Feb 2, 2009 at 3:38 PM, Plizzo - MacThemes >> >> <[email protected]> wrote: >> >> > function twitterCallback2(obj) { >> > var twitters = obj; >> > var statusHTML = ""; >> > var username = ""; >> > for (var i=0; i<twitters.length; i++){ >> > username = twitters[i].user.screen_name >> > statusHTML += ('<li><span>'+twitters[i].text+'</span> <a >> > target="_blank" style="font-size:85%" href="http:// >> > twitter.com/'+username+'/statuses/'+twitters[i].id+'">'+relative_time >> > (twitters[i].created_at)+'</a></li>') >> > } >> > document.getElementById('twitter_update_list').innerHTML = >> > statusHTML; >> > } >> >> > function relative_time(time_value) { >> > var values = time_value.split(" "); >> > time_value = values[1] + " " + values[2] + ", " + values[5] + " " + >> > values[3]; >> > var parsed_date = Date.parse(time_value); >> > var relative_to = (arguments.length > 1) ? arguments[1] : new Date >> > (); >> > var delta = parseInt((relative_to.getTime() - parsed_date) / 1000); >> > delta = delta + (relative_to.getTimezoneOffset() * 60); >> >> > if (delta < 60) { >> > return 'less than a minute ago'; >> > } else if(delta < 120) { >> > return 'about a minute ago'; >> > } else if(delta < (60*60)) { >> > return (parseInt(delta / 60)).toString() + ' minutes ago'; >> > } else if(delta < (120*60)) { >> > return 'about an hour ago'; >> > } else if(delta < (24*60*60)) { >> > return 'about ' + (parseInt(delta / 3600)).toString() + ' hours >> > ago'; >> > } else if(delta < (48*60*60)) { >> > return '1 day ago'; >> > } else { >> > return (parseInt(delta / 86400)).toString() + ' days ago'; >> > } >> > } >> >> > ---------------------------------------- >> > I have this code which I use to fetch the friends_timeline. I want to >> > add so that you can see who sent the tweet like "Matt said:" and also >> > I want to have backgrounds alternating between every post. Like having >> > first one light grey background and the second tweet having a darker >> > one. Is this possible, I would require help with this as I have been >> > tinkering with it for days. Any help is appreciated >> >> > Regards // Jonathan >
