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

Add getLayer and getSource snapshotter methods #292

Merged
merged 3 commits into from Mar 25, 2020

Conversation

alexshalamov
Copy link
Contributor

@alexshalamov alexshalamov commented Mar 25, 2020

Add getLayer, getSource and Observer interface for a MapSnapshotter, so that the developer can modify snapshotter's layer and/or source object's properties.

<changelog>A snapshotter's layers and sources can now be modified by using the onDidFinishLoadingStyle() callback and APIs to retrieve layer/source properties from a style.</changelog>

note from @chloekraw: I'm not sure if a separate changelog entry for this PR is needed since it's a necessary part of enabling runtime styling in the snapshotter, and the latter will be discussed in the changelog. Let's see when we draft the changelog.

@alexshalamov
Copy link
Contributor Author

@tobrun should I create new snapshotter activity, something like 72152aa?

@alexshalamov
Copy link
Contributor Author

alexshalamov commented Mar 25, 2020

Added example activity. Top view is a map view, the view below is an image view with an image from snapshotter with modified style.

Copy link
Member

@tobrun tobrun left a comment

Choose a reason for hiding this comment

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

some nits to improve consumption from Kotlin in terms of nullability :shipit:

@alexshalamov alexshalamov force-pushed the alexshalamov_add_snapshotter_methods branch 2 times, most recently from 9938c04 to cffb842 Compare March 25, 2020 11:00
@alexshalamov alexshalamov force-pushed the alexshalamov_add_snapshotter_methods branch from cffb842 to 3621dbc Compare March 25, 2020 11:08
@alexshalamov alexshalamov marked this pull request as ready for review March 25, 2020 11:09
@alexshalamov alexshalamov merged commit 6984dbf into master Mar 25, 2020
@tobrun tobrun added this to the release-vanillashake milestone Mar 26, 2020
Chaoba pushed a commit that referenced this pull request Mar 26, 2020
* Expose snapshotter layer and source objects

* Add example snapshotter + within activity

* Bump vendor/mapbox-gl-native

(cherry picked from commit 6984dbf)
Chaoba pushed a commit that referenced this pull request Mar 27, 2020
* Expose snapshotter layer and source objects

* Add example snapshotter + within activity

* Bump vendor/mapbox-gl-native

(cherry picked from commit 6984dbf)
@chloekraw chloekraw deleted the alexshalamov_add_snapshotter_methods branch April 1, 2020 05:47
@chloekraw chloekraw restored the alexshalamov_add_snapshotter_methods branch April 1, 2020 05:48
@chloekraw chloekraw deleted the alexshalamov_add_snapshotter_methods branch May 1, 2020 00:49
@chloekraw chloekraw restored the alexshalamov_add_snapshotter_methods branch May 1, 2020 00:49
@chloekraw chloekraw deleted the alexshalamov_add_snapshotter_methods branch May 1, 2020 00:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants