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

Circle Component #3

Merged
merged 48 commits into from
Sep 13, 2016
Merged

Circle Component #3

merged 48 commits into from
Sep 13, 2016

Conversation

Josh-ES
Copy link
Owner

@Josh-ES Josh-ES commented Sep 12, 2016

  • Circle
  • Introduced a new React component, Circle, to create circles on maps provided by HERE Maps.
  • Encapsulate the Circle behaviour from the HERE Maps API, but not that of the other provided Geoshapes.
  • Component is used as a child of a HEREMap component.
  • Component simply creates a circle of the given radius at the given location.
  • Automatic Position Changes of Circle and Marker instances
  • When the lat/lng props of a Circle or Marker instance change, their position automatically changes.
  • When the rad prop of a Circle instance changes, the radius automatically changes to the new value.

@coveralls
Copy link

coveralls commented Sep 12, 2016

Coverage Status

Coverage decreased (-1.3%) to 94.33% when pulling 7b3ea5f on circle into 25b6eb8 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.3%) to 92.377% when pulling ca9d167 on circle into 25b6eb8 on master.

1 similar comment
@coveralls
Copy link

coveralls commented Sep 12, 2016

Coverage Status

Coverage decreased (-3.3%) to 92.377% when pulling ca9d167 on circle into 25b6eb8 on master.

@coveralls
Copy link

coveralls commented Sep 12, 2016

Coverage Status

Coverage decreased (-3.3%) to 92.377% when pulling 20a833e on circle into 25b6eb8 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.9%) to 97.549% when pulling 40b602c on circle into 25b6eb8 on master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+1.9%) to 97.549% when pulling 40b602c on circle into 25b6eb8 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.9%) to 97.549% when pulling 40b602c on circle into 25b6eb8 on master.

@coveralls
Copy link

coveralls commented Sep 13, 2016

Coverage Status

Coverage increased (+1.9%) to 97.549% when pulling 5d25065 on circle into 25b6eb8 on master.

@coveralls
Copy link

coveralls commented Sep 13, 2016

Coverage Status

Coverage increased (+1.9%) to 97.549% when pulling 5d25065 on circle into 25b6eb8 on master.

@coveralls
Copy link

coveralls commented Sep 13, 2016

Coverage Status

Coverage increased (+1.9%) to 97.549% when pulling 8f5088e on circle into 25b6eb8 on master.

@Josh-ES Josh-ES merged commit dead892 into master Sep 13, 2016
@Josh-ES Josh-ES deleted the circle branch September 13, 2016 17:34
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.

2 participants