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

map.centerZoomEase #3967

Closed
2 tasks done
bhousel opened this issue Apr 13, 2017 · 0 comments
Closed
2 tasks done

map.centerZoomEase #3967

bhousel opened this issue Apr 13, 2017 · 0 comments
Labels
core An issue with one of the core iD components

Comments

@bhousel
Copy link
Member

bhousel commented Apr 13, 2017

We have:

  • map.center()
  • map.zoom()
  • map.centerZoom()

... for instantaneous changes that dispatch a 'move' event.

We later added:

  • map.centerEase()
  • map.zoomEase()
  • map.transformEase()

... that build on d3.zoom transform and support transitions. The 'move' event fires later.

  • For simplicity, all of these things should be refactored to wrap the internal setTransform()

Also we don't have:

  • map.centerZoomEase()

There are many places in the walkthrough that I'd use this.

@bhousel bhousel added the core An issue with one of the core iD components label Apr 13, 2017
bhousel added a commit that referenced this issue Jan 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core An issue with one of the core iD components
Projects
None yet
Development

No branches or pull requests

1 participant