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

[bug] Fix puck animation transition #256

Merged
merged 1 commit into from
Apr 12, 2021
Merged

Conversation

neelmistry94
Copy link
Contributor

Pull request checklist:

  • Briefly describe the changes in this PR.
  • Include before/after visuals or gifs if this PR includes visual changes.
  • Document any changes to public APIs.
  • Apply changelog label ('breaking change', 'bug 🪲', 'build', 'docs', 'feature 🍏', 'performance ⚡', 'testing 💯') or use the label 'skip changelog'
  • Add an entry inside this element for inclusion in the mapbox-maps-ios changelog: <changelog>Bug fix for location puck not transition/animating smoothly on location updates. </changelog>.

Summary of changes

This PR updates the locationTransition layer property in the 2D puck to use a StyleTransition. It is discovered in the documentation that Transitions use milliseconds, and therefore we needed to provide an appropriate duration value to see a smooth animation.

Please see the gif below.

@neelmistry94 neelmistry94 added the bug 🪲 Something is broken! label Apr 12, 2021
@neelmistry94 neelmistry94 enabled auto-merge (squash) April 12, 2021 17:44
@neelmistry94 neelmistry94 merged commit b8f2b5a into main Apr 12, 2021
@neelmistry94 neelmistry94 deleted the nm/puck_animation_fix branch April 12, 2021 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something is broken!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants