Skip to content

Conversation

@zhang-zimin
Copy link

Added an entry for MapboxImagery to the localization JSON files.

Added an entry for MapboxImagery to the localization JSON files.
@zhang-zimin
Copy link
Author

image

Localized WellKnownBaseMaps are missing Mapbox entries.

Copy link
Member

@aruniverse aruniverse left a comment

Choose a reason for hiding this comment

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

lgtm; but missing rush change files

"satellite-layerRoadmap": "Google Maps: Aerial Imagery with labels",
"roadmap": "Google Maps: Streets"
},
"MapBoxImagery": {
Copy link
Member

Choose a reason for hiding this comment

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

Should this be "MapBoxImagery", or "MapboxImagery" with a lowercase "b"? I have been using the test viewer to try to reproduce the fix locally with your fork, but am still unable to see the correct localization with this change.

I suspect it should be lowercase because the format ID in MapLayerSettings.ts is "MapboxImagery". Do you have a screenshot of this change fixing the UI?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants