NetNewsWire icon indicating copy to clipboard operation
NetNewsWire copied to clipboard

Feed Embedded Icons

Open bradr3d opened this issue 2 years ago • 3 comments

Would it be possible to support feed embedded icons (or whatever their official name is) for feeds like OpenRSS?

Example is https://openrss.org/twitter.com/taylorswift13 But any OpenRSS feed has this issue

An idea, if said images are not sized correctly, it to still use whatever method you’re using now as a fallback.

bradr3d avatar Jul 15 '23 16:07 bradr3d

This was discussed a bit on Slack. To recap: Apparently currently if the feed does not guarantee for the image to be square, no image is used. An inappropriate image is not displayed as aspect fit or aspect fill. @brentsimmons replied to this with

The issue with aspect fill is that it can crop things really weirdly — a picture of a person might be all torso and no head, for instances that it can crop things really weirdly — a picture of a person might be all torso and no head, for instance The issue with aspect fit, on the other hand, is that any image very far from square will have to be shrunk down a bunch to fit, which won’t look good

However I personally would prefer an aspect fill image – it would still display a square image which would give some better identification and also keep the visual structure. This seems like a better option to me than no image, even if weirdly cropped.

Kamik423 avatar Jul 17 '23 20:07 Kamik423

I understand NNW trying to optimize feed images. But making a decision to not show the feed's image at all is something different. Can we still achieve the former while not affecting the latter?

markcellus avatar Jul 17 '23 20:07 markcellus

Can it pull the image from the feed if the image is square, and if it isn’t, use its method that it’s using now?

bradr3d avatar Jul 19 '23 13:07 bradr3d