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

Simplified route view #953

Open
Mygod opened this issue Feb 15, 2024 · 0 comments
Open

Simplified route view #953

Mygod opened this issue Feb 15, 2024 · 0 comments

Comments

@Mygod
Copy link
Collaborator

Mygod commented Feb 15, 2024

Current route view has many issues.

  1. It is impossible to read on ReactMap unless it is the only thing enabled.
  2. If there are multiple routes coming out of the same POI, it is impossible to open the popup for all but the first route.
  3. The current system of having two colored flags does nothing but creating more visual noise.
  4. Even worse, the two flags overlap each other, making it very hard to tap on especially on mobile, even if you zoom all the way in. This is an accessibility issue since every tappable element should have at least 48dp x 48dp hit region or something like that.

I suggest to add a simplified view to make the map less cluttered. Maybe even replacing the old system outright.

  1. Since routes must start/end at a POI (stop or gym), by default the only thing displayed on a map should be a 🚩 icon overlayed onto the starting point or the ending point too if it is reversible, indicating that there are at least one routes starting from this POI.
  2. When a POI is clicked on, all the routes (paths) starting from that POI will be displayed and the user can click on one of the paths to view details about the route.
  3. Ideally the route should be displayed even if the map is being dragged around. In fact, this should be applied to every other element as well: the popup should not be closed just because the associated object was dragged out of the map view.

Fortunately we can just disable routes for now and use Campfire instead. :)

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

No branches or pull requests

1 participant