Fixed bug with API use on basic supported browsers

This commit is contained in:
Sam Potts 2015-08-21 21:36:15 +10:00
parent 9133247cdf
commit 6543fc1e5b
9 changed files with 33 additions and 17 deletions

View File

@ -1,5 +1,8 @@
# Changelog # Changelog
# v1.3.5
- Fixed bug with API use on basic supported browsers
# v1.3.4 # v1.3.4
- Code cleanup by @calvintam236 - Code cleanup by @calvintam236

2
dist/plyr.js vendored

File diff suppressed because one or more lines are too long

2
dist/sprite.svg vendored

File diff suppressed because one or more lines are too long

Before

Width:  |  Height:  |  Size: 5.1 KiB

After

Width:  |  Height:  |  Size: 5.1 KiB

View File

@ -6,7 +6,7 @@
<meta name="viewport" content="width=device-width, initial-scale=1"> <meta name="viewport" content="width=device-width, initial-scale=1">
<!-- Docs styles --> <!-- Docs styles -->
<link rel="stylesheet" href="//cdn.plyr.io/1.3.4/docs.css"> <link rel="stylesheet" href="//cdn.plyr.io/1.3.5/docs.css">
</head> </head>
<body> <body>
<main> <main>

View File

@ -8,10 +8,10 @@
<meta name="viewport" content="width=device-width, initial-scale=1"> <meta name="viewport" content="width=device-width, initial-scale=1">
<!-- Styles --> <!-- Styles -->
<link rel="stylesheet" href="https://cdn.plyr.io/1.3.4/plyr.css?3"> <link rel="stylesheet" href="https://cdn.plyr.io/1.3.5/plyr.css">
<!-- Docs styles --> <!-- Docs styles -->
<link rel="stylesheet" href="https://cdn.plyr.io/1.3.4/docs.css?1"> <link rel="stylesheet" href="https://cdn.plyr.io/1.3.5/docs.css">
</head> </head>
<body> <body>
<header> <header>
@ -97,13 +97,13 @@
b.insertBefore(c, b.childNodes[0]); b.insertBefore(c, b.childNodes[0]);
} }
} }
})(document, "https://cdn.plyr.io/1.3.4/sprite.svg"); })(document, "https://cdn.plyr.io/1.3.5/sprite.svg");
</script> </script>
<!-- Plyr core script --> <!-- Plyr core script -->
<script src="https://cdn.plyr.io/1.3.4/plyr.js?1"></script> <script src="https://cdn.plyr.io/1.3.5/plyr.js"></script>
<!-- Docs script --> <!-- Docs script -->
<script src="https://cdn.plyr.io/1.3.4/docs.js?1"></script> <script src="https://cdn.plyr.io/1.3.5/docs.js"></script>
</body> </body>
</html> </html>

View File

@ -66,7 +66,12 @@ package = loadJSON(path.join(root, "package.json"));
// Load json // Load json
function loadJSON(path) { function loadJSON(path) {
try {
return JSON.parse(fs.readFileSync(path)); return JSON.parse(fs.readFileSync(path));
}
catch(err) {
return {};
}
} }
var build = { var build = {
@ -215,8 +220,12 @@ options = {
}, },
gzippedOnly: true gzippedOnly: true
} }
}, };
cdnpath = new RegExp(aws.cdn.bucket + "\/(\\d+\\.)?(\\d+\\.)?(\\*|\\d+)","gi");
// If aws is setup
if("cdn" in aws) {
var cdnpath = new RegExp(aws.cdn.bucket + "\/(\\d+\\.)?(\\d+\\.)?(\\*|\\d+)","gi");
}
// Publish version to CDN bucket // Publish version to CDN bucket
gulp.task("cdn", function () { gulp.task("cdn", function () {

View File

@ -1,6 +1,6 @@
{ {
"name": "plyr", "name": "plyr",
"version": "1.3.4", "version": "1.3.5",
"description": "A simple HTML5 media player using custom controls", "description": "A simple HTML5 media player using custom controls",
"homepage": "http://plyr.io", "homepage": "http://plyr.io",
"main": "gulpfile.js", "main": "gulpfile.js",

View File

@ -40,7 +40,7 @@ If you have any cool ideas or features, please let me know by [creating an issue
Check `docs/index.html` and `docs/dist/docs.js` for an example setup. Check `docs/index.html` and `docs/dist/docs.js` for an example setup.
**Heads up**, the example `index.html` file needs to be served from a webserver (such as Apache, Nginx, IIS or similar) unless you change the file sources to include http or https. e.g. change `//cdn.plyr.io/1.3.4/plyr.js` to `https://cdn.plyr.io/1.3.4/plyr.js` **Heads up**, the example `index.html` file needs to be served from a webserver (such as Apache, Nginx, IIS or similar) unless you change the file sources to include http or https. e.g. change `//cdn.plyr.io/1.3.5/plyr.js` to `https://cdn.plyr.io/1.3.5/plyr.js`
### Bower ### Bower
If bower is your thang, you can grab Plyr using: If bower is your thang, you can grab Plyr using:
@ -60,11 +60,11 @@ More info is on [npm](https://www.npmjs.com/package/ember-cli-plyr) and [GitHub]
If you want to use our CDN, you can use the following: If you want to use our CDN, you can use the following:
```html ```html
<link rel="stylesheet" href="https://cdn.plyr.io/1.3.4/plyr.css"> <link rel="stylesheet" href="https://cdn.plyr.io/1.3.5/plyr.css">
<script src="https://cdn.plyr.io/1.3.4/plyr.js"></script> <script src="https://cdn.plyr.io/1.3.5/plyr.js"></script>
``` ```
You can also access the `sprite.svg` file at `https://cdn.plyr.io/1.3.4/sprite.svg`. You can also access the `sprite.svg` file at `https://cdn.plyr.io/1.3.5/sprite.svg`.
### CSS ### CSS
If you want to use the default css, add the `plyr.css` file from /dist into your head, or even better use `plyr.less` or `plyr.sass` file included in `/src` in your build to save a request. If you want to use the default css, add the `plyr.css` file from /dist into your head, or even better use `plyr.less` or `plyr.sass` file included in `/src` in your build to save a request.

View File

@ -1,6 +1,6 @@
// ========================================================================== // ==========================================================================
// Plyr // Plyr
// plyr.js v1.3.4 // plyr.js v1.3.5
// https://github.com/selz/plyr // https://github.com/selz/plyr
// License: The MIT License (MIT) // License: The MIT License (MIT)
// ========================================================================== // ==========================================================================
@ -1914,6 +1914,10 @@
if (player.type == 'video' || player.type == 'audio') { if (player.type == 'video' || player.type == 'audio') {
// Bail if no support // Bail if no support
if (!player.supported.full) { if (!player.supported.full) {
// Successful setup
player.init = true;
// Don't inject controls if no full support
return; return;
} }