starfishmod/jquery-oembed-all — pull request #35 by joerussbowman (2012-08-08 15:07:11+00:00)
Title: Several changes I made. Sorry for the 1 big commit, I wasn't tracking the changes in git as I made them originally.

unscatter.com. I've decided to do a pull request in order to
contribute some of the fixes back to project.

1: Pretty sure all shortened links were not working. The issue was
that the url returned by longurl wasn't being set to replace
the current one for the yql query, so the yql query ran on
the short url and that doesn't follow redirects.

2: insertCode, using append, was moving already resolved instances
on the tabs on unscatter.com. Since the same url may come up
for multiple searches I needed the created nodes to be clones
not indivual instances that got moved.

3: I modified the open graph output to put images or videos first
so that I could float left them in css and have the output
look nicer. One improvement that could be done here is to
have the ability pass templates, say underscore ones so
that it's easier to manage display. Or just have the option
to return an object with the data that could be passed to
a template.
4: onEmebed was being overwritten by insertCode if passed as a
setting. While you could add the third argument as the
embedAction function, it didn't make sense that onEmbed
was actually overwritten if passed. I did not remove
embedAction in order to break backwards compatibility but
I would suggest deprecating it.