From 62a12e8825b1a06692c69feba4b0376cda9285a5 Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Thu, 12 Mar 2020 13:40:58 +0100
Subject: [PATCH] Cache an throttle the markdown rendering in kanban

---
 www/kanban/inner.js   | 12 +++++++++---
 www/kanban/jkanban.js | 20 +++++++++++++++++++-
 2 files changed, 28 insertions(+), 4 deletions(-)

diff --git a/www/kanban/inner.js b/www/kanban/inner.js
index c48138a3a..ffabd8951 100644
--- a/www/kanban/inner.js
+++ b/www/kanban/inner.js
@@ -152,10 +152,14 @@ define([
         var dataObject = {};
         var isBoard, id;
 
-        var commit = function () {
-            framework.localChange();
+        var update = Util.throttle(function () {
             kanban.setBoards(kanban.options.boards);
             addEditItemButton(framework, kanban);
+        }, 400);
+
+        var commit = function () {
+            framework.localChange();
+            update();
         };
         if (editModal) { return editModal; }
         var conflicts, conflictContainer, titleInput, tagsDiv, colors, text;
@@ -249,7 +253,9 @@ define([
         };
         SFCodeMirror.mkIndentSettings(editor, framework._.cpNfInner.metadataMgr);
         editor.on('change', function () {
-            dataObject.body = editor.getValue();
+            var val = editor.getValue();
+            if (dataObject.body === val) { return; }
+            dataObject.body = val;
             commit();
         });
 
diff --git a/www/kanban/jkanban.js b/www/kanban/jkanban.js
index a7fd5f720..30b9ff5e4 100644
--- a/www/kanban/jkanban.js
+++ b/www/kanban/jkanban.js
@@ -45,6 +45,7 @@
                 this.drake = '';
                 this.drakeBoard = '';
                 this.addItemButton = false;
+                this.cache = {};
                 defaults = {
                     element: '',
                     gutter: '15px',
@@ -84,6 +85,12 @@
                     this.options = __extendDefaults(defaults, arguments[0]);
                 }
 
+                var checkCache = function (boards) {
+                    Object.keys(self.cache).forEach(function (id) {
+                        if (boards.items[id]) { return; }
+                        delete self.cache[id];
+                    });
+                };
                 var removeUnusedTags = function (boards) {
                     var tags = self.options.getTags(boards);
                     var filter = self.options.tags || [];
@@ -355,6 +362,7 @@
                     // If it's a deletion, remove the item data
                     if (!board) {
                         delete boards.items[eid];
+                        delete self.cache[eid];
                         removeUnusedTags(boards);
                         self.options.refresh();
                         return;
@@ -414,7 +422,16 @@
                         }
                     }
                     if (element.body) {
-                        var html = self.renderMd(element.body);
+                        var html;
+                        if (self.cache[element.id] && self.cache[element.id].body === element.body) {
+                            html = self.cache[element.id].html;
+                        } else {
+                            html = self.renderMd(element.body);
+                            self.cache[element.id] = {
+                                body: element.body,
+                                html: html
+                            };
+                        }
                         var nodeBody = document.createElement('div');
                         nodeBody.classList.add('kanban-item-body');
                         $(nodeBody).on('click', 'a', function (e) {
@@ -604,6 +621,7 @@
                 this.setBoards = function (boards) {
                     var scroll = {};
                     // Fix the tags
+                    checkCache(boards);
                     removeUnusedTags(boards);
                     // Get horizontal scroll
                     var $el = $(self.element);