Move the "-" in remaining-time-display.js so it is not announced by screen readers. Fixes #5168

src/js/control-bar/time-controls/remaining-time-display.js

@@ -48,7 +48,7 @@ class RemainingTimeDisplay extends TimeDisplay {
    */
   formatTime_(time) {
     // TODO: The "-" should be decorative, and not announced by a screen reader
-    return '-' + super.formatTime_(time);
+    return super.formatTime_(time);
   }
 
   /**

src/js/control-bar/time-controls/time-display.js

@@ -51,6 +51,10 @@ class TimeDisplay extends Component {
 
     this.updateTextNode_();
     el.appendChild(this.contentEl_);
+    if (className === 'vjs-remaining-time') {
+      el.insertBefore(Dom.createEl('span', {}, {'aria-hidden': true}, '-'), this.contentEl_);
+    }
+
     return el;
   }
 

src/js/utils/fn.js

@@ -55,7 +55,7 @@ export const bind = function(context, fn, uid) {
  * @param    {Function} fn
  *           The function to be throttled.
  *
- * @param    {Number}   wait
+ * @param    {number}   wait
  *           The number of milliseconds by which to throttle.
  *
  * @return   {Function}

src/js/utils/url.js

@@ -126,7 +126,7 @@ export const getAbsoluteURL = function(url) {
  * @param    {string} path
  *           The fileName path like '/path/to/file.mp4'
  *
- * @returns  {string}
+ * @return  {string}
  *           The extension in lower case or an empty string if no
  *           extension could be found.
  */