Add map movement

[FlorianK13]

No description provided.

[FlorianK13]

I find it rather hard to understand that you search the adress and then walk along the compass. Probably non IT people will not really understand it?

@kpoeppel Could we maybe set the compass behind an advanced settings button?

[FlorianK13]

Definitely looks better now. Could we have maybe a small ✔️ button in the middle of the arrows to resubmit the simulation? Is there also the possibility to have a small circle in the middle of the map indicating the coordinates (maybe some three js object)? This circle would then be moved by the arrows?

@kpoeppel I can also help in development here if you want.

Edit: Maybe the circle in three js is a feature for later development and not the most important thing at the moment.

[FlorianK13]

Also when I type in random letters as address (like 'aklsjdh') it does not catch the error anymore but throws a mistake in the browser: ERROR loc is null

[kpoeppel]

Improved the movement on the map now to make it more intuitive. There are arrow buttons and the movement is immanent, the simulation is done once the start button is pushed. The perspective now also is kept, unless a new location is typed in.

[kpoeppel]

Also when I type in random letters as address (like 'aklsjdh') it does not catch the error anymore but throws a mistake in the browser: ERROR loc is null

Should be fixed with second latest commit.

[kpoeppel]

Definitely looks better now. Could we have maybe a small heavy_check_mark button in the middle of the arrows to resubmit the simulation? Is there also the possibility to have a small circle in the middle of the map indicating the coordinates (maybe some three js object)? This circle would then be moved by the arrows?

@kpoeppel I can also help in development here if you want.

Add now also the object showing the center. Disappearing over time for clearer view of the simulation.

[kpoeppel]

Fixed now also #5

[FlorianK13]

To make the UI more intuitive I would also just fix the stepsize to 10 and delete the slider from the sidebar? Super nice decaying three js pointer by the way.

[kpoeppel]

Last commit fixed now also Issue #13 .

[kpoeppel]

To make the UI more intuitive I would also just fix the stepsize to 10 and delete the slider from the sidebar? Super nice decaying three js pointer by the way.

I don't know actually. I think it is a good functionality, also "Anzahl an Simulationen" and "Radius Gebäude" is not that intuitive. Rather than removing it, I would recommend to add some explaining hover text?

[FlorianK13]

To make the UI more intuitive I would also just fix the stepsize to 10 and delete the slider from the sidebar? Super nice decaying three js pointer by the way.

I don't know actually. I think it is a good functionality, also "Anzahl an Simulationen" and "Radius Gebäude" is not that intuitive. Rather than removing it, I would recommend to add some explaining hover text?

Maybe we can put all of them behind a Advanced options button within a new PR.

[FlorianK13]

@kpoeppel Another error I found: If I type in an adress outside of Bavaria (like Hauptstraße Stuttgart) the website does not catch that error anymore and instead throws a "Can't read the data of 'the loaded zip file'. Is it in a supported JavaScript type (String, Blob, ArrayBuffer, etc) ?"

[FlorianK13]

I think the PR is ready to merge now @kpoeppel