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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Webmention avatars are sometimes very large #126

Open
zachleat opened this issue Jun 10, 2019 · 4 comments

Comments

Projects
None yet
2 participants
@aaronpk

This comment has been minimized.

Copy link
Owner

commented Jun 10, 2019

256px is the smaller resized version. webmention.io will resize all avatars down to 256px regardless of whether they come from Twitter accounts or individual blogs.

The avatars don't always come from Twitter so I can't rely on using the twitter API for fetching profile images.

@zachleat

This comment has been minimized.

Copy link
Author

commented Jun 10, 2019

A ha鈥攃ruised through the source a bit and found https://github.com/aaronpk/ca3db-ruby

I wonder if it鈥檚 possible to entertain reducing this max size? Is this a change you might be comfortable with? Or a query param to opt-in to save some bandwidth?

Your site renders these at 36脳36 (which would need a 7脳 retina screen to see 256脳256 level of detail, but I believe that鈥檚 well beyond what the human eye is capable of鈥攏ot counting for page zoom, of course)

@aaronpk

This comment has been minimized.

Copy link
Owner

commented Jun 10, 2019

yeah I show them kinda small, but some people show them a lot bigger.

One trick here is that the resizing is done system-wide, not per user account. So I can't make it resize avatars for you differently than for other users of webmention.io. The only way around that would be to let you put in your own S3 credentials to save the resized avatars to your own S3 account, and then I could let you also choose the dimensions. Is that something you'd be interested in?

There is already a checkbox in the settings where you can opt out of the resizing/archiving entirely, for example if you wanted to handle resizing on your own.

@zachleat

This comment has been minimized.

Copy link
Author

commented Jun 10, 2019

Yeah that would be great! Just as info this request is not high priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can鈥檛 perform that action at this time.