Floor media dimensions to prevent blurhash errors (matrix-org#8157)

Author: robintown

src/settings/enums/ImageSize.ts

@@ -52,9 +52,9 @@ export function suggestedSize(size: ImageSize, contentSize: Dimensions, maxHeigh
 
     if (constrainedSize.h * aspectRatio < constrainedSize.w) {
         // Height dictates width
-        return { w: constrainedSize.h * aspectRatio, h: constrainedSize.h };
+        return { w: Math.floor(constrainedSize.h * aspectRatio), h: constrainedSize.h };
     } else {
         // Width dictates height
-        return { w: constrainedSize.w, h: constrainedSize.w / aspectRatio };
+        return { w: constrainedSize.w, h: Math.floor(constrainedSize.w / aspectRatio) };
     }
 }

test/settings/enums/ImageSize-test.ts

@@ -34,5 +34,9 @@ describe("ImageSize", () => {
             const size = suggestedSize(ImageSize.Normal, { w: null, h: null });
             expect(size).toStrictEqual({ w: 324, h: 324 });
         });
+        it("returns integer values", () => {
+            const size = suggestedSize(ImageSize.Normal, { w: 642, h: 350 }); // does not divide evenly
+            expect(size).toStrictEqual({ w: 324, h: 176 });
+        });
     });
 });