![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 4 years ago
Last modified 4 years ago
#57076 new enhancement
Allow fullscreen in oembed videos
| Reported by: | booteille | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Awaiting Review |
| Component: | Embeds | Version: | |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
Hi!
We're trying to enable fullscreen from a video loaded through oembed (and not by copying directly the iframe) from PeerTube.
Here is the link to the video: https://video.davduf.net/w/4V6DMLcQDGQHnZ9wf6h4KH
And to the oembed: https://video.davduf.net/services/oembed/?url=https://video.davduf.net/w/4V6DMLcQDGQHnZ9wf6h4KH
I tried to understand why it was not working and found this piece of code: https://github.com/WordPress/WordPress/blob/82f3535083c0d8859bd3b8333fe006394978c09e/wp-includes/embed.php#L926
I guess it's because allowed attributes do not include allowfullscreen but I am not confident enough with WP code to be sure it's bound to this part.
Anyway, would it be possible to allow the allowfullscreen attribute in iframe loaded through oembed?
Thanks.
