Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate Mapbox Map into Web App #72

Closed
14 of 15 tasks
Tracked by #100
mantuok opened this issue Nov 4, 2024 · 0 comments · Fixed by #88
Closed
14 of 15 tasks
Tracked by #100

Integrate Mapbox Map into Web App #72

mantuok opened this issue Nov 4, 2024 · 0 comments · Fixed by #88
Assignees

Comments

@mantuok
Copy link
Collaborator

mantuok commented Nov 4, 2024

Context

Integrate Mapbox map into web application. This integration should align with existing design: Figma

Definition of Done

  • The Mapbox map is successfully integrated into the specified section of the web app.
  • The map displays correctly across all supported devices.
  • The integration follows the design requirements, including the following defaults that the map ought to load with:
    • center of the map at: -122.463733, 37.777448*
    • Tdefault zoom at: 11*
    • minimum zoom at: 10.5*
    • maximum zoom at: 15*
    • ability for the user to pan and zoom at will across devices
    • map ought to be responsive** to the size of the page.
      *these may change after eg visualizing with soft story data
      **possibly fluid and edge-to-edge pending design decision
  • The map displays a pin with mock coordinates.

Engineering Details

  • Install Mapbox GL JS library
  • Create a dedicated component for the Mapbox map
  • Utilize Chakra UI's layout components to ensure the map is responsive and integrates seamlessly within the overall design.
  • Test on various screen sizes to confirm adaptability.
  • Write unit tests for the map component using Jest. #98
@nickvisut nickvisut self-assigned this Nov 5, 2024
@nickvisut nickvisut moved this from Todo to In Progress in QuakeSafe Project Nov 7, 2024
@nickvisut nickvisut linked a pull request Nov 23, 2024 that will close this issue
5 tasks
@nickvisut nickvisut mentioned this issue Nov 24, 2024
3 tasks
@nickvisut nickvisut moved this from In Progress to In Review in QuakeSafe Project Nov 24, 2024
@github-project-automation github-project-automation bot moved this from In Review to Done in QuakeSafe Project Nov 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants