do not force autoplay

friends-of-contao · May 24, 2018 · cdb5705 · cdb5705
1 parent 695387a
commit cdb5705
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 12 deletions.
diff --git a/src/Elements/Vimeo.php b/src/Elements/Vimeo.php
@@ -35,7 +35,6 @@ protected function compile()
                 $objSplash = new \stdClass();
                 $this->addImageToTemplate($objSplash, $this->arrData, null, null, $objFile);
                 $this->Template->splashImage = $objSplash;
-                $this->autoplay = true;
             }
             else
             {

diff --git a/src/Elements/YouTube.php b/src/Elements/YouTube.php
@@ -33,7 +33,6 @@ protected function compile()
                 $objSplash       = new \stdClass();
                 $this->addImageToTemplate($objSplash, $this->arrData, null, null, $objFile);
                 $this->Template->splashImage = $objSplash;
-                $this->autoplay              = true;
             } else {
                 $this->videoSplash = false;
             }

diff --git a/src/Resources/contao/dca/tl_content.php b/src/Resources/contao/dca/tl_content.php
@@ -27,22 +27,21 @@
     'label'                   => &$GLOBALS['TL_LANG']['tl_content']['youtubeNoCookie'],
     'exclude'                 => true,
     'inputType'               => 'checkbox',
-    'eval'                    => array('tl_class'=>'w50', 'tl_style'=>'height:auto;'),
+    'eval'                    => array('tl_class'=>'clr', 'tl_style'=>'height:auto;'),
     'sql'                     => "char(1) NOT NULL default ''"
 );
 
 /**
  * Palettes
  */
-PaletteManipulator::create()
-    ->addField('videoSplash', 'player_legend', PaletteManipulator::POSITION_APPEND)
-    ->applyToPalette('youtube', 'tl_content')
-    ->applyToPalette('vimeo', 'tl_content');
-
 PaletteManipulator::create()
     ->addField('youtubeNoCookie', 'player_legend', PaletteManipulator::POSITION_APPEND)
     ->applyToPalette('youtube', 'tl_content');
 
+PaletteManipulator::create()
+    ->addField('videoSplash', 'player_legend', PaletteManipulator::POSITION_APPEND)
+    ->applyToPalette('youtube', 'tl_content')
+    ->applyToPalette('vimeo', 'tl_content');
 
 $GLOBALS['TL_DCA']['tl_content']['palettes']['__selector__'][] = 'videoSplash';
 $GLOBALS['TL_DCA']['tl_content']['subpalettes']['videoSplash'] = 'singleSRC,size';
diff --git a/src/Resources/contao/templates/ce_vimeo.html5 b/src/Resources/contao/templates/ce_vimeo.html5
@@ -11,8 +11,8 @@
       document.getElementById('videoSplash<?= $this->id ?>').addEventListener('click', function(e) {
         var iframe = document.createElement('iframe');
         iframe.src = this.dataset['video'];
-        iframe.width = <?= $this->width; ?>;
-        iframe.height = <?= $this->height; ?>;
+        iframe.width = <?= $this->width ?>;
+        iframe.height = <?= $this->height ?>;
         iframe.allowfullscreen = '';
         this.parentNode.replaceChild(iframe, this);
       });

diff --git a/src/Resources/contao/templates/ce_youtube.html5 b/src/Resources/contao/templates/ce_youtube.html5
@@ -11,8 +11,8 @@
       document.getElementById('videoSplash<?= $this->id ?>').addEventListener('click', function(e) {
         var iframe = document.createElement('iframe');
         iframe.src = this.dataset['video'];
-        iframe.width = <?= $this->width; ?>;
-        iframe.height = <?= $this->height; ?>;
+        iframe.width = <?= $this->width ?>;
+        iframe.height = <?= $this->height ?>;
         iframe.allowfullscreen = '';
         this.parentNode.replaceChild(iframe, this);
       });