Merge pull request #3122 from plotly/3121-fix-diff-config

ignore private attributes in diffConfig

Author: antoinerg

src/plot_api/plot_api.js

@@ -2641,6 +2641,7 @@ function diffConfig(oldConfig, newConfig) {
     var key;
 
     for(key in oldConfig) {
+        if(key.charAt(0) === '_') continue;
         var oldVal = oldConfig[key];
         var newVal = newConfig[key];
         if(oldVal !== newVal) {

test/jasmine/assets/mock_lists.js

@@ -9,7 +9,7 @@ var svgMockList = [
     ['17', require('@mocks/17.json')],
     ['21', require('@mocks/21.json')],
     ['22', require('@mocks/22.json')],
-    ['airfoil', require('@mocks/airfoil.json')],
+    ['airfoil', require('@mocks/airfoil.json')], // important to keep because it's the only mock with config options
     ['annotations-autorange', require('@mocks/annotations-autorange.json')],
     ['axes_enumerated_ticks', require('@mocks/axes_enumerated_ticks.json')],
     ['axes_visible-false', require('@mocks/axes_visible-false.json')],