Skip to content

Resolve the data-layer anchor against the running style #278

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.

Sign up for GitHub

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

Merged
yorickdewid merged 1 commit into from
Jul 25, 2026
Merged

Resolve the data-layer anchor against the running style #278

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 14 additions & 1 deletion src/components/Mapbox/useMapLayers.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,20 @@ export const useMapLayers = function useMapLayers(
// Add ownership fencing
applyOwnershipFilterToLayerSpecification(layerSpecification)

mapInstance.value.addLayer(layerSpecification, import.meta.env.VITE_FUNDERMAPS_NUMBER_LAYER || 'building-number-label-hover')
// Insert data layers below the purple admin-boundary lines (and
// thus below all labels). The anchor is resolved against the
// running style: the env override first, then the boundary
// layer, then the first symbol layer, then simply on top -
// a missing anchor id must never take the whole mapset down
// (the legacy hardcoded 'building-number-label-hover' anchor
// did exactly that after the basemap swap).
const anchorCandidates = [
import.meta.env.VITE_FUNDERMAPS_NUMBER_LAYER,
'fundermaps-municipality',
].filter(Boolean) as string[]
const anchor = anchorCandidates.find((id) => mapInstance.value?.getLayer(id))
|| mapInstance.value.getStyle().layers.find((l) => l.type === 'symbol')?.id
mapInstance.value.addLayer(layerSpecification, anchor)

attachEventHandlers(layerId)

Expand Down
Loading