Revert "Masonry: Update fullWidthLayout to not floor column widths" (#…

…3885)
pinterest · Nov 19, 2024 · 8d905a4 · 8d905a4
1 parent ef3e902
commit 8d905a4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/gestalt/src/Masonry/fullWidthLayout.ts b/packages/gestalt/src/Masonry/fullWidthLayout.ts
@@ -45,7 +45,7 @@ const fullWidthLayout = <T>({
   // original implementation takes with CSS.
   const colguess = Math.floor(width / idealColumnWidth);
   const columnCount = Math.max(Math.floor((width - colguess * gutter) / idealColumnWidth), minCols);
-  const columnWidth = width / columnCount - gutter;
+  const columnWidth = Math.floor(width / columnCount) - gutter;
   const columnWidthAndGutter = columnWidth + gutter;
   const centerOffset = gutter / 2;
 

diff --git a/playwright/masonry/flexible-resize.spec.ts b/playwright/masonry/flexible-resize.spec.ts
@@ -76,7 +76,7 @@ test.describe('Masonry: flexible resize', () => {
     const itemRectsAfter = await Promise.all(
       gridItemsAfter.map((gridItemAfter) => gridItemAfter.boundingBox()),
     );
-    expect(Math.floor(itemRectsAfter[0]?.width)).toBe(273);
+    expect(itemRectsAfter[0]?.width).toBe(273);
     expect(itemRectsAfter[0]?.height).toBe(216);
 
     // Get new sizes of grid items.