Closed Ventero closed 11 years ago
Great fix! Just a couple of comments:
resultingImage
instead of img
inside complete()?Thanks for reviewing the pull request. I've added a commit which updates the documentation and fixes the variable name. I hope the updated documentation makes sense - since English isn't my first language, feel free to correct me if something's wrong. ;)
Looks great!
Previously, the image object was passed on to onTileLoad regardless if it loaded successfully or not. In case an image failed to load, this would result in trying to draw a non-existing image onto the canvas, which (at least in Firefox) throws a DOM exception and results in the drawer being stuck mid-update.