Skip to content

Conversation

yann300
Copy link
Contributor

@yann300 yann300 commented Sep 26, 2025

Allow rendering Videos tag using markdown.

@yann300 yann300 added the WIP label Sep 26, 2025
@yann300 yann300 changed the title render videos in md Render videos in md Sep 26, 2025
Copy link

netlify bot commented Sep 26, 2025

Deploy Preview for reliable-cocada-166884 ready!

Name Link
🔨 Latest commit 2a1b4de
🔍 Latest deploy log https://app.netlify.com/projects/reliable-cocada-166884/deploys/68e7a79ca5512400087a7a1c
😎 Deploy Preview https://deploy-preview-6414--reliable-cocada-166884.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@joeizang
Copy link
Collaborator

I added some css to enable and fix the aspect ratio. The logic is driven by this below
56.25% → 16:9 ratio (since 9 ÷ 16 = 0.5625 = 56.25%). So the paddingBottom helps
to achieve this. We could also achieve a 4:3 aspect ratio using the same concept which
means padding would be 70%

@joeizang
Copy link
Collaborator

joeizang commented Oct 2, 2025

The code LGTM

@yann300 yann300 added ready-to-review PR ready to review and removed WIP labels Oct 9, 2025
@yann300 yann300 merged commit 4737665 into master Oct 9, 2025
37 checks passed
@yann300 yann300 deleted the yann300-patch-66 branch October 9, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-review PR ready to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants