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

separate files for projections #5

Merged
merged 7 commits into from
Oct 24, 2016
Merged

separate files for projections #5

merged 7 commits into from
Oct 24, 2016

Conversation

derhuerst
Copy link
Contributor

This PR introduces single files (e.g. p/mercator.js) for every projection. This makes bundle sizes smaller and allows to easily add more projections.

ccf5000 is a breaking change, so i bumped the version number to 0.6.0.

It is common in the JS ecosystem to use dasherized file names & camel-cased identifiers. I adapted the code to this convention.

Since this is a breaking change, I bumped the version number to `0.6.0`.
@juliuste
Copy link
Owner

juliuste commented Oct 13, 2016

Wouldn't it be better (for usability) to move the projections to the main folder?

@derhuerst
Copy link
Contributor Author

derhuerst commented Oct 13, 2016

Wouldn't it be better (for usability) to move the projections to the main folder?

It's a matter of taste. Since there's lots of meta files (readme, gitignore) and since there might easily be more projections in the future, I moved them into a directory.

@derhuerst
Copy link
Contributor Author

bump. (:

I will adapt the location of the projections to your taste (imho it's good like it this). would really like to see projections@0.6.0 being released, as geojson-svgify depends on it.

@derhuerst
Copy link
Contributor Author

bump.

@juliuste juliuste merged commit 8b12c22 into juliuste:master Oct 24, 2016
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.

None yet

2 participants