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

Show all events Leaflet - Frontend #28

Closed
wants to merge 2 commits into from
Closed

Conversation

mick
Copy link

@mick mick commented Sep 30, 2019

Since we can fetch all events quickly on load with PR #27 Let's display them all on the map. This PR isnt meant to be merged Its just a quick pass at what map with all events looks like, and if leaflet will be able to handle this number of custom markers.

Screen Shot 2019-09-30 at 12 47 46 PM

It's great to see all the events across the Map. But you'll notice if/when you run this the map is SLOW. Leaflet + custom markers + a lot of events makes the interaction super slow, basically unusable.

I see we already need to figure out switching map tile providers in #25 This is another good reason to upgrade to mapbox-gl with that switch.

cc @jasonkalmeida

@mick
Copy link
Author

mick commented Oct 1, 2019

closing this in favor of #29

@mick mick closed this Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant