MazeMap / Leaflet.TileLayer.PouchDBCached

A Leaflet tile layer which caches into PouchDB for offline use
MIT License
208 stars 68 forks source link

Multiple fixes #26

Open bydos opened 7 years ago

bydos commented 7 years ago

1- Fix PouchDB 6.x compatibility 2- Fix Tile coordinates generation for TMS layers (y coordinates) 3- Asynchronous call for the recursive SeedOneTile() method 4- Add dbSizeLimit option to avoid excessive seeding (needs more considerations) 5- Extending L.TileLayer to L.TileLayer.PouchDB to avoid conflicts with other plugins that modify internal methods

IvanSanchez commented 7 years ago

@bydos Looks good, but there are a number of things I'm not happy about. The comments might seem a bit harsh (I don't have the time to write them nicer, sorry!), please have a look at them.

bydos commented 7 years ago

@IvanSanchez , Yes it was done a bit quickly. It was just to give some orientations, not really a clean PR. I am a little bit busy this couple of days sorry, but when I get enough time I will consider a global review and also some enhancements. Thanks

kevinejohn commented 7 years ago

Any updates on this?