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

💡 [REQUEST] - Split up data by island instead of showing all at once #106

Open
MichaelTamaki opened this issue Sep 12, 2023 · 1 comment
Assignees
Labels
dev 🦍 Development Team enhancement 🪄 New feature or request question for UX ❓ Further information is needed

Comments

@MichaelTamaki
Copy link

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

Currently, the project is loading all data for all islands at once. And it will draw all overlays all the time. This is a big performance burden, especially since the typical use case is to only show one island at a time.

Idea:

Screenshot 2023-09-11 at 6 55 59 PM
  • Add the ability to select one island to show. Can be both or either:
    • The URL has a path for each island, like https://hawaiizoningatlas.com/oahu
    • A select UI component is added to select an island.
  • Split up the data set for each island. Then, load and draw that specific island's data.

This is mean to be very primitive. Lazy loading would be cool but would be tough. This should be a relatively straight forward way to get a performance increase. This might even be a better UX. For example, when an island is selected, it can automatically center and zoom in the map on that island.

Basic Example

See summary.

Drawbacks

  • What happens if you want to view multiple islands at the same time?
    • Have to open up two browser tabs?
    • Add an option to show all islands?

Unresolved questions

No response

@github-actions github-actions bot added the triage 🏥 Needs attention label Sep 12, 2023
@ggordn3r
Copy link
Collaborator

To start with, let's go with the UI idea, hyperlinks for each county that "jump" to center that county at the appropriate zoom level.

@ggordn3r ggordn3r added enhancement 🪄 New feature or request question for UX ❓ Further information is needed dev 🦍 Development Team and removed triage 🏥 Needs attention labels Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev 🦍 Development Team enhancement 🪄 New feature or request question for UX ❓ Further information is needed
Projects
None yet
Development

No branches or pull requests

2 participants