Skip to content

Conversation

@domlander
Copy link
Contributor

@domlander domlander commented Dec 8, 2025

What does this change?

Fixes the non-5:4 video display in the Flexible container slots.

Videos with a width shorter than the width of the slot will fill the width of the slot

Why?

Video should be rendered within a 5:4 slot above the tablet breakpoint. The progress bar should be constrained by the width of the video.

Video should be full-width up until the tablet breakpoint.

Screenshots

Before After
mobile - half-width h-mobile-before h-mobile-after
mobile - boosted b-mobile-before b-mobile-after
mobile - giga-boost g-mobile-before g-mobile-after
desktop - half-width h-desktop-before h-desktop-after
desktop - boosted b-desktop-before b-desktop-after
desktop - giga-boost g-desktop-before g-desktop-after

@github-actions
Copy link

github-actions bot commented Dec 8, 2025

@domlander domlander force-pushed the doml/fix-video-aspect-ratio-mobile branch from 03ae9b9 to 8619f7d Compare December 8, 2025 10:53
@domlander domlander self-assigned this Dec 8, 2025
@domlander domlander added fronts + curation maintenance Departmental tracking: maintenance work, not a fix or a feature labels Dec 8, 2025
@domlander domlander force-pushed the doml/self-hosted-video-dimensions branch 2 times, most recently from 5c7034e to 76713a8 Compare December 10, 2025 09:36
Base automatically changed from doml/self-hosted-video-dimensions to main December 10, 2025 09:51
@domlander domlander force-pushed the doml/fix-video-aspect-ratio-mobile branch from 8619f7d to 1212d17 Compare December 10, 2025 09:52
@domlander domlander force-pushed the doml/fix-video-aspect-ratio-mobile branch from 1212d17 to 7f1b86b Compare December 11, 2025 13:26
@domlander
Copy link
Contributor Author

Superseded by #15029

@domlander domlander closed this Dec 17, 2025
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.

2 participants