Pārlūkot izejas kodu

fix(large-video): do not try to show background on safari with webrtc (#2606)

The animation for toggling filmstrip visibility was lagging on
Safari. Even though the background video is set to hidden, it is
still causing issues. Setting the background to display none
instead does help but might interfere with animations. So instead
do the easy thing and re-use logic used for Firefox to not show
the background video.
master
virtuacoplenny 7 gadus atpakaļ
vecāks
revīzija
bfb45ed0e8
1 mainītis faili ar 5 papildinājumiem un 1 dzēšanām
  1. 5
    1
      modules/UI/videolayout/VideoContainer.js

+ 5
- 1
modules/UI/videolayout/VideoContainer.js Parādīt failu

687
      * @returns {void}
687
      * @returns {void}
688
      */
688
      */
689
     _updateBackground() {
689
     _updateBackground() {
690
-        if (browser.isFirefox() || browser.isTemasysPluginUsed()) {
690
+        // Do not the background display on browsers that might experience
691
+        // performance issues from the presence of the background.
692
+        if (browser.isFirefox()
693
+                || browser.isSafariWithWebrtc()
694
+                || browser.isTemasysPluginUsed()) {
691
             return;
695
             return;
692
         }
696
         }
693
 
697
 

Notiek ielāde…
Atcelt
Saglabāt