Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Zoom map to facility location on selecting its details page from the facilities tab text list #244

Closed
kellyi opened this issue Mar 3, 2019 · 1 comment
Assignees
Milestone

Comments

@kellyi
Copy link
Contributor

kellyi commented Mar 3, 2019

Overview

#237 restores a text listing of facilities search results in a new "Facilities" tab in the sidebar. Each list item links to its details page.

In the legacy implementation, clicking on a facility to see its details would also cause the map to zoom in to its marker. We should restore that functionality.

We should also have the map zoom to the facility location in cases when the user has visited the facility route directly, arriving at the application via the /facilities/oar_id route. In this case, the facility location data loads after the component has loaded, so the zoom will need to happen when the facility data returns successfully from the API.

Describe the solution you'd like

On clicking the facility link in the sidebar's facilities tab, have the map zoom to the facility marker on the map.

On visiting the /facilities/oar_id route directly, the map should zoom to the facility marker when the facility data has returned from the API.

When clicking on a marker on the map to see facility data, the map should not zoom at all.

Requires #209
Requires #264

@kellyi
Copy link
Contributor Author

kellyi commented Mar 3, 2019

See also #128

@jwalgran jwalgran added queue and removed 1 labels Mar 7, 2019
@kellyi kellyi added in progress and removed queue labels Mar 13, 2019
@kellyi kellyi self-assigned this Mar 13, 2019
kellyi added a commit that referenced this issue Mar 14, 2019
…ers-and-zoom

Adjust map style and behaviors

Connects #302
Connects #244
Connects #128
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants