Move mercator transforms to Projection.js. #153

Merged
merged 6 commits into from Jan 17, 2012

Projects

None yet

3 participants

@tschaub
OpenLayers member

The SphericalMercator mixin is not required for coordinate transforms. Default transforms included whenever Projection.js is included.

@tschaub tschaub Move mercator transforms to Projection.js.
The SphericalMercator mixin is not required for coordinate transforms.  Default transforms included whenever Projection.js is included.
4b94917
@elemoine
OpenLayers member

Awesome. We may just want something in the release notes for this change. Thank you for making this real. Please merge.

@probins

Wow! Fast work!

We can also remove the SphericalMercator dependency that @elemoine has just added to Layer/OSM :-)

One small api change from removing SphericalMercator from Bing that has just occurred to me is that forwardMercator() and inverseMercator() are api methods, so it's possible (though probably not very likely) that people were using them (with OpenLayers.Layer.SphericalMercator.forwardMercator(...)). This will no longer work if SphericalMercator isn't present. So, yes, I agree that something to this effect should be in the release notes.

tschaub added some commits Jan 17, 2012
@tschaub tschaub Merge branch 'master' of github.com:openlayers/openlayers into proj 00975e7
@tschaub tschaub Removing redundant dependency.
Projection.js gets pulled in as a transitive dependency of all layers.
6134ad6
@tschaub tschaub Removing dependency on SphericalMercator.
The default transforms (b/w EPSG:4326 & EPSG:900913) now come with any layer type.
bfa6f06
@tschaub tschaub Documenting the functionality included in Projection.js.
This adds documentation for projection methods and includes detail in the release notes about build configuration related changes.
9a6827f
@tschaub tschaub merged commit 974ed29 into openlayers:master Jan 17, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment