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
}

Localisez la fonction diffConfig et utilisez l’exemple plotlyConfig avec des tableaux modeBarButtons imbriqués comme cas de reproduction. Comparez les tableaux imbriqués équivalents et vérifiez que les configurations inchangées ne sont pas signalées comme différentes, afin d’éviter le re-rendu inutile ; l’issue ne nomme aucun fichier ni test.