Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

dropdown usermenu overflow issue #5096

Closed
openstreetmap-trac opened this issue Jul 23, 2021 · 3 comments
Closed

dropdown usermenu overflow issue #5096

openstreetmap-trac opened this issue Jul 23, 2021 · 3 comments
Assignees
@openstreetmap-trac
Labels
Component: website Priority: minor Resolution: fixed Type: defect

Comments

@openstreetmap-trac
Copy link

Reporter: Aerilius
[Submitted to the original trac issue database at 12.10am, Saturday, 11th January 2014]

After creating an account on http://www.openstreetmap.org, the dropdown usermenu shows a long urlencoded path string so that the button covers almost the whole browser width. This is probably because a new user does not yet have a profile image.

Expected behavior: The path string would not be visible, no profile image or a default avatar would be displayed.

Browser: Firefox 26.0

Operating System: Ubuntu 13.10
@openstreetmap-trac openstreetmap-trac self-assigned this Jul 23, 2021
@openstreetmap-trac
Copy link
Author

Author: Aerilius
[Added to the original trac issue at 3.50pm, Saturday, 11th January 2014]

What I reported above is with what a new user was greeted without going through the available settings. I just found that disabling Gravatar works around the issue (which was preselected).
Apart from that it could be thought whether to use [http://arstechnica.com/security/2013/07/got-an-account-on-a-site-like-github-hackers-may-know-your-e-mail-address/ Gravatar] only without the user's consent.

@openstreetmap-trac
Copy link
Author

Author: TomH
[Added to the original trac issue at 7.26pm, Sunday, 12th January 2014]

This shouldn't happen because gravatar should redirect back to the osm URL that we pass as the default, which will then display the default image.

If you look at the image you posted you'll see there is a d= parameter in the link text, which is the URL it should have redirected back to, and testing here seems to show it working for me...

@openstreetmap-trac
Copy link
Author

Author: TomH
[Added to the original trac issue at 7.30pm, Sunday, 12th January 2014]

I've pushed [changeset:67dbde0/rails] to stop the long URL displaying if something goes wrong.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@openstreetmap-trac openstreetmap-trac

Labels
Component: website Priority: minor Resolution: fixed Type: defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@openstreetmap-trac