oEmbed videos do not allow full screen

Created on 9 October 2018, over 5 years ago
Updated 9 September 2023, 10 months ago

Problem/Motivation

Embedding a video as media remote_video entity does not allow Fullscreen for the video. On the inner frame it says allowfullscreen=""
To allow fullscreen, it would have to say
allowfullscreen="allowfullscreen"
I cannot find how one would influence this, this must have to do with the Oembed implementation.

Filed as a bug report since Fullscreen is a basic feature IMHO.

Proposed resolution

For the maximum cross-browser compatibility copy youtube and add allowfullscreen allow="fullscreen" to the iframe tag.

This aligns with the iframe documentation - https://developer.mozilla.org/en-US/docs/Web/HTML/Element/iframe

Remaining tasks

User interface changes

remote video embeds now are allowed to go fullscreen.

API changes

None

Data model changes

None

Release notes snippet

N/a

✨ Feature request
Status

Closed: cannot reproduce

Version

11.0 πŸ”₯

Component
MediaΒ  β†’

Last updated about 16 hours ago

Created by

πŸ‡©πŸ‡ͺGermany eigentor

Live updates comments and jobs are added and updated live.
  • Needs manual testing

    The change/bugfix cannot be fully demonstrated by automated testing, and thus requires manual testing in a variety of environments.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡·πŸ‡΄Romania vasike Ramnicu Valcea

    i know it happened and i had this issue .... solving with override template ... years ago.

    Checked again ... and i couldn't replicate anymore ... with both drupal 9 and latest 10.1
    But also the browser were updated ...
    And maybe also what providers provides for their iframes

    for example:
    Vimeo provides both allowfullscreen="" allow="fullscreen"
    YouTube provides only allowfullscreen=""

    So. not sure if this issue is still a "valid one" ... And also the related one ...

    Could we close it as Closed (cannot reproduce) ?

  • Status changed to Postponed: needs info about 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    Is anyone seeing this issue still?

    If no response this potentially could be closed.

  • Status changed to Closed: cannot reproduce 10 months ago
  • πŸ‡ΊπŸ‡ΈUnited States Chris Matthews

    I also cannot reproduce this issue on a clean drupal 10.1.3 install using the latest Firefox (117), Chrome (116) or Safari (Version 16.5.1) so I believe this can be closed as cannot reproduce, but please feel free to reopen if necessary.

  • πŸ‡ΊπŸ‡ΈUnited States johns996 Marquette, MI

    I had this issue show up on my newly updated 10.1 site. #34 fixed the issue and I can go full screen again. I don't know if it makes a difference but my video embeds are all fields within paragraphs.

Production build 0.69.0 2024