Skip to content

Commit 91464af

Browse files
committed
Merge branch 'fix/bounding-box-padding' into 'develop'
Added padding to bounding box See merge request genaiic-reusable-assets/engagement-artifacts/genaiic-idp-accelerator!298
2 parents e1d5d32 + 396d0f4 commit 91464af

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/ui/src/components/document-viewer/VisualEditorModal.jsx

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,13 +117,14 @@ const BoundingBox = memo(({ box, page, currentPage, imageRef, zoomLevel = 1, pan
117117
return null;
118118
}
119119

120+
const padding = 5;
120121
const bbox = box.boundingBox;
121122

122123
// Calculate position and size directly on the transformed image
123-
const finalLeft = bbox.left * dimensions.transformedWidth + dimensions.transformedOffsetX;
124-
const finalTop = bbox.top * dimensions.transformedHeight + dimensions.transformedOffsetY;
125-
const finalWidth = bbox.width * dimensions.transformedWidth;
126-
const finalHeight = bbox.height * dimensions.transformedHeight;
124+
const finalLeft = (bbox.left * dimensions.transformedWidth + dimensions.transformedOffsetX) - padding;
125+
const finalTop = (bbox.top * dimensions.transformedHeight + dimensions.transformedOffsetY) - padding;
126+
const finalWidth = (bbox.width * dimensions.transformedWidth) + padding * 2;
127+
const finalHeight = (bbox.height * dimensions.transformedHeight) + padding * 2;
127128

128129
// Position the bounding box directly without additional transforms
129130
const style = {

0 commit comments

Comments
 (0)