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

Add blank.mp4 to dist files #545

Merged
merged 2 commits into from
Apr 25, 2017
Merged

Conversation

silverwind
Copy link
Contributor

Link to related issue (if applicable)

#504

Sumary of proposed changes

For a complete "offline" experience, it's necessary to self-host the file, so include it in the dist files for that purpose.

@silverwind silverwind changed the title Add blank.html to dist files Add blank.mp4 to dist files Mar 25, 2017
For a complete "offline" experience, it's necessary to self-host the
file, so include it in the dist files for that purpose.
@silverwind
Copy link
Contributor Author

silverwind commented Mar 25, 2017

A cleaner approach could be to include the file in the form of a data-uri, in which case, the blankUrl option can be dropped, and plyr.js would gain 2395 bytes. Shall I go for it?

@sampotts
Copy link
Owner

I had that before - there was a reason I reverted to the remote file. I can't remember why...

@silverwind
Copy link
Contributor Author

Okay, then let's keep it like this. I'm fine with it.

@sampotts sampotts closed this Apr 25, 2017
@sampotts sampotts changed the base branch from master to develop April 25, 2017 02:39
@sampotts sampotts reopened this Apr 25, 2017
@sampotts sampotts merged commit 0b7d0de into sampotts:develop Apr 25, 2017
@sampotts
Copy link
Owner

Merged to develop so will go with v3.0.0 when it's done 👍

filips123 pushed a commit to filips123/plyr that referenced this pull request Nov 22, 2018
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