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

[ios, macos] Make feature attributes and identifier writable #6728

Merged
merged 4 commits into from
Oct 19, 2016

Conversation

boundsj
Copy link
Contributor

@boundsj boundsj commented Oct 18, 2016

Instances of MGLFeature subclasses can now have their attributes and identifier properties mutated.

Follow up to #6302

cc @1ec5

Instances of MGLFeature subclasses can now have their attributes
and identifier properties mutated.
@boundsj boundsj added iOS Mapbox Maps SDK for iOS runtime styling labels Oct 18, 2016
@boundsj boundsj added this to the ios-v3.4.0 milestone Oct 18, 2016
@boundsj boundsj self-assigned this Oct 18, 2016
@mention-bot
Copy link

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

Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

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

Also note the change in the iOS and macOS changelogs, since MGLFeature was introduced in the last release.

@@ -50,7 +50,7 @@ NS_ASSUME_NONNULL_BEGIN
<a href="https://www.mapbox.com/vector-tiles/mapbox-streets/">Mapbox Streets</a>
layer reference.
Copy link
Contributor

@1ec5 1ec5 Oct 18, 2016

Choose a reason for hiding this comment

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

Point out that you can change this property and attributes on an MGLFeature obtained using feature querying, but doing so has no effect on the map.

@boundsj boundsj changed the base branch from master to release-ios-v3.4.0 October 18, 2016 20:34
@boundsj
Copy link
Contributor Author

boundsj commented Oct 18, 2016

@1ec5 40681d4 just removes a category in a test that I just remembered. It is no longer required.

@boundsj boundsj merged commit baf82fc into release-ios-v3.4.0 Oct 19, 2016
@boundsj boundsj deleted the boundsj-readwrite-attributes branch October 19, 2016 16:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS runtime styling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants