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

feat(polylines): add support for Polylines #554

Closed
wants to merge 1 commit into from

Conversation

andir
Copy link
Contributor

@andir andir commented Aug 9, 2016

This adds support for Polylines on google maps.

Since angular2's QueryLists do not yet implement the Observable interface (https://angular.io/docs/js/latest/api/core/index/QueryList-class.html) the amount of polygons can't really change after initial creation.

@GitCop
Copy link

GitCop commented Aug 9, 2016

There were the following issues with your Pull Request

  • Commit: f3e3fbb
    • 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

@andir andir force-pushed the add-polyline branch 2 times, most recently from b49e175 to 0664f23 Compare August 9, 2016 13:35
@andir andir changed the title feat: Add support for Polylines feat(polylines): Add support for Polylines Aug 9, 2016
@andir andir changed the title feat(polylines): Add support for Polylines feat(polylines): add support for Polylines Aug 9, 2016
@sebholstein
Copy link
Owner

@andir thank you! I will add some comments this weekend but that looks very good already 👍

*/
lineRightClick: EventEmitter<PolyMouseEvent> = new EventEmitter<PolyMouseEvent>();

@ContentChildren(SebmGoogleMapPolylinePoint) _points: QueryList<SebmGoogleMapPolylinePoint>;
Copy link
Owner

Choose a reason for hiding this comment

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

this should be private

@sebholstein
Copy link
Owner

@andir Review ready

@andir
Copy link
Contributor Author

andir commented Aug 14, 2016

Thank you @SebastianM
I've pushed the changes and added some comments above.


let polylineId = 0;
/**
* SebmGoogleMapPolyline renders a polyline on a {
Copy link
Owner

Choose a reason for hiding this comment

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

Something is missing here

@sebholstein
Copy link
Owner

Thank you @andir . Added one little comment. Then I will test it (and when I find no bugs, merge it)

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