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

Avatar Quality Issue (I missed one) #376

Closed
Nutbolt52 opened this issue Mar 5, 2015 · 4 comments
Closed

Avatar Quality Issue (I missed one) #376

Nutbolt52 opened this issue Mar 5, 2015 · 4 comments

Comments

@Nutbolt52
Copy link
Contributor

Logging this so someone can do it if I never get round to it, but I noticed on the character view the toons on the same api key avatar images are still lower quality. They need changing to _64 image server images.

@mglinski
Copy link
Contributor

mglinski commented Mar 6, 2015

Images now are lazy loaded based on the browser's current viewport and a flag representing a "retina" or hidpi screen. If you are browsing on a device that is hidpi, it will detect that and load x2 images instead of x1 images. Loading a larger image for regular browsers is generally a waste of bandwidth, this behavior attempts to fix both needs.

@Nutbolt52
Copy link
Contributor Author

Yup, true, had forgotton (but seat is much faster now on those bigger pages!). However the image is still shitty quality. It grabs the x32 version from CCPs image servers, then scales it to x64 resolution. It should just grab the x64 res image from CCP. I'm sure peoples bandwidth can cope with the slightly bigger picture :)

@mglinski
Copy link
Contributor

mglinski commented Mar 6, 2015

The image is rendering at 32px for me. Are they supposed to be 64px?

@Nutbolt52
Copy link
Contributor Author

In #354 I went through and increased the quality to pull from ccp to 64x64, 32 just looks crap, no matter how small the image is.

As for your screenshot, err, mine is displaying 64px lol. Which means the 2 avatars actually touch each other.

I think 64px is the right size, but would need some padding or something. So 2 issues here lol. Increase quality grabbed from CCP, and work out why mine displays bigger than yours (im running 0.14)

eve-seat added a commit that referenced this issue Mar 9, 2015
@eve-seat eve-seat closed this as completed Mar 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants