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

Update iOS and macOS changelogs #9919

Merged
merged 2 commits into from
Sep 12, 2017
Merged

Update iOS and macOS changelogs #9919

merged 2 commits into from
Sep 12, 2017

Conversation

boundsj
Copy link
Contributor

@boundsj boundsj commented Sep 5, 2017

This adds entries for work done on the master branch since v3.6.0 of our iOS Maps SDK that was not included in any of the 3.6.x patch releases.

@boundsj boundsj added the iOS Mapbox Maps SDK for iOS label Sep 5, 2017
@boundsj boundsj self-assigned this Sep 5, 2017
@boundsj
Copy link
Contributor Author

boundsj commented Sep 5, 2017

I think the first common ancestor is b7b6706 so I will need to go back even more. Will push another commit later with any updates that come from changes that happened since that ancestor (the time of the fork release-ios-v3.6.0-android-v5.1.0) and the v3.6.0 release.

@boundsj boundsj mentioned this pull request Sep 5, 2017
3 tasks
@boundsj boundsj changed the title Update changelog Update iOS and macOS changelogs Sep 6, 2017
@boundsj
Copy link
Contributor Author

boundsj commented Sep 6, 2017

@1ec5 I just added a few more items that I found while scanning through commits from May 10 through June 29. I had only looked at June 29 to date before. It is certainly possible I missed something but this PR now captures everything as best I can tell.

Copy link
Contributor

@fabian-guerra fabian-guerra left a comment

Choose a reason for hiding this comment

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

👍🏼

@1ec5 1ec5 added documentation macOS Mapbox Maps SDK for macOS labels Sep 6, 2017

### Styles

* Added a new type of source, represented by the `MGLImageSource` class at runtime, that displays a georeferenced image. ([#9110](https://github.com/mapbox/mapbox-gl-native/pull/9110))
* Setting a style using `MGLMapView`'s `styleURL` property now smoothly transitions from the previous style to the new style and maintains equivalent layers and sources along with their identifiers. ([#9256](https://github.com/mapbox/mapbox-gl-native/pull/9256))
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this mean it’s safe to maintain a reference to a MGLStyleLayer or MGLSource object across style changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That was my understanding after reading #9256. The copy here (and the copy in question in #9919 (comment)) was taken from that PR.

@@ -23,12 +25,14 @@ Mapbox welcomes participation and contributions from everyone. Please read [CONT

### Annotations and user interaction

* Fixed several bugs and performance issues related to the use of annotations backed by `MGLAnnotationImage`s. These fixes also apply to images used to represent icons in `MGLSymbolStyleLayer`s. ([#9213](https://github.com/mapbox/mapbox-gl-native/pull/9213))
Copy link
Contributor

Choose a reason for hiding this comment

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

Specifically, this fixed #3185, which was previously a major issue for iOS developers. Maybe we can say that it eliminated the too-many-sprites problem for all practical purposes.

* Increased the default maximum zoom level from 20 to 22. ([#9835](https://github.com/mapbox/mapbox-gl-native/pull/9835))
* Added an `overlays` property to `MGLMapView`. ([#8617](https://github.com/mapbox/mapbox-gl-native/pull/8617))

### Other changes

* Fixed an issue that could cause line label rendering glitches when the line geometry is projected to a point behind the plane of the camera. ([#9865](https://github.com/mapbox/mapbox-gl-native/pull/9865))
* Fixed an issue that could cause a crash when using `MGLMapView`'s `flyToCamera:` APIs with zoom levels at or near the maximum value. ([#9381](https://github.com/mapbox/mapbox-gl-native/pull/9381))
Copy link
Contributor

Choose a reason for hiding this comment

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

when using -[MGLMapView flyToCamera:completionHandler:] and related methods


### Styles

* Added a new type of source, represented by the `MGLImageSource` class at runtime, that displays a georeferenced image. ([#9110](https://github.com/mapbox/mapbox-gl-native/pull/9110))
* Setting a style using `MGLMapView`'s `styleURL` property now smoothly transitions from the previous style to the new style and maintains equivalent layers and sources along with their identifiers. ([#9256](https://github.com/mapbox/mapbox-gl-native/pull/9256))
Copy link
Contributor

Choose a reason for hiding this comment

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

This makes it sound like a style change will crossfade by default. Is that the case?

Copy link
Contributor

Choose a reason for hiding this comment

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

crossfade by default. Is that the case?

Yeah, it does (and it’s pretty slick).


* Added a new `MGLMapSnapshotter` class for capturing rendered map images from an `MGLMapView`’s camera. ([#9891](https://github.com/mapbox/mapbox-gl-native/pull/9891))
* Reduced the time it takes to create new `MGLMapView` instances in some cases. ([#9864](https://github.com/mapbox/mapbox-gl-native/pull/9864))
* Added support for forced cache revalidation that will eliminate flickering that was sometimes visible for certain types of tiles (i.e., traffic tiles). ([#9670](https://github.com/mapbox/mapbox-gl-native/pull/9670), [#9103](https://github.com/mapbox/mapbox-gl-native/issues/9103))
Copy link
Contributor

Choose a reason for hiding this comment

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

Grammar nitpick: if this affects other types of tiles beyond traffic, we should use “e.g.” instead of “i.e.” — the former indicates one possible example of a thing, while the latter indicates it is the only example of the thing.

* The previously-deprecated support for style classes has been removed. For interface compatibility, the API methods remain, but they are now non-functional.
* Added a new type of source, represented by the `MGLImageSource` class at runtime, that displays a georeferenced image. ([#9110](https://github.com/mapbox/mapbox-gl-native/pull/9110))
* Setting a style using `MGLMapView`'s `styleURL` property now smoothly transitions from the previous style to the new style and maintains equivalent layers and sources along with their identifiers. ([#9256](https://github.com/mapbox/mapbox-gl-native/pull/9256))
* Added new properties to `MGLCircleStyleLayer` and `MGLSymbolStyleLayer` to control whether circles and symbols lie flat against a tilted map. ([#9426](https://github.com/mapbox/mapbox-gl-native/pull/9426), [#9479](https://github.com/mapbox/mapbox-gl-native/pull/9479))
Copy link
Contributor

Choose a reason for hiding this comment

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

This makes me wonder: what’s the name of the new property? Perhaps we could work in *-pitch-alignment somehow.

@friedbunny friedbunny added this to the ios-v3.7.0 milestone Sep 8, 2017
@friedbunny friedbunny assigned friedbunny and unassigned boundsj Sep 11, 2017
@friedbunny
Copy link
Contributor

Made changes based on review comments (and incidentally rebased).

Copy link
Contributor

@friedbunny friedbunny left a comment

Choose a reason for hiding this comment

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

Self-approved. ✅

boundsj and others added 2 commits September 12, 2017 12:37
This adds entries for work done on the master branch since v3.6.0 of the iOS Maps SDK that was not included in any of the 3.6.x patch releases.
This adds entries for work done on the master branch since v0.5.1 of the macOS Maps SDK.
@friedbunny friedbunny merged commit 161c5ed into master Sep 12, 2017
@friedbunny friedbunny deleted the boundsj-update-changelog branch September 12, 2017 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants