Merge pull request #1691 from HenryKeen/fix-ie11-image-promise

Move the promise resolution to address IE11 bug
plotly · May 16, 2017 · a729acc · a729acc
2 parents e58dcb0 + 310f762
commit a729acc
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/components/images/draw.js b/src/components/images/draw.js
@@ -99,11 +99,15 @@ module.exports = function draw(gd) {
  var dataURL = canvas.toDataURL('image/png');
 
  thisImage.attr('xlink:href', dataURL);
+
+ // resolve promise in onload handler instead of on 'load' to support IE11
+ // see https://github.com/plotly/plotly.js/issues/1685
+ // for more details
+ resolve();
  };
 
 
  thisImage.on('error', errorHandler);
- thisImage.on('load', resolve);
 
  img.src = d.source;