diffConfig always returns true if modeBarButtons (array of array) is populated

diffConfig function has bug where it always returns true if modeBarButtons (array of array) is populated. This results in unnecessary re-rendering. The function should check if the array member is array inside a for loop, It should be -

/*

• simple diff for config - for now, just treat all changes as equivalent
  */
  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) {
  if(Lib.isPlainObject(oldVal) && Lib.isPlainObject(newVal)) {
  if(diffConfig(oldVal, newVal)) {
  return true;
  }
  } else if(Array.isArray(oldVal) && Array.isArray(newVal)) {
  if(oldVal.length !== newVal.length) {
  return true;
  }

           for(var i = 0; i < oldVal.length; i++) {
               if(oldVal[i] !== newVal[i]) {
   				**if(Array.isArray(oldVal[i]) && Array.isArray(newVal[i])){
   					if(diffConfig(oldVal[i], newVal[i])) {
   						return true;
   					}
   				}							
                   else** if(Lib.isPlainObject(oldVal[i]) && Lib.isPlainObject(newVal[i])) {
                       if(diffConfig(oldVal[i], newVal[i])) {
                           return true;
                       }
                   } else {
                       return true;
                   }
               }
           }
       } else {
           return true;
       }
   }
  

  }
  }

e.g Config object used -

const plotlyConfig: Partial = {
responsive: true,
displaylogo: false,
displayModeBar: true,
doubleClick: false,
showTips: false,
scrollZoom: false,
modeBarButtons: [
[
'select2d',
'lasso2d',
'zoom2d',
'pan2d',
'resetScale2d',
],
],
locale: language,
locales: locales
}

diffConfig 関数を見つけ、ネストされた modeBarButtons 配列を含む plotlyConfig の例を再現ケースとして使用します。等価なネストされた配列を比較し、変更されていない設定が異なるものとして報告されないことを確認して、不要な再レンダリングを避けます。この issue ではファイルもテストも指定されていません。