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.

Sign up for GitHub

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

Jump to bottom

Reorganize _Control in order to destroy #516

Merged
merged 3 commits into from
Mar 24, 2016
Merged

Reorganize _Control in order to destroy #516

merged 3 commits into from
Mar 24, 2016

Conversation

AlexAThomas
Copy link
Contributor

Reorganize _Control in order to destroy it on removeLayerControl calls.

  • grouped the layer handlers into one section, and remove them all upon destroy

@AlexThomasEOG
Reorganize _Control in order to destroy
bf69c2a 
Reorganize _Control in order to destroy
@tmcgee
Copy link
Member

tmcgee commented Mar 1, 2016

Thanks! Will review.

@tmcgee tmcgee added this to the v1.4.0 milestone Mar 5, 2016
@tmcgee tmcgee merged commit 7aa96da into cmv:develop Mar 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.0.0-beta.1
Development

Successfully merging this pull request may close these issues.
3 participants
@AlexAThomas @tmcgee @AlexThomasEOG