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
Avatars via Gravatar: inform users #1389
Comments
Seems very strange to opt out of a service you've signed up for whose entire purpose is exactly the behavior they got. I absolutely agree that it would be nice if it was obvious that the image comes from gravatar. Maybe just a title attribute? |
The entire purpose of Gravatar was to put an avatar (not necessarily your real face) near a username. I do not see any reason for adding this feature because if someone doesn't want to use a photo they have at least two options: to not use gravatar or to user an alternative email on pypi. Adding a parameter to your real email is enough to hide gravatar (john.doe+pypi@gmail.com will not render the gravatar unless is registered explicitly on gravatar). For the sake of reaching M1 and avoiding feature-creeping, I would vote to close this as wontfix or to move it to another milestone. |
How about just adding some hover text or something on the image so it is a little more discoverable that it comes from Gravatar, to make it more obvious where it's coming from, so people who find it an unpleasant surprise don't waste time hunting around their PyPI settings fruitlessly? |
I'm going to kick this out of launch, I don't think it is really needed as a high priority item and can be added later. |
Closing this as it was implemented in #1444 |
A friend of mine who maintains some Python packages took a look at pypi.io and mentioned:
So I think this is two related tasks:
I'd predict that, if we don't have these set up at launch, at least a dozen people are going to unhappily say "hey I don't want my picture there" or "where's that coming from," so it's worth it to just implement this ahead of launch to save all of us time and frowns. :)
Potentially related to #815.
The text was updated successfully, but these errors were encountered: