Chart curves redrawing is optimized

Author: demvlad

js/graph_config.js

@@ -24,6 +24,11 @@ function GraphConfig(graphConfig) {
     this.getGraphs = function() {
         return graphs;
     };
+    
+    let redrawChart = true;
+    this.setRedrawChart = function(isRedraw) {
+        redrawChart = isRedraw;
+    }
 
     /**
      * newGraphs is an array of objects like {label: "graph label", height:, fields:[{name: curve:{power:, MinMax:, steps:}, color:, }, ...]}
@@ -32,9 +37,12 @@ function GraphConfig(graphConfig) {
         graphs = newGraphs;
 
         hiddenGraphFields.clear();
-
-        notifyListeners();
+        if (redrawChart) {
+            notifyListeners();
+        }
     };
+    
+    
 
     /**
      * Convert the given graph configs to make them appropriate for the given flight log.

js/graph_config_dialog.js

@@ -460,29 +460,34 @@ function GraphConfigurationDialog(dialog, onSave) {
             return;
         }
 
-        if (cfgMustBeRestored)
-            onSave(prevCfg);
+        if (cfgMustBeRestored) {
+            const noRedraw = false;
+            onSave(prevCfg, noRedraw);
+        }
     });
 
     $(".graph-configuration-dialog-save").click(function() {
         if (isMinMaxContextMenuActive())
             closeMinMaxContextMenu();
 
         cfgMustBeRestored = false;
-        onSave(convertUIToGraphConfig());
+        const noRedraw = true;
+        onSave(convertUIToGraphConfig(), noRedraw);
     });
 
     $(".graph-configuration-dialog-cancel").click(function() {
         if (isMinMaxContextMenuActive())
             closeMinMaxContextMenu();
-
+        
+        const noRedraw = !cfgMustBeRestored;
+        onSave(prevCfg, noRedraw);
         cfgMustBeRestored = false;
-        onSave(prevCfg);
     });
 
     function RefreshCharts() {
         cfgMustBeRestored = true;
-        onSave(convertUIToGraphConfig());
+        const noRedraw = false;
+        onSave(convertUIToGraphConfig(), noRedraw);
     }
 
     var

js/main.js

@@ -997,10 +997,10 @@ function BlackboxLogViewer() {
         GpxExporter(flightLog).dump(onSuccess);
     }
 
-    function newGraphConfig(newConfig) {
+    function newGraphConfig(newConfig, noRedraw) {
         lastGraphConfig = graphConfig; // Remember the last configuration.
         graphConfig = newConfig;
-
+        activeGraphConfig.setRedrawChart(noRedraw ? false : true);
         activeGraphConfig.adaptGraphs(flightLog, graphConfig, false);
 
         prefs.set('graphConfig', graphConfig);
@@ -1410,8 +1410,8 @@ function BlackboxLogViewer() {
         }
 
         var
-            graphConfigDialog = new GraphConfigurationDialog($("#dlgGraphConfiguration"), function(newConfig) {
-                newGraphConfig(newConfig);
+            graphConfigDialog = new GraphConfigurationDialog($("#dlgGraphConfiguration"), function(newConfig, redrawChart) {
+                newGraphConfig(newConfig, redrawChart);
             }),
 
             headerDialog = new HeaderDialog($("#dlgHeaderDialog"), function(newSysConfig) {