Skip to content
This repository has been archived by the owner on May 20, 2024. It is now read-only.

Feat/support multiple points and markers #7

Merged

Conversation

literat
Copy link
Collaborator

@literat literat commented Feb 7, 2020

Support for multiple points with or without markers.

Calculation of center is needed for multiple points without markers. But maybe not if we extend bounds even for single point.

src/lib/utils/geoMath.ts Outdated Show resolved Hide resolved
src/lib/utils/geoMath.ts Outdated Show resolved Hide resolved
src/lib/lmc-maps.ts Outdated Show resolved Hide resolved
src/lib/utils/geoMath.ts Outdated Show resolved Hide resolved
Copy link
Member

@OndraM OndraM left a comment

Choose a reason for hiding this comment

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

LGTM for me, could you please squash the commits?

@literat literat force-pushed the feat/support-multiple-points-and-markers branch from 1ea6eb7 to 2d3d0ae Compare February 10, 2020 09:29
@literat
Copy link
Collaborator Author

literat commented Feb 10, 2020

@OndraM Changes squashed.

src/lib/utils/geoMath.ts Outdated Show resolved Hide resolved
src/lib/types.ts Show resolved Hide resolved
src/lib/lmc-maps.ts Outdated Show resolved Hide resolved
@yurikgbp
Copy link
Contributor

Could you please add changes to readme and index.d.ts and squash the commits?

@literat literat force-pushed the feat/support-multiple-points-and-markers branch from 63260f3 to 3823a2d Compare February 12, 2020 09:06
@literat
Copy link
Collaborator Author

literat commented Feb 12, 2020

Changes applied and commits squashed.

@OndraM OndraM merged commit bb2fb40 into lmc-eu:master Feb 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants