search

stadiamaps / ferrostar

A FOSS navigation SDK built from the ground up for the future
https://stadiamaps.github.io/ferrostar/
Other
179 stars 24 forks source link

Dynamic or Zoom based on speed #175

Open Archdoog opened 3 months ago

Archdoog commented 3 months ago

At speeds above 25 mph/40 kph, our zoom level is too high to see upcoming maneuvers. I found that zooming out 2 increments on the button was the most ideal during several hours of driving on android. We should match iOS to this if needed.

Solution:

Subtract 2 from the default android zoom level and make sure iOS matches that new value.

ianthetechie commented 3 months ago

This will be a lot nicer with #89.

ianthetechie commented 3 weeks ago

@Archdoog this was expressed relative to some point in time whose values I no longer remember ;) Can you have a look again at where the defaults are relative to "optimal?"

Archdoog commented 3 weeks ago

In some PR I adjusted android (and probably matched iOS, though not sure). It's quite nice currently for automotive at around 40-50 mph and passable for lower and higher speed. This current Android value should absolutely remain as is for the fixed default automotive camera.

I'd say for dynamic we could probably zoom 1 or 2 increment in below 35 mph and out above 65 mph. I would expect this to need revision, but testing those values will probably illustrate pretty well if the values are ideal after some use.

Once we define a decent range we can decide if it's good or needs to be fancier.

ianthetechie commented 3 weeks ago

Sound good; thanks! I'll update the title on this ticket and start thinking about the interface for more dynamic customization.