Merge pull request #3294 from soetji/fix/breadthfirst-in-cy-extent

Fix breadthfirst layout to stay inside viewport on fit: false and no boundingBox
cytoscape · Nov 11, 2024 · dd0e9f8 · dd0e9f8
2 parents 14b3730 + f2b8b8d
commit dd0e9f8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/extensions/layout/breadthfirst.js b/src/extensions/layout/breadthfirst.js
@@ -48,8 +48,9 @@ BreadthFirstLayout.prototype.run = function(){
   const maximal = options.acyclic || options.maximal || options.maximalAdjustments > 0; // maximalAdjustments for compat. w/ old code; also, setting acyclic to true sets maximal to true
 
   const hasBoundingBox = !!options.boundingBox;
+  const cyExtent = cy.extent();
   const bb = math.makeBoundingBox( hasBoundingBox ? options.boundingBox : {
-    x1: 0, y1: 0, w: cy.width(), h: cy.height()
+    x1: cyExtent.x1, y1: cyExtent.y1, w: cyExtent.w, h: cyExtent.h
   } );
 
   let roots;