Low Resolution Flickr/500px Onebox

Howdy all!

Just ran into this on my instance (v1.8.0.beta1 +33) at https://discuss.pixls.us.

Oneboxing of images from image sites like Flickr and 500px are using low-resolution thumbnails:

Flickr image:
Mairi Mid-Air (In the Lane) | At the pre-LGM photowalk in Fi… | Flickr


500px image:
Mairi B&W by Pat David - Photo 169440993 / 500px


Imgur album:
http://imgur.com/a/FPk70
http://imgur.com/a/FPk70


Imgur image:
http://imgur.com/eiomPF8
http://imgur.com/eiomPF8

2 Likes

I agree. Added to my list :pencil:

5 Likes

That should fix it :wink:

https://github.com/discourse/onebox/commit/15fae6cc6f595e6f753afa585a25f53bed6e6782

https://github.com/discourse/onebox/commit/731a93a5fc35fd58d0d42492be19d9a10ddab9e7

5 Likes

I’ve noticed some odd scaling problems with an xkcd onebox link on my instance. Testing if it happens here as well (and if so, there’s a weird scaling issue with xkcd onebox embeds! :slight_smile: ).

2 Likes

The good news is, on email the ratio is correct.

–Edit–
I think it is fixed in latest master. Just tested it.

It looks like the xkcd image dimensions are swapped.
i.e. height is assigned to width and width is assigned to height.

Is the same onebox code used for both flickr and xkcd?

This is still a problem for Flickr’s shortened URL’s. Example:

https://flic.kr/p/T3UqgH

2 Likes

This appears to still be a problem with xkcd links for instance:

(The preview does show the correct aspect).

Here is the rendered result just now:

3 Likes

It is due to fixed height requirement on onebox, @zogstrip has it on his list.

3 Likes

Awesome, thank you very much (both for this and in general. :slight_smile: )!

I have fixes for the XKCD and Flickr shortened URLs in the latest builds of Discourse!

6 Likes