Skip to content
This repository has been archived by the owner on Dec 10, 2020. It is now read-only.

fix muting video before play (muted on the video element) #205

Merged
merged 1 commit into from
May 23, 2016
Merged

fix muting video before play (muted on the video element) #205

merged 1 commit into from
May 23, 2016

Conversation

brandonocasey
Copy link
Contributor

rebuild swf for change

@gkatsev
Copy link
Member

gkatsev commented May 20, 2016

No need to include the built swf (we should probably just gitignore it), we'll rebuild it when deploying.

@@ -128,6 +128,10 @@ package{
_app.model.preload = String(loaderInfo.parameters.preload);
}

if(loaderInfo.parameters.muted != undefined){
_app.model.muted = loaderInfo.parameters.muted;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this basically loads in the muted attribute value when set? I can't believe that we this was the issue, lol.

@gkatsev
Copy link
Member

gkatsev commented May 20, 2016

LGTM

@brandonocasey
Copy link
Contributor Author

Should I just git rm the dist dir? or just dist/video-js.swf

@dmlap
Copy link
Member

dmlap commented May 23, 2016

@brandonocasey just remove the dist update from this PR (git reset HEAD dist/, I think).

@brandonocasey
Copy link
Contributor Author

brandonocasey commented May 23, 2016

@dmlap guess I committed it again when I added the if check for loop, I fixed it

@brandonocasey brandonocasey merged commit 46619bd into videojs:master May 23, 2016
@brandonocasey brandonocasey deleted the fix/muted-at-start branch May 23, 2016 18:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants