From f5ee2740529e23132ed0b1ff97df96479e59a31e Mon Sep 17 00:00:00 2001 From: jrking4 Date: Thu, 18 Jan 2018 09:40:00 -0500 Subject: [PATCH] Update use of getBoundingRect in Guides Example (#4631) * Update use of getBoundingRect in Guides Example Aligning Guidelines Example broken. Update to use getBoundingRect() function * Ensure getBoundingRect is called only once --- lib/aligning_guidelines.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/aligning_guidelines.js b/lib/aligning_guidelines.js index b78a4935..68c6b3f8 100644 --- a/lib/aligning_guidelines.js +++ b/lib/aligning_guidelines.js @@ -66,8 +66,9 @@ function initAligningGuidelines(canvas) { activeObjectCenter = activeObject.getCenterPoint(), activeObjectLeft = activeObjectCenter.x, activeObjectTop = activeObjectCenter.y, - activeObjectHeight = activeObject.getBoundingRectHeight() / viewportTransform[3], - activeObjectWidth = activeObject.getBoundingRectWidth() / viewportTransform[0], + activeObjectBoundingRect = activeObject.getBoundingRect(), + activeObjectHeight = activeObjectBoundingRect.height / viewportTransform[3], + activeObjectWidth = activeObjectBoundingRect.width / viewportTransform[0], horizontalInTheRange = false, verticalInTheRange = false, transform = canvas._currentTransform; @@ -84,8 +85,9 @@ function initAligningGuidelines(canvas) { var objectCenter = canvasObjects[i].getCenterPoint(), objectLeft = objectCenter.x, objectTop = objectCenter.y, - objectHeight = canvasObjects[i].getBoundingRectHeight() / viewportTransform[3], - objectWidth = canvasObjects[i].getBoundingRectWidth() / viewportTransform[0]; + objectBoundingRect = canvasObjects[i].getBoundingRect(), + objectHeight = objectBoundingRect.height / viewportTransform[3], + objectWidth = objectBoundingRect.width / viewportTransform[0]; // snap by the horizontal center line if (isInRange(objectLeft, activeObjectLeft)) {