like so?

we can also increase the height:

Yeah, increasing the height sounds good, so the aspect ratio doesn't get too wide. Could go even taller than what you have there, IMO. Elk doesn't seem to limit the height at all, which seems like a bit much too me. I'd try 4:3 or square as the max aspect ratio.

4:3

square

I think square could work, but I would not force an aspect ratio for landscape images, and just show the whole image instead, i.e. images taller than 1:1 would be scaled up and clipped, while images wider than 1:1 would be scaled down and shown without cropping (maybe up to a minimum height of like 100px or so, after which they should probably be scaled up and cropped to keep them a reasonable aspect ratio).

This looks fixed to me on the latest main:

It just makes the attachment bigger, it does not follow the previous comment #180 (comment) so I left it open (unless it's good enough as is?)

Mmmh, not sure I follow what it doesn't do to be honest :D

Currently the attachment is either cover fitted (the default) or contain fitted (if enabled in settings (letterbox))

images taller than 1:1 would be scaled up and clipped

✔️ cover takes care of it

while images wider than 1:1 would be scaled down and shown without cropping (maybe up to a minimum height of like 100px or so, after which they should probably be scaled up and cropped to keep them a reasonable aspect ratio).

❌ The following image

will look like this

while you probably are looking for something like this

Ah, gotcha! Thanks for explaining, and yes that is what I had in mind :)