Skip to content

Use correct aspect ratio for poster image in self-hosted video#15746

Draft
domlander wants to merge 1 commit intomainfrom
doml/sh-video-fix-poster-image-aspect-ratio
Draft

Use correct aspect ratio for poster image in self-hosted video#15746
domlander wants to merge 1 commit intomainfrom
doml/sh-video-fix-poster-image-aspect-ratio

Conversation

@domlander
Copy link
Copy Markdown
Contributor

@domlander domlander commented Apr 23, 2026

What does this change?

The poster image requests a crop that matches the aspect ratio of the video.

Why?

A self-hosted video in a Card component that is not 5:4 was using a poster image that is cropped to 5:4. This will be important once we use video that isn't 5:4 and doesn't autoplay.

Screenshots

Before After
before after

@domlander domlander force-pushed the doml/sh-video-fix-poster-image-aspect-ratio branch from 5098304 to a6c7ea5 Compare April 23, 2026 11:37
@domlander domlander self-assigned this Apr 23, 2026
@domlander domlander added run_chromatic Runs chromatic when label is applied fronts + curation maintenance Departmental tracking: maintenance work, not a fix or a feature labels Apr 23, 2026
@github-actions github-actions Bot removed the run_chromatic Runs chromatic when label is applied label Apr 23, 2026
abeddow91 added a commit that referenced this pull request Apr 30, 2026
This is a temporary hotfix whilst #15746 is developed.
abeddow91 added a commit that referenced this pull request Apr 30, 2026
This is a temporary hotfix whilst #15746 is developed.
abeddow91 added a commit that referenced this pull request Apr 30, 2026
* Use an aspectRatio prop for getOptimisedPosterImage

* Infer an aspect ratio from where the video is letterboxed.
This is a temporary hotfix whilst #15746 is developed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fronts + curation maintenance Departmental tracking: maintenance work, not a fix or a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant