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

Value of minimap in validation UI #3593

Closed
jonfroehlich opened this issue Jul 10, 2024 · 2 comments · Fixed by #3604
Closed

Value of minimap in validation UI #3593

jonfroehlich opened this issue Jul 10, 2024 · 2 comments · Fixed by #3604

Comments

@jonfroehlich
Copy link
Member

We have long discussed adding a minimap to the validation UI, which @misaugstad then recently added (thanks @misaugstad); however, now that it's there, I question the value of the information it provides. How does it help with the validation task? Currently, it seems minimally helpful and not worth the tradeoff in pixels (it takes up screen real estate and can cause occlusion problems with seeing a user's label #3582).

I think, in my mind, I was envisioning that the user could see street names and more context about the area they were validating—so perhaps if we zoom out on the minimap a bit it could help.

But as now, it seems minimally useful.... and this is just my first impression/reaction to finally having the minimap available in validation (and I'm surprised by my reaction).

image

@misaugstad
Copy link
Member

I did play around with the zoom level and road labels a bit, and was struggling to find a way to get more info onto the minimap without it getting too crowded or hard to read, or by making the minimap bigger. And the minimap I added is actually significantly bigger than the one in our mocks. But I was pressed for time, so I could play around with it a bit more. There may have been an issue with not having control over the text size for the road labels?

We could also use Mapbox instead of Google Maps here, as I expect that we would have more control over it. But it would mostly mean starting over on the minimap, so we should carefully consider if it's worth it.

@misaugstad
Copy link
Member

We talked about this during our 1:1 and decided that the minimap doesn't provide nearly enough value given the space it takes up on the screen. We've both struggled to find situations where the additional context is helpful. So the plan is to remove the minimap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: done / on test servers
Development

Successfully merging a pull request may close this issue.

2 participants