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

#29, #30, 32 Updates to map properties, fixed memory leak. #33

Closed
wants to merge 1 commit into from

Conversation

AlexKven
Copy link

@AlexKven AlexKven commented Jun 3, 2020

#29 Enables the Center and Zoom properties to set these properties on the map in the background.
#30 Adds a Bounds property to the map that is automatically kept up to date with the map bounds.
#32 Fixes a memory leak related to creating layers.

If you don't think one of these issues should be merged in here, let me know and I can remove them from this PR.

…leak.

Mehigh17#29 Enables the Center and Zoom properties to set these properties on the map in the background.
Mehigh17#30 Adds a Bounds property to the map that is automatically kept up to date with the map bounds.
Mehigh17#32 Fixes a memory leak related to creating layers.
@Mehigh17
Copy link
Owner

Mehigh17 commented Jun 6, 2020

Thanks for the contribution, could you split the contribution in three commits? I think one commit per issue in this case should be enough.

@AlexKven
Copy link
Author

AlexKven commented Jun 6, 2020

I can break this up into multiple PRs. I'll close this one when I do that.

@AlexKven AlexKven closed this Jun 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants