Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Landscape format images have blurry thumbnails #375

Closed
Letiliel opened this issue Mar 19, 2023 · 5 comments
Closed

[Bug]: Landscape format images have blurry thumbnails #375

Letiliel opened this issue Mar 19, 2023 · 5 comments
Labels
bug-report Something might not be working

Comments

@Letiliel
Copy link

What happened?

I uploaded images for model reviews and all 16:9 ones get blurry thumbnails.
From looking at other images it seems the wider the image, the blurrier the thumbnail.

Steps to reproduce the problem

Check the reviews on this model's page for example : https://civitai.com/models/7241/mix-pro-v3
Look closely, some thumbnails are sharp, others are a bit or quite blurry.

What should have happened?

--

What platforms do you use to access the site?

No response

What browsers do you use to access the site?

No response

Additional information, context and logs

No response

@Letiliel Letiliel added the bug-report Something might not be working label Mar 19, 2023
@WASasquatch
Copy link

WASasquatch commented Mar 19, 2023

No blur here. You probably didn't wait long enough and were probably seeing the placeholder image (which is just a blur) before the actual image downloaded.

@JustMaier
Copy link
Contributor

From looking at other images it seems the wider the image, the blurrier the thumbnail.

We use scaling within Cloudflare Images to size everything down to 400px wide inside the review preview carousels, they then get scaled down further by your browser to 300px min based on how many review tiles we're showing. There are a few things that could be happening here:

  1. Really wide images scaling down to 400px might lose detail
  2. The browser isn't scaling 400px to 300px appropriately

Either way, when I browse that page I don't see anything particularly blurry, would you mind sharing a few screenshots so I can make more sense of how blurry you mean?

@Letiliel
Copy link
Author

Letiliel commented Mar 19, 2023

blur
Here the top one is blurry, the bottom one is sharp.

And I know what's happening : the images get resized to 400px width as you said, then they get displayed as cropped squares of 400*400. So for landscape ratio images, since their height is less than 400px after the resize, they actually get zoomed in to fit in the 400px height. And zoomed in means blurry.

Solution: resize landscape ratio images to 400px height not width

@WASasquatch
Copy link

WASasquatch commented Mar 20, 2023

Honestly images should be 2-4x the actual viewing size for retina displays. Using AVIF will cut out on size substantially. We're long past the ages of 54k dialup and needing such tiny images.

@manuelurenah
Copy link
Contributor

Hey all, with the newest release on Civitai, this should have been sorted out. Please let us know if there are any issues with it again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-report Something might not be working
Projects
None yet
Development

No branches or pull requests

4 participants