A user uploaded two similar images, one 1172x103 and the other 1180x82 pixels. I have my thumbnailing sizes set to 350x200 so these were both too big and resized to 350x30 and 350x24 pixels respectively. When the post is viewed the first image is lightboxed but the second isn’t.
I suspect this may be related to a bug that was fixed towards the end of last year…
It makes a statement that I think has been coded in…
It should probably need to exceed both dimensions, as opposed to just one – I can’t quite picture a use case where you would want an image that’s, say, 690×10 pixels (on the page) to be lightboxed.
I think the bug made lightboxing mandatory for any image bigger than 100x100, so I suspect the second image doesn’t get a lightbox because it’s intial height was less than 100 pixels. However, the resized image is now too small at 350x24 pixels for the text on it to be read and there’s no lightbox to allow the original image to be viewed.
This is still an issue after the recent Lightbox changes. I’m still getting images that are resized to be smaller that then don’t get lightboxed. is there any chance that this could be fixed so that anything that gets resized is also lightboxed?
I’ve tried but I can’t replicate it here. I suspect meta uses the default values for max_image_width (690) and max_image_height(500). My max width is set to 350 and max height to 200.
That’s highlighted another problem. As a test I tried changing my sizes back to the defaults of 690 and 500, and whilst the values in the settings now show 690x500 when I upload images they are still resized to the old 350x200 maximum sizes. I’ve tried restarting Discourse and that hasn’t brought the default values into use.
New uploads of the same files. Not sure if that makes a difference. They get new upload names, but perhaps there’s only one underlying copy if there’s some form of deduplication going on using a checksum for the file that’s being uploaded?