StrandedKitty / streets-gl

🗺 OpenStreetMap 3D renderer powered by WebGL2
http://streets.gl
MIT License
658 stars 49 forks source link

Saved Places - Go: Falsifies angles and distance to default values 45,0,1500 #121

Closed DerKarlos closed 1 year ago

DerKarlos commented 1 year ago

Describe the bug The UI Saved Places stores the actual position exactly. One can copy the URL and put it in the browser address line. Impressing. But: Using Go sets the angles and distances to default values 45.00,0.00,1500.00

Expected behaviour It should be: https://streets.gl/#53.55496,9.99391,5.25,254.00,1869.90 not this one: https://streets.gl/#53.55496,9.99391,45.00,0.00,1500.00

Map location any

Screenshots -/-

System information

Additional context Streets GL is a great tool!

DerKarlos commented 1 year ago

Meanwhile, I branched StrandedKitty/streets-gl and by the README, I could set up the development environment, build, run and debug the Frontend. Great job! After exploring the modules and sources, I could see, what's missing to "Go" to the stored state:

The "Go" button calls goToLatLon/setLatLon, which sets the defaults. So I added goToState: (lat: number, lon: number, pitch: number, yaw: number, distance: number) => void; and setState(lat, lon, pitch, yaw, distance)

In SavedPlaceParams and addNewPlace I added pitch, yaw and distance. I did NOT add them to addNewPlace, that would mean no backward compatibility. I even removed lat, lon because it is redundant in .link. In useLayoutEffect, all values are taken from .link now.

Pull Request #150

DerKarlos commented 1 year ago

A very fast merge, thanks. And it works - on the main page