Skip to content
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.

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

OpenStreetMap attributions #781

Open
albanm opened this issue Nov 15, 2024 · 3 comments
Open

OpenStreetMap attributions #781

albanm opened this issue Nov 15, 2024 · 3 comments

Comments

@albanm
Copy link

albanm commented Nov 15, 2024

As far as I can see the metadata generated by tilemaker does not include attribution info.

It is always possible to add this afterwards or on the client side. But the result is that a "by default" approach of using tilemaker + maplibre for example results in not following the openstreetmap and openmaptiles requirements.

Should not the attribution be included as expected by the specs ?

@systemed
Copy link
Owner

Interesting question! If (say) Maplibre JS displays the attribution metadata by default then, yes, we should - I hadn't realised it does that. It should just be a matter of adding the relevant line to "settings" -> "metadata" in config-openmaptiles.json, I think.

@albanm
Copy link
Author

albanm commented Nov 15, 2024

Maplibre uses the attribution contained in the tileset metadata automatically (https://github.com/maplibre/maplibre-gl-js/blob/87486a5ef2085e600e8fa4e31252629dd8488dcd/src/ui/control/attribution_control.ts#L151). And the attributionControl option is active by default.

@systemed
Copy link
Owner

That's great. I'd be very happy to receive a PR for this or failing that I'll look at it when I get the proverbial spare minute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants