-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Conversation
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 |
@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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏼
platform/ios/CHANGELOG.md
Outdated
|
||
### 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)) |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
platform/ios/CHANGELOG.md
Outdated
@@ -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)) |
There was a problem hiding this comment.
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.
platform/ios/CHANGELOG.md
Outdated
* 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)) |
There was a problem hiding this comment.
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
platform/macos/CHANGELOG.md
Outdated
|
||
### 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)) |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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).
platform/ios/CHANGELOG.md
Outdated
|
||
* 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)) |
There was a problem hiding this comment.
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.
platform/ios/CHANGELOG.md
Outdated
* 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)) |
There was a problem hiding this comment.
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.
2751a86
to
a5afb08
Compare
Made changes based on review comments (and incidentally rebased). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Self-approved. ✅
8b9a804
to
b826ed9
Compare
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.
b826ed9
to
161c5ed
Compare
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.