From 27d0388d7973256864ec12081d554abbaf5c6c68 Mon Sep 17 00:00:00 2001
From: Tom <7283497+thfrei@users.noreply.github.com>
Date: Sun, 10 Apr 2022 13:55:39 +0200
Subject: [PATCH] add some comments

---
 src/public/app/dialogs/note_revisions.js | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/src/public/app/dialogs/note_revisions.js b/src/public/app/dialogs/note_revisions.js
index eb6eddf5f..3303b1ec6 100644
--- a/src/public/app/dialogs/note_revisions.js
+++ b/src/public/app/dialogs/note_revisions.js
@@ -176,6 +176,12 @@ async function setContentPane() {
          * can the revisions called without being on the note type before?
          * if so: load excalidraw
          */
+        // FIXME: Does it make sense to use EXCALIDRAW_UTILS  that are 1.5 MB
+        //        whereas excalidraw (650kB) +react(12kB)+reactdom(118kB)
+        /**
+         * FIXME: We load a font called Virgil.wof2, which originates from excalidraw.com
+         *        REMOVE external dependency!!!! This is defined in the svg in defs.style
+         */ 
         await libraryLoader.requireLibrary(libraryLoader.EXCALIDRAW_UTILS);
         const {exportToSvg} = window.ExcalidrawUtils
 
@@ -192,11 +198,15 @@ async function setContentPane() {
                 files: data.files,
             }
             const svg = await exportToSvg(excData);
-            console.log("canvas-note", data, svg);
-            $content.html($("<div>").html(svg));
+            console.log("canvas-note revision", data, svg);
+            $content
+                .html(
+                    $('<div>')
+                    .html(svg)
+                );
         } catch(err) {
             console.error("error parsing fullNoteRevision.content as JSON", fullNoteRevision.content, err);
-            $content.html($("<div>").text("error parsing content"));
+            $content.html($("<div>").text("Error parsing content. Please check console.error() for more details."));
         }        
 
     }