linterna-magica-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[linterna-magica-commit] [323] Shows the Linterna M?\195? \161gica inter


From: Ivaylo Valkov
Subject: [linterna-magica-commit] [323] Shows the Linterna M?\195? \161gica interface when the YouTube HTML5 interface
Date: Sat, 03 Nov 2012 14:40:31 +0000

Revision: 323
          
http://svn.sv.gnu.org/viewvc/?view=rev&root=linterna-magica&revision=323
Author:   valkov
Date:     2012-11-03 14:40:30 +0000 (Sat, 03 Nov 2012)
Log Message:
-----------
Shows the Linterna M?\195?\161gica interface when the YouTube HTML5 interface
renders a fallback to flash warning, there is no flash plugin
installed and HTML5 has higher priority than Linterna M?\195?\161gica. Fixes
bugs #37630. Related bugs #35992.

Ticket Links:
------------
    http://savannah.gnu.org/bugs/?37630
    http://savannah.gnu.org/bugs/?35992

Modified Paths:
--------------
    trunk/src/lm_site_youtube.js

Modified: trunk/src/lm_site_youtube.js
===================================================================
--- trunk/src/lm_site_youtube.js        2012-10-26 10:55:16 UTC (rev 322)
+++ trunk/src/lm_site_youtube.js        2012-11-03 14:40:30 UTC (rev 323)
@@ -610,6 +610,35 @@
            // will not be displaced.
            html5_wrapper.style.setProperty('display', "none", "important");
 
+           // If the HTML5 player has a fallback CSS class and LM can
+           // access it it means there is no other option for
+           // playback. We take over the player.
+           //
+           // See bug 37630
+           // https://savannah.nongnu.org/bugs/index.php?37630
+           if (this.priority.html5 > this.priority.self)
+           {
+               // It should be available, because it is used above in
+               // this function
+               id = id ? id : object_data.linterna_magica_id;
+
+               this.show_lm_interface(object_data.linterna_magica_id);
+           
+               // It should be available, because it is used above in
+               // this functon.
+               toggle_plugin = toggle_plugin  ? toggle_plugin :
+                   document.getElementById("linterna-magica-"+
+                                           "toggle-plugin-"+id);
+
+               toggle_plugin.parentNode.removeChild(toggle_plugin);
+
+               toggle_plugin_header =
+                   document.getElementById("linterna-magica-"+
+                                           "toggle-plugin-header-"+id);
+               
+               
toggle_plugin_header.parentNode.removeChild(toggle_plugin_header);
+           }
+           
            // Hide site controls and video container. Should not
            // overlap with Linterna Magica now. Fixes the overlapping
            // of HTML5 player and LM.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]