Migrate the edit location page from leaflet to Maplibre #6675
+45
−49
Conversation
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
There was a problem hiding this comment.
Pull request overview
This PR successfully migrates the edit location page from Leaflet to MapLibre GL JS, modernizing the mapping library while maintaining functionality. The migration includes coordinate system adjustments, API method updates, and improvements to the zoom behavior with exponential easing.
Key changes:
- Replaced Leaflet-specific map controls with MapLibre equivalents (NavigationControl, GeolocateControl)
- Updated coordinate handling from Leaflet's lat/lng order to MapLibre's lng/lat order throughout
- Disabled rotation features (roll, drag rotate, pitch) for better location selection UX
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This commit migrates the user home location page (the "Edit" tab on the user profile) from Leaflet to MapLibre GL JS. This is part of a wider migration to MapLibre across the website. The change follows the pattern established in the dashboard migration.
CommanderStorm
force-pushed
the
migrate-user-location-edit
branch
from
4f649fd to
7a8134e
Compare
CommanderStorm
commented
Jan 4, 2026
1ec5
reviewed
Jan 5, 2026
CommanderStorm
force-pushed
the
migrate-user-location-edit
branch
from
0d419b3 to
3aa7d53
Compare
hlfan
reviewed
Jan 5, 2026
…le between the secondary maps
CommanderStorm
force-pushed
the
migrate-user-location-edit
branch
from
3aa7d53 to
a801f7f
Compare
Member
|
For crying out loud I thought I had turned off copilot access to this repository! If anybody can navigate the github documentation better than me to find a way of getting rid of it please let me know... |
hlfan
approved these changes
Jan 6, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Description
This PR migrates the edit location page from leaflet to maplibre.
This has two intentional changes:
I can change this to the previous behaviour, but that did not seem entirely intentional.
I disabled rotation, pich and yaw for this one, since for selecting a location you likely want this.
How has this been tested?
Via docker, manually starting the map, interactig with it.
Locations get set appropriately, all buttons work.