Can't use the usual reverse-order for splicing in depths of BFS layou…

…t on account of saving the indices #1992
cytoscape · Oct 23, 2017 · 391ce4e · 391ce4e
1 parent 66ca5a9
commit 391ce4e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/extensions/layout/breadthfirst.js b/src/extensions/layout/breadthfirst.js
@@ -193,11 +193,12 @@ BreadthFirstLayout.prototype.run = function(){
     for( let i = 0; i < depths.length; i++ ){
       let eles = depths[ i ];
 
-      for( let j = eles.length - 1; j >= 0; j-- ){
+      for( let j = 0; j < eles.length; j++ ){
         let ele = eles[ j ];
 
         if( ele == null ){
           eles.splice( j, 1 );
+          j--;
           continue;
         }