fix: large post placeholder width regression, failing naturally sized…

… media
aeharding · Nov 24, 2024 · e70add5 · e70add5
1 parent ebbb5b5
commit e70add5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/features/media/MediaPlaceholder.module.css b/src/features/media/MediaPlaceholder.module.css
@@ -1,6 +1,9 @@
 .placeholderContainer {
   display: flex;
 
+  /* Sometimes the <img> natural size is loaded, but the media cannot be played */
+  overflow: hidden;
+
   &:global(.not-loaded) {
     align-items: center;
     justify-content: center;

diff --git a/src/features/post/inFeed/large/media/LargeFeedPostMedia.tsx b/src/features/post/inFeed/large/media/LargeFeedPostMedia.tsx
@@ -3,6 +3,7 @@ import { ComponentProps } from "react";
 
 import LargeFeedRedgifMedia from "#/features/media/external/redgifs/LargeFeedRedgifMedia";
 import { isRedgif } from "#/features/media/external/redgifs/helpers";
+import { cx } from "#/helpers/css";
 
 import usePostSrc from "../../usePostSrc";
 import LargeFeedMedia from "./LargeFeedMedia";
@@ -33,7 +34,7 @@ export default function LargeFeedPostMedia(
         src={src}
         autoPlay={!props.blur}
         alt={props.post.post.alt_text}
-        className={styles.lightbox}
+        className={cx(styles.lightbox, props.className)}
         shouldPortal
       />
     );