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

Add animation field to markers #580

Closed
wants to merge 3 commits into from
Closed

Add animation field to markers #580

wants to merge 3 commits into from

Conversation

orlov-vo
Copy link

No description provided.

@GitCop
Copy link

GitCop commented Aug 17, 2016

There were the following issues with your Pull Request

  • Commit: 83e15d2
    • Commits must be in the following format: %{type}(%{scope}): %{description}

Contribution guidelines are available at https://github.com/SebastianM/angular2-google-maps/blob/master/CONTRIBUTING.md


This message was auto-generated by https://gitcop.com

* Which animation to play when marker is added to a map.
* This can be "bounce" or "drop"
*/
animation: string;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should use BOUNCE and DROP here

@sebholstein
Copy link
Owner

@orlov-vo thank you! and sorry for the late review. Added some comments.

xdimension pushed a commit to xdimension/angular2-google-maps that referenced this pull request Jan 13, 2017
Feature to set animation for map markers

issue: Add animation field to markers sebholstein#580
xdimension pushed a commit to xdimension/angular2-google-maps that referenced this pull request Jan 22, 2017
Feature to set animation for map markers
Updated according to comments in sebholstein#852

issue: Add animation field to markers sebholstein#580
xdimension pushed a commit to xdimension/angular2-google-maps that referenced this pull request Jan 29, 2017
Feature to set animation for map markers

issue: Add animation field to markers sebholstein#580

feat: add animation field to markers

Feature to set animation for map markers
Updated according to comments in sebholstein#852

issue: Add animation field to markers sebholstein#580
xdimension pushed a commit to xdimension/angular2-google-maps that referenced this pull request Jan 29, 2017
Feature to set animation for map markers

issue: Add animation field to markers sebholstein#580
xdimension pushed a commit to xdimension/angular2-google-maps that referenced this pull request Jan 29, 2017
Feature to set animation for map markers

issue: Add animation field to markers sebholstein#580
xdimension pushed a commit to xdimension/angular2-google-maps that referenced this pull request Feb 11, 2017
Feature to set animation for map markers

issue: Add animation field to markers sebholstein#580
xdimension pushed a commit to xdimension/angular2-google-maps that referenced this pull request Feb 13, 2017
Feature to set animation for map markers

issue: Add animation field to markers sebholstein#580
sebholstein pushed a commit that referenced this pull request Dec 18, 2017
Feature to set animation for map markers

Closes #580
Closes #852
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants