Leaflet to maplibre migration#104
Open
flockhopperdev wants to merge 4 commits intoFoggedLens:masterfrom
Open
Conversation
Replace Leaflet with MapLibre GL for WebGL-powered vector tile rendering, improving performance and enabling 3D map capabilities. Key changes: - Replace LeafletMap.vue with MapLibreMap.vue using MapLibre GL JS - Replace tiles store with geojson store for GeoJSON-based data loading - Add alpr_geojson serverless function and Terraform module for generating and serving GeoJSON data from S3 - Update DFMapPopup, Map view, and apiService for new architecture - Remove unused Leaflet dependencies, add MapLibre GL JS - Add migration documentation in docs/plans/ Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Include the full migration spec and per-file change summary in docs/plans/ for PR reviewers and future reference. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Fix layer toggles staying visible on mobile by removing Vuetify d-flex
utility class (!important) that overrode v-show, using scoped CSS instead
- Fix GPS button crash by converting geolocation [lat, lng] tuple to
{lat, lng} object expected by center ref and updateURL
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed:
boundaries) now use MapLibre's native GeoJSON sources and style layers instead
of Leaflet overlays
boundary toggles
refer to /maplibre migration docs for detailed changes & deployment instructions
I have tested locally, works great. Instructions for backend setup in setup.md
Would recommend setting up vector tiles before deploying this to production, the map is much snappier now so people will probably be consuming way more tiles