WordPress.org

Ready to get started?Download WordPress

Forums

Adam's Twitter Ticker Feed
Any suggestions? (7 posts)

  1. bythegram
    Member
    Plugin Author

    Posted 1 year ago #

    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!

    http://wordpress.org/extend/plugins/twitter-feed-ticker/

  2. toes21
    Member
    Posted 1 year 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.

      LINE 37:

    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?

  3. bythegram
    Member
    Plugin Author

    Posted 1 year ago #

    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.

  4. toes21
    Member
    Posted 1 year ago #

    Genius!
    cant believe I didn't try changing it to span.
    Great work!

  5. bythegram
    Member
    Plugin Author

    Posted 1 year ago #

    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.

  6. toes21
    Member
    Posted 1 year 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.

  7. bythegram
    Member
    Plugin Author

    Posted 1 year ago #

    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.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic