Low Resolution Flickr/500px Onebox

AI-generated summary

The discussion revolves around the issue of low-resolution thumbnails when oneboxing images from Flickr and 500px. Patrick_David initially reported the issue and shared examples from Flickr and 500px. zogstrip acknowledged the issue and later posted a commit on GitHub that should fix it zogstrip.

However, the issue persisted for some users, including Patrick_David who also reported problems with xkcd onebox links. bekicot mentioned that the issue might be fixed in the latest master version.

Further investigation by Mittineague revealed that the xkcd image dimensions were swapped, and SimeonGriggs pointed out that the issue still existed for Flickr’s shortened URLs. Patrick_David shared another example of the problem with xkcd links.

codinghorror explained that the issue is due to a fixed height requirement on onebox, which zogstrip had on his list. Finally, eviltrout announced that fixes for the XKCD and Flickr shortened URLs are available in the latest builds of Discourse.

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

Imgur


500px image:
500px


Imgur album:
Imgur: The magic of the Internet
Imgur: The magic of the Internet


Imgur image:
Imgur: The magic of the Internet
Imgur: The magic of the Internet

2 лайка

I agree. Added to my list :pencil:

5 лайков

That should fix it :wink:

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

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

5 лайков

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 лайка

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

Imgur

2 лайка

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 лайка

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

3 лайка

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 лайков