Merge pull request #74 from ulaharbmg/JS-355

JS-355: Video image should be shown while video is loading in fullscreen

Author: uharbachou

app/code/Magento/ProductVideo/view/frontend/web/js/fotorama-add-video-events.js

@@ -524,7 +524,6 @@ define([
 
                     $(this).removeClass('video-unplayed');
                     $(this).find('.' + PV).productVideoLoader();
-                    $(this).find('img').hide();
 
                     if (!self.isFullscreen) {
                         self._showCloseVideo();

app/code/Magento/ProductVideo/view/frontend/web/js/load-player.js

@@ -189,6 +189,7 @@ define(['jquery', 'jquery/ui'], function ($) {
                              */
                             'onReady': function onPlayerReady() {
                                 self._player.getDuration();
+                                self.element.closest('.fotorama-video-container').find('img').hide();
                             },
 
                             /**
@@ -334,7 +335,8 @@ define(['jquery', 'jquery/ui'], function ($) {
             this._player = window.$f(this.element.children(':first')[0]);
 
             // Froogaloop throws error without a registered ready event
-            this._player.addEvent('ready', function () {
+            this._player.addEvent('ready', function (id) {
+                $('#' + id).closest('.fotorama-video-container').find('img').hide();
             });
         },