small fixes for oembed:

- make container inline-block to fit content (for overlay buttons to be centered)
- fix javascript ("src" instead of "href" for iframes), also disambiguate variable names
This commit is contained in:
Florian Staudacher 2012-03-15 17:39:02 +01:00
parent b3868fc6dd
commit 3d1925ec63
2 changed files with 6 additions and 5 deletions

View file

@ -3,7 +3,7 @@ app.views.Content = app.views.StreamObject.extend({
events: {
"click .oembed .thumb": "showOembedContent"
},
presenter : function(){
return _.extend(this.defaultPresenter(), {
text : app.helpers.textFormatter(this.model),
@ -37,10 +37,10 @@ app.views.Content = app.views.StreamObject.extend({
showOembedContent: function() {
var oembed = $(this.el).find(".oembed");
var embedHTML = $( this.embedHTML() );
var paramSeparator = ( /\\?/.test(embedHTML.attr("href")) ) ? "&" : "?";
embedHTML.attr("src", embedHTML.attr("src") + paramSeparator + "autoplay=1");
oembed.html( embedHTML );
var insertHTML = jQuery( this.embedHTML() );
var paramSeparator = ( /\?/.test(insertHTML.attr("src")) ) ? "&" : "?";
insertHTML.attr("src", insertHTML.attr("src") + paramSeparator + "autoplay=1");
oembed.html( insertHTML );
}
});

View file

@ -1830,6 +1830,7 @@ ul#press_logos
.collapsible
.oembed
:background url('/images/ajax-loader2.gif') no-repeat center center
:display inline-block
.thumb
:position relative