The plugin has hit the 200 mark! Anyone have any improvement suggestions that they wish they would see? To be honest I was working on trying to allow for multiple widgets but I just can't get the javascript working properly so that will have to wait. How is everything else working for you?
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.
Cheers!
Posted 8 months ago #
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.
Any ideas?
Thanks!
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>';
with
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.
Posted 8 months ago #
Genius!
cant believe I didn't try changing it to span.
Great work!
Thank you!
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.
Posted 8 months ago #
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.
Thanks again for the feedback! 300 downloads and I haven't heard from anyone else, I assume that means it's doing the trick or they hate it/deleted it lol.
You must log in to post.
