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

URL accessor for raster and vector sources #7398

Merged
merged 3 commits into from
Dec 21, 2016

Conversation

1ec5
Copy link
Contributor

@1ec5 1ec5 commented Dec 13, 2016

Added a method to MGLTileSource (and thus MGLRasterSource and MGLVectorSource) that returns the TileJSON URL, if any, that was used to initialize the source. #7377 removed this method, but this PR adds it back in; this time, it works for sources obtained via -[MGLStyle layerWithIdentifier:] too.

The “Labels In” feature in macosapp now avoids localizing sources other than Mapbox Streets v6–7, demonstrating how tantalizingly close we are to implementing #7031.

Working towards #6584. Depends on #7377.

@ivovandongen to review the core parts; @frederoni to review the iOS/macOS parts.

@1ec5 1ec5 added Core The cross-platform C++ core, aka mbgl localization Human language support and internationalization macOS Mapbox Maps SDK for macOS runtime styling labels Dec 13, 2016
@1ec5 1ec5 added this to the ios-v3.4.0 milestone Dec 13, 2016
@1ec5 1ec5 self-assigned this Dec 13, 2016
@1ec5 1ec5 requested a review from ivovandongen December 13, 2016 10:15
@mention-bot
Copy link

@1ec5, thanks for your PR! By analyzing this pull request, we identified @boundsj, @incanus and @friedbunny to be potential reviewers.

@1ec5 1ec5 added the iOS Mapbox Maps SDK for iOS label Dec 13, 2016
@1ec5 1ec5 force-pushed the 1ec5-tile-layer-url branch from d50baa5 to f1330d3 Compare December 13, 2016 10:21
@1ec5 1ec5 requested a review from frederoni December 13, 2016 18:32
@1ec5 1ec5 force-pushed the 1ec5-tile-layer-url branch from f1330d3 to 37768e9 Compare December 14, 2016 01:42
Copy link
Contributor

@frederoni frederoni left a comment

Choose a reason for hiding this comment

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

iOS and macOS LGTM

@1ec5 1ec5 force-pushed the 1ec5-tile-layer-url branch from 37768e9 to e7fa769 Compare December 21, 2016 01:41
@1ec5 1ec5 merged commit f431b1b into release-ios-v3.4.0 Dec 21, 2016
@1ec5 1ec5 deleted the 1ec5-tile-layer-url branch December 21, 2016 05:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Core The cross-platform C++ core, aka mbgl iOS Mapbox Maps SDK for iOS localization Human language support and internationalization macOS Mapbox Maps SDK for macOS runtime styling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants