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

Merge map overview and venue map page #149

Open
micuintus opened this issue Oct 27, 2017 · 2 comments
Open

Merge map overview and venue map page #149

micuintus opened this issue Oct 27, 2017 · 2 comments

Comments

@micuintus
Copy link
Owner

Open UX questions:

  1. How does the navigation look like? How to get to that page?
  2. Gray out other items in new page when it is in 'single venue' mode
@jmastr jmastr mentioned this issue Oct 27, 2017
@jmastr jmastr added this to the 3.1 milestone Oct 28, 2017
@jmastr
Copy link
Collaborator

jmastr commented Oct 28, 2017

I thought about it and I still don't think it is necessary to merge those views and that we just should bring back the "get directions" icon in the venue detail view. I think it does not matter to have two maps on the stack. One can always go quickly back to the venue list by pressing on the navigation
items.

This also fixes the issue, how to handle a "press" in 'single venue' mode, because we can not open the venue details again, which we do in the map overview, but we have to switch to a Maps App to get the route calculated.

So in my opinion we should change the ticket to do the following:

  • show "get directions" icon in venue details
  • color the location icon the same way, we do in map overview.

What do you think?

@micuintus
Copy link
Owner Author

No it is not necessary. But if we find a suitable solution where the map pages are merged, this could leave us with a pretty amazing UX, I believe. This is why i think we should leave this ticket open and open a new ticket for the solution you suggested. Lets run with this until we find a nice UX to merge the maps?

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

No branches or pull requests

2 participants