Conversation
|
This needs a rebase and conflict resolution now because of #15456 |
|
PS. In every browser I tried to load a direct mp4 file, the iframe seemed to include video controls (on 6.x). Was there a specific situation where this was helping? Regardless, this looks fine. |
f87e6dd to
b984afb
Compare
|
You're right, and my premise was wrong. I checked in Chromium: pointing an iframe at an What's actually left is too thin to justify the change: the iframe viewer sets Closing this. The |
A direct video URL like
https://example.com/clip.mp4is rendered inside an<iframe>— the same element used for YouTube and Vimeo — so it gets no playback controls and relies on whatever the browser decides to do with a bare video document.The component already distinguishes these via its
isVideocomputed, so this renders them in a real<video controls>element instead. Lazy-loading behaviour is unchanged; thesrcis still gated onisVisible.Split out of #11871 so it can land on its own.