Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Update label languages for Mapbox Streets v8.3 #173

Merged
merged 1 commit into from
Feb 21, 2020

Conversation

1ec5
Copy link
Contributor

@1ec5 1ec5 commented Feb 21, 2020

Mapbox Streets source v8.3 adds names in Traditional Chinese and Vietnamese, according to this documentation. With this change, the -[MGLStyle localizeLabelsIntoLocale:] and -[NSExpression mgl_expressionLocalizedIntoLocale:] methods can now localize text into these languages (automatically by default if the system’s Preferred Language setting is set to one of them).

Fixes #169.

/cc @mapbox/maps-ios @lukasmartinelli @jcsg @lloydsheng

@1ec5 1ec5 added enhancement New feature or request localization labels Feb 21, 2020
@1ec5 1ec5 added this to the release-vanillashake milestone Feb 21, 2020
@1ec5 1ec5 requested a review from a team February 21, 2020 01:10
@1ec5 1ec5 self-assigned this Feb 21, 2020
@@ -89,17 +89,17 @@ @implementation MGLVectorTileSource (Private)
https://www.mapbox.com/vector-tiles/mapbox-streets-v8/
*/
static NSArray * const MGLMapboxStreetsLanguages = @[
@"ar", @"de", @"en", @"es", @"fr", @"ja", @"ko", @"pt", @"ru", @"zh",
@"zh-Hans",
@"ar", @"de", @"en", @"es", @"fr", @"ja", @"ko", @"pt", @"ru", @"vi",
Copy link
Contributor Author

@1ec5 1ec5 Feb 21, 2020

Choose a reason for hiding this comment

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

-[NSExpression mgl_expressionLocalizedIntoLocale:] has logic to fall back from zh-Hant to zh if the style happens to be using Streets source v7: mapbox/mapbox-gl-native#12387.

Copy link
Contributor

@captainbarbosa captainbarbosa left a comment

Choose a reason for hiding this comment

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

Tuyệt vời!

謝謝!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request localization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It is invalid to switch traditional languages
2 participants