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

Dynamic camera, first pass #601

Closed
4 tasks
ericrwolfe opened this issue Dec 15, 2017 · 2 comments
Closed
4 tasks

Dynamic camera, first pass #601

ericrwolfe opened this issue Dec 15, 2017 · 2 comments
Labels
feature New feature request.
Milestone

Comments

@ericrwolfe
Copy link
Contributor

ericrwolfe commented Dec 15, 2017

Android version of mapbox/mapbox-navigation-ios#773

The camera should:

  • Be more dynamic, communicating maneuver urgency by camera distance/zoom level
  • Be timed with voice announcements: what's communicated on screen should match audio cues
  • Provide ample context for maneuvers

As a rough first pass, we should implement basic stepped camera movements achieving the following:

  • Triggered by/synchronized with voice announcements
  • On the completion announcement of a maneuver: zoom out to provide context for long steps with long periods of inactivity
  • At the first approach maneuver announcement (In 2 miles...): zoom to show the upcoming maneuver on the map, whether by a fixed zoom level based on distance, or a zoom to fit function.
  • At the final turn maneuver announcement: zoom in closely to our current default zoom level

Once this basic set of triggers/logic is in place, we can experiment with more sophisticated zoom logic, camera tilt, anchor points, etc.

cc @mapbox/navigation-android

@zugaldia
Copy link
Member

cc: @tobrun who looked into camera animation in the context of mapbox/mapbox-gl-native#9808 (comment).

@davols
Copy link

davols commented Jan 4, 2018

Don't want to hijack this at all. Thought about the camera and its zoom levels the other day. But having basic camera movements in relation to maneuvers sound great. My two thoughts were:

  • Allow the user to set the zoom level (don't override it when pressing re-center)
  • Zoom levels depending on speed

Both might be fixed with camera movement on voice announcement, but as of now the first one feels quite important since it's really annoying from a UX perspective that it takes over the zoom at re-center.

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

No branches or pull requests

4 participants