max-holland / Peeves

A free premium ghost theme
MIT License
110 stars 30 forks source link

Flickr Embeds incorrectly sized #9

Closed mattdperkins closed 10 years ago

mattdperkins commented 10 years ago

I've just been going thru older posts, and about to post some new ones when I found that the flickr embeds are being sized strangely. You can see it here:

http://www.mattsrespite.com/2014/01/14/2-weeks-on-ometepe/

I tested with the casper theme and the embeds are shown correct.

mattdperkins commented 10 years ago
<object width="800" height="600"> <param name="flashvars" value="offsite=true&lang=en-us&page_show_url=%2Fphotos%2F96928432%40N05%2Fsets%2F72157639100799046%2Fshow%2F&page_show_back_url=%2Fphotos%2F96928432%40N05%2Fsets%2F72157639100799046%2F&set_id=72157639100799046&jump_to="></param> <param name="movie" value="https://www.flickr.com/apps/slideshow/show.swf?v=143270"></param> <param name="allowFullScreen" value="true"></param><embed type="application/x-shockwave-flash" src="https://www.flickr.com/apps/slideshow/show.swf?v=143270" allowFullScreen="true" flashvars="offsite=true&lang=en-us&page_show_url=%2Fphotos%2F96928432%40N05%2Fsets%2F72157639100799046%2Fshow%2F&page_show_back_url=%2Fphotos%2F96928432%40N05%2Fsets%2F72157639100799046%2F&set_id=72157639100799046&jump_to=" width="800" height="600"></embed></object>
max-holland commented 10 years ago

Hey Dude,

I'm using the same plugin(fitvid.js) as casper uses so it should be the same here is what it looks like for me on Peeves:

screen shot 2014-06-19 at 07 16 01

And here is what it looks like for me on Casper:

screen shot 2014-06-19 at 07 24 47

It looks pretty much the same for me. What does it look like for you?

mattdperkins commented 10 years ago

Max,

Now that is odd - I am using chrome on Linux, but see the same thing in Firefox. I just set up a chrome profile with no extensions and still see the same.

I also have a couple of other themes installed ("DayandNight, and "The Shell"), in all of them the flickr embeds are showing correct.

I also downloaded a fully vanilla copy of Peeves last night and see the same thing.

Now I'm at a bit of a loss as to what to check, but thanks for looking at it. I really didn't expect it to look fine for you.

here is what I am seeing with Peeves: s peeves

And here is what I see with Casper: casper

max-holland commented 10 years ago

Hey Dude,

Luckily the same issue was happening in IE11. I've committed the css change. If you could download it and test it that would be great! :)

https://github.com/mholland1337/Peeves/archive/master.zip

I'll release the fix if your happy with it :)

mattdperkins commented 10 years ago

Tested and it works - thanks for the quick fix :) I'm going to have to keep using the unfixed version until tomorrow so I can copy over some customizations, but forked your project yesterday to make this easier.

max-holland commented 10 years ago

No worries dude!