painfully work through more IE bugs with snapshot

Author: timelyportfolio

src/components/modebar/buttons.js

@@ -53,17 +53,17 @@ modeBarButtons.toImage = {
 
         Lib.notifier('Taking snapshot - this may take a few seconds', 'long');
 
-        if(Lib.isIE()){
+        if(Lib.isIE()) {
             Lib.notifier('IE only supports svg.  Changing format to svg.', 'long');
             format = 'svg';
         }
 
-        downloadImage(gd, {'format':format})
+        downloadImage(gd, {'format': format})
           .then(function(filename) {
               Lib.notifier('Snapshot succeeded - ' + filename, 'long');
           })
           .catch(function() {
-              Lib.notifier('Sorry there was a problem downloading your snapshot', 'long');
+              Lib.notifier('Sorry there was a problem downloading your snapshot!', 'long');
           });
     }
 };

src/plot_api/to_image.js

@@ -37,6 +37,8 @@ function toImage(gd, opts) {
             if(isNumeric(size) && size > 1) {
                 return true;
             }
+
+            return false;
         };
 
         if(!isSizeGood(opts.width) || !isSizeGood(opts.height)) {
@@ -80,7 +82,7 @@ function toImage(gd, opts) {
                         //  compatibility
                         promise: true
                     }).then(function(url) {
-                        if(clonedGd) clonedGd.remove();
+                        if(clonedGd) document.body.removeChild(clonedGd);
                         resolve(url);
                     }).catch(function(err) {
                         reject(err);

src/snapshot/filesaver.js

@@ -52,8 +52,8 @@ var fileSaver = function(url, name) {
         }
 
         // IE 10+ (native saveAs)
-        if(typeof navigator !== 'undefined' && navigator.msSaveOrOpenBlob) {
-            navigator.msSaveOrOpenBlob(url, name);
+        if(typeof navigator !== 'undefined' && navigator.msSaveBlob) {
+            navigator.msSaveBlob(new Blob([url]), name);
             resolve(name);
         }
 

src/snapshot/svgtoimg.js

@@ -56,7 +56,11 @@ function svgToImg(opts) {
         img.onload = function() {
             var imgData;
 
-            ctx.drawImage(img, 0, 0);
+            // don't need to draw to canvas if svg
+            //  save some time and also avoid failure on IE
+            if(format !== 'svg') {
+                ctx.drawImage(img, 0, 0);
+            }
 
             switch(format) {
                 case 'jpeg':

src/snapshot/toimage.js

@@ -57,7 +57,7 @@ function toImage(gd, opts) {
             });
 
             ev.clean = function() {
-                if(clonedGd) clonedGd.remove();
+                if(clonedGd) document.body.removeChild(clonedGd);
             };
 
         }, delay);