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

when returning to searched place, re-open suggestions #426

Open
wbazant opened this issue Jul 15, 2024 · 3 comments
Open

when returning to searched place, re-open suggestions #426

wbazant opened this issue Jul 15, 2024 · 3 comments
Labels
enhancement New feature request

Comments

@wbazant
Copy link
Contributor

wbazant commented Jul 15, 2024

Say we searched for a place, moved thee map, and want to come back to the populated search box:
Screenshot from 2024-07-15 16-09-34

when it comes back to focus, we should reopen the suggestions.

Extra fun to develop because the component can disappear off the map and get re-rendered.

Full test procedure for when this is fixed:

  1. search for a text, suggestions should open
  2. select first suggestion: map should pan and zoom to that place and cross should show up and suggestions should disappear
  3. return to the searched text. suggestions should reopen
  4. repeat 2.
  5. click on location, then dismiss it. searched text should still be there, but no suggestions
  6. repeat 3.
  7. repeat 2.
@ezwelty
Copy link
Collaborator

ezwelty commented Aug 16, 2024

@wbazant I believe this is now done in #419, as described? I'll copy here my (low priority) comment from there (#419 (comment)):

I find the crosshair icon for the selected search result confusing because we already use that symbol for device location. How about either making all of the others a simple marker icon (without the filled circle, see maps.google.com) and the selected one with a circle to indicate selection? Or use a clock icon (i.e. "previously selected")

@wbazant
Copy link
Contributor Author

wbazant commented Aug 17, 2024

I agree with you, the icon means something else in our app and shouldn't be used. The icon choice was always there in the code but not working due to a bug, I thought it was quite nice when I fixed the bug but didn't think much of it.

How about only differentiating the icon with color:
Screenshot from 2024-08-17 15-15-02

a95f3c0

@ezwelty
Copy link
Collaborator

ezwelty commented Aug 17, 2024

@wbazant That's fine. But if we want to dive into design, then my preference would be a grey map box icon for the results (better matching the design of the inactive track-location icon), a grey history box icon for the previously-selected result(s) in the dropdown, and none of those blue arrows on the right.

image

My point earlier was that when the selected result isn't marked on the map with a crosshair, it made less sense to me to show it as selected. The history icon seems more general to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature request
Projects
None yet
Development

No branches or pull requests

2 participants