Set tms option for custom layer

ckan / ckanext-geoview

CKAN Geospatial ResourceView

MIT License

43 stars 58 forks source link

Set tms option for custom layer #24

Closed nbtetreault closed 8 years ago

nbtetreault commented 9 years ago

Set the L.TileLayer as a TMS. It's required if the base layer specified in custom.url is a TMS. The default value of tms is false in Leaflet. To use the TMS option, ckanext.spatial.common_map.tms = true has to be set in the ckan .INI file.

gignacnic commented 8 years ago

Does this pull request can be merged soon?

amercader commented 8 years ago

Sorry, this looks great, merged now.

It would be great if you could send a PR with the same change to ckanext-spatial:

https://github.com/ckan/ckanext-spatial/blob/master/ckanext/spatial/public/js/common_map.js

Otherwise the change won't apply to other maps like the search widget or the one one the dataset page.

See https://github.com/ckan/ckanext-geoview#common-base-layers-for-map-widgets

nbtetreault commented 8 years ago

Great. I did the PR for ckanext-spatial https://github.com/ckan/ckanext-spatial/pull/129