The patch was committed in Drupal core with css-placeholder token
#2998091: Remote videos overflow their containing element

media-oembed-iframe--remote-video--varbase-media-hero-slider.html.twig

<!DOCTYPE html>
<html>
  <head>
    <style>
      iframe {
        position: absolute;
        left: 0;
        top: 0;
        right: 0;
        bottom: 0;
        margin: 0;
        height: 100%;
        width: 100%;
      }
    </style>
    <script src="{{ base_path }}{{ varbase_heroslider_media_path }}/js/oembed-frame.heroslider.{{ provider }}.js"></script>
  </head>
  <body style="margin: 0">
    {{ media|raw }}
  </body>
</html>

To media-oembed-iframe--remote-video--varbase-media-hero-slider.html.twig

{#
/**
 * @file
 * Varbase Medi Hero slider theme implementation to display an oEmbed resource.
 *
 * Display in an iframe and add the right behavior player for the provider.
 *
 * @ingroup themeable
 */
#}
<!DOCTYPE html>
<html>
  <head>
    <css-placeholder token="{{ placeholder_token }}">
    <script src="{{ base_path }}{{ varbase_heroslider_media_path }}/js/oembed-frame.heroslider.{{ provider }}.js"></script>
  </head>
  <body style="margin: 0">
    {{ media|raw }}
  </body>
</html>

https://git.drupalcode.org/project/drupal/blob/3ab46e219dcb6eaa4e7373dbf...
https://git.drupalcode.org/project/drupal/blob/3ab46e219dcb6eaa4e7373dbf...
https://git.drupalcode.org/project/drupal/blob/3ab46e219dcb6eaa4e7373dbf...

Comments

RajabNatshah created an issue. See original summary.

  • RajabNatshah committed 788d46a on 8.x-7.x
    Issue #3101609: Change [Varbase Media Hero Slider] media oembed iframe...
rajab natshah’s picture

Assigned: rajab natshah » mohammed j. razem
Status: Active » Needs review
Issue tags: +varbase-8.7.4, +varbase-8.8.0
rajab natshah’s picture

Assigned: mohammed j. razem » Unassigned
rajab natshah’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.