Skip to content

Let administrators choose the background map - #54

Merged
trasher merged 1 commit into
developfrom
feature/tilesprefs
Sep 2, 2026
Merged

Let administrators choose the background map#54
trasher merged 1 commit into
developfrom
feature/tilesprefs

Conversation

@trasher

@trasher trasher commented Aug 31, 2026

Copy link
Copy Markdown
Member

Replacing the CARTO URL with OpenFreeMap only moved the hardcoding. The OSMF tile usage policy asks the opposite outright: avoid hard-coding the tile URL, allow switching without a software update. A settings page now offers every provider documented in README.tiles.md, plus one's own address, attribution, maximum zoom and subdomains.

Settings live in Galette preferences under pref_maps_tiles_*, so the plugin needs no table of its own and no database version bump. The provider list stays here rather than in the core schema: tile URLs die, and retiring one has to be a plugin release.

The address is stored as a string rather than a URL. Galette's URL validation rejects a host starting with a brace, which would turn away the {s} subdomain form the OSM-FR and Humanitarian providers rely on.

The raster path no longer pays for MapLibre: the megabyte of GL bundle is requested only when the chosen background is vector. Browsers without WebGL 2 still fall back to raster OpenStreetMap, now taken from the provider list rather than written out again.

Requires galette/galette#952

@trasher
trasher marked this pull request as ready for review September 2, 2026 09:20
A settings page now offers every provider documented in README.tiles.md,
plus one's own address, attribution, maximum zoom and subdomains.

Settings live in Galette preferences under pref_maps_tiles_*, so the plugin
needs no table of its own and no database version bump.

The address is stored as a string rather than a URL. Galette's URL validation
rejects a host starting with a brace, which would turn away the {s} subdomain
form the OSM-FR and Humanitarian providers rely on.
@trasher
trasher merged commit 8a8c2da into develop Sep 2, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant