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

[Feat] Dynamic map lib config #2678

Merged
merged 3 commits into from
Oct 10, 2024
Merged

[Feat] Dynamic map lib config #2678

merged 3 commits into from
Oct 10, 2024

Conversation

heshan0131
Copy link
Contributor

  • Support dynamic map lib

Copy link

netlify bot commented Oct 7, 2024

Deploy Preview for keplergl2 ready!

Name Link
🔨 Latest commit 3b828cf
🔍 Latest deploy log https://app.netlify.com/sites/keplergl2/deploys/67084c925847880008dd39a3
😎 Deploy Preview https://deploy-preview-2678--keplergl2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@ilyabo ilyabo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

really cool!

Copy link
Collaborator

@ibgreen ibgreen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice. It would be really cool to support multiple basemaps, giving the user a choice, with little cost as they are dynamically loaded.

@@ -5,7 +5,7 @@ import React, {Component} from 'react';
import {polyfill} from 'react-lifecycles-compat';
import classnames from 'classnames';
import styled from 'styled-components';
import MapGLMap, {MapRef} from 'react-map-gl/maplibre';
import {Map, MapboxMap, MapRef} from 'react-map-gl';
Copy link
Collaborator

@ibgreen ibgreen Oct 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, @Pessimistress is separating maplibre into a new repo (since the APIs have forked), so there will be two dependencies down the road and we may need to build a higher level map lib abstraction.

@heshan0131 heshan0131 force-pushed the upstream-dynamic-maplib branch from f4a952e to 767d322 Compare October 10, 2024 19:57
ilyabo and others added 3 commits October 10, 2024 14:52
* Dynamic map lib

* Type fixes

Signed-off-by: Shan He <heshan0131@gmail.com>
Signed-off-by: Shan He <heshan0131@gmail.com>
Signed-off-by: Shan He <heshan0131@gmail.com>
@heshan0131 heshan0131 force-pushed the upstream-dynamic-maplib branch from 4d83c06 to 3b828cf Compare October 10, 2024 21:52
@heshan0131 heshan0131 merged commit 69fc6c6 into master Oct 10, 2024
12 checks passed
@heshan0131 heshan0131 deleted the upstream-dynamic-maplib branch October 10, 2024 22:54
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.

4 participants