cityofaustin / atd-data-tech

Austin Transportation Data & Technology Services
17 stars 2 forks source link

[Enhancement] Rework geocoder functionality on VZE #15175

Open patrickm02L opened 7 months ago

patrickm02L commented 7 months ago

Description

The current Vision Zero website's mapbox geocoder component is not functioning as expected. Currently, when the map is moved, the geocoder value resets, hindering the ability to search by address.

Additionally, upcoming library upgrades (#14729) are causing the geocoder to malfunction more severely. The objective is to rework the geocoder functionality to ensure it remains stable during map movements, allows easy address searches, and functions correctly after library upgrades.

Requirements

Notes

patrickm02L commented 7 months ago

1/17

@frankhereford can you create a video that shows the behavior that the geocoder is exhibiting?

Geolocater would be address in a branch off of this PR.

frankhereford commented 7 months ago

@patrickm02L You bet! I put this on my PR as well, but here's the details:

Here's the video of the geocoder behavior that prompted me to talk to the group where we decided to disable the geocoder and spin off an issue to fix it back up.

https://github.com/cityofaustin/atd-vz-data/assets/3653206/eeefb3b8-11d3-4237-8131-ddb36beaffe9

It's broken in a similar way in production, except it doesn't drop down the geocoder suggestions, which makes it easy to overlook the regression. It's the new version of the library that causes the existing regression to be disruptive and significantly degrade the usability of the tool.