From 301d141ce06b87f565b3c31936d5ad1bdd4174af Mon Sep 17 00:00:00 2001
From: Paulo Gustavo Veiga <pveiga@wisemapping.com>
Date: Tue, 15 Feb 2022 18:29:02 -0800
Subject: [PATCH] Define image export size limit

---
 .../components/export/BinaryImageExporter.ts  | 11 +---------
 .../src/components/export/SVGExporter.ts      | 21 +++++++++++++++----
 2 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/packages/mindplot/src/components/export/BinaryImageExporter.ts b/packages/mindplot/src/components/export/BinaryImageExporter.ts
index 3d0d7ff0..c7805ad3 100644
--- a/packages/mindplot/src/components/export/BinaryImageExporter.ts
+++ b/packages/mindplot/src/components/export/BinaryImageExporter.ts
@@ -21,8 +21,6 @@ import SVGExporter from './SVGExporter';
  * Based on https://mybyways.com/blog/convert-svg-to-png-using-your-browser
  */
 class BinaryImageExporter extends Exporter {
-  private static maxAllowedSize = 10000;
-
   private svgElement: Element;
 
   private width: number;
@@ -65,14 +63,7 @@ class BinaryImageExporter extends Exporter {
         height = (this.height * dpr);
       }
 
-      // // Prevents an image too big. Failures seen during export in couple of browsers
-      // if (Math.max(width, height) > BinaryImageExporter.maxAllowedSize) {
-      //   const scale = Math.max(width, height) / BinaryImageExporter.maxAllowedSize;
-      //   width /= scale;
-      //   height /= scale;
-      // }
-
-      console.log(`Export size: ${width}: ${height}`);
+      console.log(`Export size: ${width}:${height}`);
       canvas.setAttribute('width', width.toFixed(0));
       canvas.setAttribute('height', height.toFixed(0));
 
diff --git a/packages/mindplot/src/components/export/SVGExporter.ts b/packages/mindplot/src/components/export/SVGExporter.ts
index 6a45ae81..8613f9da 100644
--- a/packages/mindplot/src/components/export/SVGExporter.ts
+++ b/packages/mindplot/src/components/export/SVGExporter.ts
@@ -29,6 +29,8 @@ class SVGExporter extends Exporter {
 
   private adjustToFit: boolean;
 
+  private static MAX_SUPPORTED_SIZE = 2500;
+
   constructor(svgElement: Element, adjustToFit = true) {
     super('svg', 'image/svg+xml');
     this.svgElement = svgElement;
@@ -111,8 +113,16 @@ class SVGExporter extends Exporter {
       minX, maxX, minY, maxY,
     } = this._calcualteDimensions();
 
-    const width = maxX + Math.abs(minX);
-    const height = maxY + Math.abs(minY);
+    let width: number = maxX + Math.abs(minX);
+    let height: number = maxY + Math.abs(minY);
+
+    // Prevents an image too big. Failures seen during export in couple of browsers
+    if (Math.max(width, height) > SVGExporter.MAX_SUPPORTED_SIZE) {
+      const scale = Math.max(width, height) / SVGExporter.MAX_SUPPORTED_SIZE;
+      width /= scale;
+      height /= scale;
+    }
+
     return { width, height };
   }
 
@@ -127,8 +137,11 @@ class SVGExporter extends Exporter {
 
     svgElem.setAttribute('viewBox', `${minX} ${minY} ${width}  ${height}`);
     svgElem.setAttribute('preserveAspectRatio', 'xMinYMin');
-    svgElem.setAttribute('width', width.toFixed(0));
-    svgElem.setAttribute('height', height.toFixed(0));
+
+    // Get image size ...
+    const imgSize = this.getImgSize();
+    svgElem.setAttribute('width', imgSize.width.toFixed(0));
+    svgElem.setAttribute('height', imgSize.height.toFixed(0));
 
     return document;
   }