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

Corrected #529 in master (Leaflet 1.0.0) #644

Merged
merged 1 commit into from
Mar 6, 2016
Merged

Conversation

ghybs
Copy link
Contributor

@ghybs ghybs commented Mar 4, 2016

Same as #344, which was corrected by commit 254198a in leaflet-0.7 branch only. Do the same in master branch (for Leaflet 1.0.0) as well.
Also added a dedicated test spec in zoomAnimationSpec test suite.

Took the opportunity to refactor the test suite to avoid PhantomJS memory leak.

Same as Leaflet#344, which was corrected by commit 254198a in leaflet-0.7 branch only. Do the same in master branch (for Leaflet 1.0.0) as well.
Also added a dedicated test spec in zoomAnimationSpec test suite.

Took the opportunity to refactor the test suite to avoid PhantomJS memory leak.
danzel added a commit that referenced this pull request Mar 6, 2016
Corrected #529 in master (Leaflet 1.0.0)
@danzel danzel merged commit cb05eb2 into Leaflet:master Mar 6, 2016
@danzel
Copy link
Member

danzel commented Mar 6, 2016

Thanks, hope we didn't mess any other bits from the 0.7 branch.

@ghybs
Copy link
Contributor Author

ghybs commented Mar 6, 2016

Indeed, very strange that it did not made it…

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

2 participants