Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make it possible to synchronize map navigation #1155

Closed
wants to merge 1 commit into from
Closed

Make it possible to synchronize map navigation #1155

wants to merge 1 commit into from

Conversation

tschaub
Copy link
Member

@tschaub tschaub commented Oct 22, 2013

As the current side-by-side example demonstrates, using one view with two maps doesn't work for animated zoom/pan/rotate.

We should be able to provide a way for applications to have more than one map that share the rendered view state. As the examples below demonstrate, this is possible, but it requires the use of non-exported code and some awkwardness.

This pull request is meant to serve as a discussion point for making map synchronization easier.

@twpayne
Copy link
Contributor

twpayne commented Oct 23, 2013

Note that the example only synchronises navigation, not API calls. For example, executing

map1.getView().getView2D().setCenter([1e7, 1e7]);

causes the maps to desynchronise.

@tschaub
Copy link
Member Author

tschaub commented Oct 23, 2013

Right, I tried to be careful about saying "synchronized navigation" (as opposed to synchronized view).

I'm experimenting with an alternative that would allow synchronized views.

@pagameba
Copy link
Member

@tschaub did you get anywhere with this? I'm curious because I made a little app with two maps which could benefit :)

@tonio
Copy link
Member

tonio commented Jul 3, 2014

Kinda duplicate of #298, or maybe achievable with listening to postcompose on the map, & synchronize center & resolution.

@tonio tonio closed this Jul 3, 2014
@tonio tonio reopened this Jul 3, 2014
@tonio
Copy link
Member

tonio commented Jul 3, 2014

Didn’t saw that was a PR :/ Maybe could be updated??

@tschaub
Copy link
Member Author

tschaub commented Aug 19, 2014

This should be closed. I think the correct solution is to animate views instead. Working separately on that.

@tschaub tschaub closed this Aug 19, 2014
@tschaub tschaub deleted the sync branch August 19, 2014 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants