Don’t hesitate, if you want something improved and it makes sense/if I can do it, I’ll get ‘er done and put it in the next release.
Hey Adam! Great widget!
One thing I though of to add was a “target=’blank'” tag to open twitter in a new window, because I would think that people should keep my site open 😉 haha.
echo '<a href="http://www.twitter.com/'.$twittername.'" target="blank" ><h2 id="twitname">'.$twittername.'</h2></a>';
One thing I am trying to accomplish though is changing the header to say something like “Follow $twittername On Twitter” but I can’t get it to display inside the h2 tags without breaking it.
Good call on the target=”blank” for the tweets! For those wanting to do this before I add it to the next release, find the for loop in “feed.js” and replace this line:
html += '<a href="http://twitter.com/' + username + '#status_' + data[i].id_str + '">' + data[i].text + ' <i>' + Twitter.daysAgo(data[i].created_at) + '</i></a>';
html += '<a href="http://twitter.com/' + username + '#status_' + data[i].id_str + '" target="_blank">' + data[i].text + ' <i>' + Twitter.daysAgo(data[i].created_at) + '</i></a>';
I’ll add that to the next release so everyone can have it. As far as changing the h2 tag, the issue there is due to how the js is getting the username. This should solve that for you:
echo '<a href="http://www.twitter.com/'.$twittername.'" target="blank" ><h2>Follow @<span id="twitname">'.$twittername.'</span> on Twitter</h2></a>';
That seems like a pretty logical change so I’ll probably add that to the next release as well.
cant believe I didn’t try changing it to span.
This also brought to light another possible improvement:
Right now the tweet links don’t actually go anywhere unique (they have the url that looks like http://www.twitter.com/username#status_tweetid) I believe it would be improved if I changed it to http://www.twitter.com/username#stream-item-tweet-tweetid, this would send the page down to the actual tweet that was clicked. What do you think?
I’ll play around with it for a bit since it isn’t a major fix. Look for an update with all of these changes in the next couple of days.
Let me know what you come up with.
I noticed that myself, and I didn’t know if it was just me thinking of that. Like you say it isn’t a big thing, so it wasn’t very pressing for me.
- The topic ‘[Plugin: Adam's Twitter Ticker Feed] Any suggestions?’ is closed to new replies.