Merge branch 'master' of github.com:selz/plyr

Removed captions being read by screen readers
Merge pull request #108 from ChristianPV/patch-2
2015-08-11 22:38:45 +10:00 · 2015-08-11 22:38:06 +10:00 · 2015-08-10 23:31:31 +10:00 · 2015-08-10 10:03:21 -03:00 · 2015-08-09 09:29:33 +10:00 · 2015-08-08 21:02:55 +10:00
8 changed files with 23 additions and 19 deletions
--- a/changelog.md
+++ b/changelog.md
@ -1,5 +1,8 @@
 # Changelog

+# v1.3.3
+- Removed captions being read by screen readers
+
 # v1.3.2
 - Voiceover fix for captions

--- a/dist/plyr.js
+++ b/dist/plyr.js
--- a/docs/error.html
+++ b/docs/error.html
@ -6,7 +6,7 @@
        <meta name="viewport" content="width=device-width, initial-scale=1">

        <!-- Docs styles -->
-        <link rel="stylesheet" href="//cdn.plyr.io/1.3.2/docs.css">
+        <link rel="stylesheet" href="//cdn.plyr.io/1.3.3/docs.css">
    </head>
    <body>
        <main>
--- a/docs/index.html
+++ b/docs/index.html
@ -8,10 +8,10 @@
        <meta name="viewport" content="width=device-width, initial-scale=1">

        <!-- Styles -->
-        <link rel="stylesheet" href="https://cdn.plyr.io/1.3.2/plyr.css?3">
+        <link rel="stylesheet" href="https://cdn.plyr.io/1.3.3/plyr.css?3">

        <!-- Docs styles -->
-        <link rel="stylesheet" href="https://cdn.plyr.io/1.3.2/docs.css?1">
+        <link rel="stylesheet" href="https://cdn.plyr.io/1.3.3/docs.css?1">
    </head>
    <body>
        <header>
@ -48,7 +48,7 @@
                            <source src="https://cdn.selz.com/plyr/1.0/movie.webm" type="video/webm">
                            
                            <!-- Text track file -->
-                            <track kind="captions" label="English" srclang="en" src="https://cdn.selz.com/plyr/1.0/en.vtt?2" default>
+                            <track kind="captions" label="English" srclang="en" src="https://cdn.selz.com/plyr/1.0/example_captions_en.vtt" default>
                            
                            <!-- Fallback for browsers that don't support the <video> element -->
                            <a href="https://cdn.selz.com/plyr/1.0/movie.mp4">Download</a>
@ -97,13 +97,13 @@
                    b.insertBefore(c, b.childNodes[0]);
                }
            }
-        })(document, "https://cdn.plyr.io/1.3.2/sprite.svg");
+        })(document, "https://cdn.plyr.io/1.3.3/sprite.svg");
        </script>

        <!-- Plyr core script -->
-        <script src="https://cdn.plyr.io/1.3.2/plyr.js?1"></script>
+        <script src="https://cdn.plyr.io/1.3.3/plyr.js?1"></script>

        <!-- Docs script -->
-        <script src="https://cdn.plyr.io/1.3.2/docs.js?1"></script>
+        <script src="https://cdn.plyr.io/1.3.3/docs.js?1"></script>
    </body>
 </html>
--- a/package.json
+++ b/package.json
@ -1,6 +1,6 @@
 {
  "name": "plyr",
-  "version": "1.3.2",
+  "version": "1.3.3",
  "description": "A simple HTML5 media player using custom controls",
  "homepage": "http://plyr.io",
  "main": "gulpfile.js",
--- a/readme.md
+++ b/readme.md
@ -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. 

-**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.2/plyr.js` to `https://cdn.plyr.io/1.3.2/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.3/plyr.js` to `https://cdn.plyr.io/1.3.3/plyr.js`

 ### Bower
 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:

 ```html
-<link rel="stylesheet" href="https://cdn.plyr.io/1.3.2/plyr.css">
-<script src="https://cdn.plyr.io/1.3.2/plyr.js"></script>
+<link rel="stylesheet" href="https://cdn.plyr.io/1.3.3/plyr.css">
+<script src="https://cdn.plyr.io/1.3.3/plyr.js"></script>
 ```

-You can also access the `sprite.svg` file at `https://cdn.plyr.io/1.3.2/sprite.svg`.
+You can also access the `sprite.svg` file at `https://cdn.plyr.io/1.3.3/sprite.svg`.

 ### 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. 
--- a/src/js/plyr.js
+++ b/src/js/plyr.js
@ -1,6 +1,6 @@
 // ==========================================================================
 // Plyr
-// plyr.js v1.3.1
+// plyr.js v1.3.3
 // https://github.com/selz/plyr
 // License: The MIT License (MIT)
 // ==========================================================================
@ -928,6 +928,7 @@
                    rel: 0,
                    showinfo: 0,
                    iv_load_policy: 3,
+                    cc_load_policy: (config.captions.defaultActive ? 1 : 0),
                    cc_lang_pref: "en",
                    wmode: "transparent",
                    modestbranding: 1,
@ -1028,7 +1029,7 @@
        function _setupCaptions() {
            if(player.type === "video") {
                // Inject the container
-                player.videoContainer.insertAdjacentHTML("afterbegin", "<div class='" + config.selectors.captions.replace(".", "") + "' aria-live='assertive' aria-relevant='additions text'><span></span></div>");
+                player.videoContainer.insertAdjacentHTML("afterbegin", "<div class='" + config.selectors.captions.replace(".", "") + "'><span></span></div>");

                // Cache selector
                player.captionsContainer = _getElement(config.selectors.captions).querySelector("span");
--- a/src/sass/plyr.scss
+++ b/src/sass/plyr.scss
@ -19,9 +19,9 @@ $font-size-small:           14px !default;
 $font-size-base:            16px !default;

 // Captions
-$font-size-captions-base:   ceil(@font-size-base * 1.25) !default;
-$font-size-captions-medium: ceil(@font-size-base * 1.5) !default;
-$font-size-captions-large:  (@font-size-base * 2) !default;
+$font-size-captions-base:   ceil($font-size-base * 1.25) !default;
+$font-size-captions-medium: ceil($font-size-base * 1.5) !default;
+$font-size-captions-large:  ($font-size-base * 2) !default;

 // Controls
 $control-spacing:           10px !default;
@ -211,7 +211,7 @@ $bp-captions-large:         768px !default; // When captions jump to the larger
            display: none;
        }

-        @media (min-width: @bp-captions-large) {
+        @media (min-width: $bp-captions-large) {
            font-size: $font-size-captions-medium;
        }
    }
Author	SHA1	Message	Date
Sam Potts	45d4091c20	Merge branch 'master' of github.com:selz/plyr	2015-08-11 22:38:45 +10:00
Sam Potts	60c53a5894	Removed captions being read by screen readers	2015-08-11 22:38:06 +10:00
Sam Potts	032c823d3a	Merge pull request #108 from ChristianPV/patch-2 Update plyr.scss - Fix new scss!	2015-08-10 23:31:31 +10:00
khrizpv	66f724a3be	Update plyr.scss - Fix new scss! This is a fix the newly added scss.	2015-08-10 10:03:21 -03:00
Sam Potts	bf0c81b484	Docs	2015-08-09 09:29:33 +10:00
Sam Potts	aa72a17f44	Version bump	2015-08-08 21:02:55 +10:00