Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix poster image to match the video in the 'combined-tracks.html' example in sandbox #4164

Conversation

OwenEdwards
Copy link
Member

Description

Remove poster image which doesn't match the video from the 'combined-tracks.html' example in sandbox, and also fixed a typo.

Requirements Checklist

  • Feature implemented / Bug fixed
  • If necessary, more likely in a feature request than a bug fix
    • Change has been verified in an actual browser (Chome, Firefox, IE)
  • Reviewed by Two Core Contributors

@@ -25,7 +25,7 @@
<pre>open http://localhost:9999/sandbox/index.html</pre>
</div>

<video id="vid1" class="video-js vjs-default-skin" lang="en" controls preload="auto" width="640" height="264" poster="http://vjs.zencdn.net/v/oceans.png">
<video id="vid1" class="video-js vjs-default-skin" lang="en" controls preload="auto" width="640" height="264">
Copy link
Member

Choose a reason for hiding this comment

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

what about swapping to //d2zihajmogu5jn.cloudfront.net/elephantsdream/poster.png?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, I didn't know that the poster for ED was on the CDN. I can make that change - thanks @gkatsev!

@OwenEdwards OwenEdwards changed the title Remove poster image which doesn't match the video from the 'combined-tracks.html' example in sandbox Fix poster image to match the video in the 'combined-tracks.html' example in sandbox Mar 6, 2017
@gkatsev gkatsev merged commit 5ffe1cd into videojs:master Mar 6, 2017
@OwenEdwards OwenEdwards deleted the chore/remove-incorrect-poster-from-sandbox-example branch March 7, 2017 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants