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

Migrate to MapLibre GL JS #71

Merged
merged 4 commits into from
Jun 24, 2021
Merged

Conversation

webberig
Copy link
Contributor

Fixes #70

Replaces MapBox GL references with MapLibre GJ. Changed demo and viewers page.

Could use some feedback regarding following pages:

docs/style/mapbox-gl-style-spec.md

I think the style spec is still MapBox GL, added a note regarding the Javascript library to clarify MapBox style spec vs MapLibre JS.

/inspect.md

This page uses a copy of the Mapbox library /inspect/mapbox-gl.js and some custom inspect JS file. Didn't dare to touch this. Can't seem to find a reference towards this page though...

languages pages (ie. https://openmaptiles.org/languages/az/)

These pages load a Javascript file https://cdn.klokantech.com/openmaptiles-language/v1.0/openmaptiles-language.js which refers a globally defined mapboxgl. This returns an error when I migrate to MapLibre GL and it breaks the multilingual demo. This Javascript file is not in this project, so not sure how to handle this...

@daliborjanak daliborjanak merged commit 4db4c8f into openmaptiles:master Jun 24, 2021
@daliborjanak
Copy link
Collaborator

Merged. Thanks for contributing. We will update /languages/ later

@webberig webberig deleted the maplibre branch June 27, 2021 13:27
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

Successfully merging this pull request may close these issues.

Migration to MapLibre GL
3 participants