[openstreetmap/openstreetmap-website] Migrate the edit location page from leaflet to Maplibre (PR #6675)
Marwin Hochfelsner
notifications at github.com
Mon Jan 5 08:54:16 UTC 2026
@hlfan commented on this pull request.
> + map = new maplibregl.Map({
+ container: "map",
+ style: OSM.MapLibre.Styles.Mapnik,
attributionControl: false,
- zoomControl: false
- }).addLayer(new L.OSM.Mapnik());
+ locale: OSM.MapLibre.Locale,
+ rollEnabled: false,
+ dragRotate: false,
+ pitchWithRotate: false,
+ bearingSnap: 180,
+ maxPitch: 0,
+ center: OSM.home ? [OSM.home.lon, OSM.home.lat] : [0, 0],
+ zoom: OSM.home ? defaultHomeZoom : 0
+ });
An extra class for configuration seems overkill, there are simpler ways to keep it DRY:
```suggestion
map = new maplibregl.Map(OSM.MapLibre.defaultSecondaryMapOptions);
```
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6675#discussion_r2660729636
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6675/review/3625968757 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260105/9be1f32a/attachment.htm>
More information about the rails-dev
mailing list