fix(core): initial muted state is not set on media embeds

ref vime-js#111
SidneySaints · Nov 21, 2021 · df0e6bc · df0e6bc
1 parent 64a658e
commit df0e6bc
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/packages/core/src/components/providers/dailymotion/dailymotion.tsx b/packages/core/src/components/providers/dailymotion/dailymotion.tsx
@@ -186,11 +186,14 @@ export class Dailymotion implements MediaProvider<HTMLVmEmbedElement> {
     this.dispatch = createProviderDispatcher(this);
     this.dispatch('viewType', ViewType.Video);
     this.onVideoIdChange();
-    this.initialMuted = this.muted;
     this.internalState.muted = this.muted;
     this.defaultInternalState = { ...this.internalState };
   }
 
+  componentDidLoad() {
+    this.initialMuted = this.muted;
+  }
+
   private getOrigin() {
     return 'https://www.dailymotion.com';
   }

diff --git a/packages/core/src/components/providers/vimeo/vimeo.tsx b/packages/core/src/components/providers/vimeo/vimeo.tsx
@@ -184,10 +184,13 @@ export class Vimeo implements MediaProvider<HTMLVmEmbedElement> {
     this.dispatch = createProviderDispatcher(this);
     this.dispatch('viewType', ViewType.Video);
     this.onVideoIdChange();
-    this.initialMuted = this.muted;
     this.defaultInternalState = { ...this.internalState };
   }
 
+  componentDidLoad() {
+    this.initialMuted = this.muted;
+  }
+
   disconnectedCallback() {
     this.cancelTimeUpdates();
     this.pendingPlayRequest = undefined;

diff --git a/packages/core/src/components/providers/youtube/youtube.tsx b/packages/core/src/components/providers/youtube/youtube.tsx
@@ -145,10 +145,13 @@ export class YouTube implements MediaProvider<HTMLVmEmbedElement> {
     this.dispatch = createProviderDispatcher(this);
     this.dispatch('viewType', ViewType.Video);
     this.onVideoIdChange();
-    this.initialMuted = this.muted;
     this.defaultInternalState = { ...this.internalState };
   }
 
+  componentDidLoad() {
+    this.initialMuted = this.muted;
+  }
+
   /** @internal */
   @Method()
   async getAdapter() {