Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add support for multiple tileSizes in the tilegrid #258

Merged
merged 6 commits into from Mar 4, 2013

Conversation

Projects
None yet
2 participants
Member

bbinet commented Mar 3, 2013

This is needed by WMTS.

Contributor

twpayne commented Mar 4, 2013

Thanks for this. I've checked the renderers, and as I currently understand it:

  • the DOM and WebGL renderers can cope with per-z tile sizes
  • the Canvas renderer will need a minor modification [1]

so, please add [1] and merge. We can test it properly when we have WMTS layers that use different tile sizes at different Zs.

[1] In src/ol/renderer/canvas/canvastilelayerrenderer.js, after line 176 add tileSize = tileGrid.getTileSize(currentZ);

bbinet added a commit that referenced this pull request Mar 4, 2013

Merge pull request #258 from bbinet/tilegrid-tilesizes
Add support for multiple tileSizes in the tilegrid

@bbinet bbinet merged commit cedba8c into openlayers:master Mar 4, 2013

@bbinet bbinet deleted the bbinet:tilegrid-tilesizes branch Mar 4, 2013

tschaub added a commit that referenced this pull request Mar 4, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment