[CLEANUP] scripts/

ableplayer · joedolson · Jun 15, 2025 · Nov 11, 2022 · Nov 11, 2022 · Nov 20, 2022
commit a9af6f6c29b113c189281fc12332d34221705fec
diff --git a/scripts/ableplayer-base.js b/scripts/ableplayer-base.js
@@ -40,7 +40,6 @@ var AblePlayerInstances = [];
 
 (function ($) {
 	$(function () {
-
 		$('video, audio').each(function (index, element) {
 			if ($(element).data('able-player') !== undefined) {
 				AblePlayerInstances.push(new AblePlayer($(this),$(element)));
@@ -88,14 +87,14 @@ var AblePlayerInstances = [];
 
 		// autoplay (Boolean; if present always resolves to true, regardless of value)
 		if ($(media).attr('autoplay') !== undefined) {
-			this.autoplay = true; // this value remains constant 
+			this.autoplay = true; // this value remains constant
 			this.okToPlay = true; // this value can change dynamically
 		}
 		else {
 			this.autoplay = false;
 			this.okToPlay = false;
 		}
-		
+
 		// loop (Boolean; if present always resolves to true, regardless of value)
 		if ($(media).attr('loop') !== undefined) {
 			this.loop = true;
@@ -120,13 +119,13 @@ var AblePlayerInstances = [];
 			this.hasPoster = false;
 		}
 
-		// get height and width attributes, if present 
-		// and add them to variables 
-		// Not currently used, but might be useful for resizing player  
-		if ($(media).attr('width')) { 
-			this.width = $(media).attr('width'); 
+		// get height and width attributes, if present
+		// and add them to variables
+		// Not currently used, but might be useful for resizing player
+		if ($(media).attr('width')) {
+			this.width = $(media).attr('width');
 		}
-		if ($(media).attr('height')) { 
+		if ($(media).attr('height')) {
 			this.height = $(media).attr('height');
 		}
 
@@ -177,13 +176,13 @@ var AblePlayerInstances = [];
 		else {
 			this.useChaptersButton = true;
 		}
-		
-		// Control whether text descriptions are read aloud 
+
+		// Control whether text descriptions are read aloud
 		// set to "false" if the sole purpose of the WebVTT descriptions file
 		// is to integrate text description into the transcript
-		// set to "true" to write description text to a div 
-		// This variable does *not* control the method by which description is read. 
-		// For that, see below (this.descMethod) 
+		// set to "true" to write description text to a div
+		// This variable does *not* control the method by which description is read.
+		// For that, see below (this.descMethod)
 		if ($(media).data('descriptions-audible') !== undefined && $(media).data('descriptions-audible') === false) {
 			this.readDescriptionsAloud = false;
 		}
@@ -195,24 +194,24 @@ var AblePlayerInstances = [];
 			this.readDescriptionsAloud = true;
 		}
 
-		// Method by which text descriptions are read  
+		// Method by which text descriptions are read
 		// valid values of data-desc-reader are:
-		// 'brower' (default) - text-based audio description is handled by the browser, if supported  
-		// 'screenreader' - text-based audio description is always handled by screen readers 
-		// The latter may be preferable by owners of websites in languages that are not well supported 
-		// by the Web Speech API  
+		// 'brower' (default) - text-based audio description is handled by the browser, if supported
+		// 'screenreader' - text-based audio description is always handled by screen readers
+		// The latter may be preferable by owners of websites in languages that are not well supported
+		// by the Web Speech API
 		if ($(media).data('desc-reader') == 'screenreader') {
 			this.descReader = 'screenreader';
 		}
 		else {
 			this.descReader = 'browser';
 		}
 
-		// Default state of captions and descriptions 
-		// This setting is overridden by user preferences, if they exist 
-		// values for data-state-captions and data-state-descriptions are 'on' or 'off' 
+		// Default state of captions and descriptions
+		// This setting is overridden by user preferences, if they exist
+		// values for data-state-captions and data-state-descriptions are 'on' or 'off'
 		if ($(media).data('state-captions') == 'off') {
-			this.defaultStateCaptions = 0; // off 
+			this.defaultStateCaptions = 0; // off
 		}
 		else {
 			this.defaultStateCaptions = 1; // on by default
@@ -224,13 +223,13 @@ var AblePlayerInstances = [];
 			this.defaultStateDescriptions = 0; // off by default
 		}
 
-		// Default setting for prefDescPause  
-		// Extended description (i.e., pausing during description) is on by default 
-		// but this settings give website owners control over that 
-		// since they know the nature of their videos, and whether pausing is necessary 
-		// This setting is overridden by user preferences, if they exist 
+		// Default setting for prefDescPause
+		// Extended description (i.e., pausing during description) is on by default
+		// but this settings give website owners control over that
+		// since they know the nature of their videos, and whether pausing is necessary
+		// This setting is overridden by user preferences, if they exist
 		if ($(media).data('desc-pause-default') == 'off') {
-			this.defaultDescPause = 0; // off 
+			this.defaultDescPause = 0; // off
 		}
 		else {
 			this.defaultDescPause = 1; // on by default
@@ -276,7 +275,7 @@ var AblePlayerInstances = [];
 			if (this.transcriptSrcHasRequiredParts()) {
 				this.transcriptType = 'manual';
 			}
-			else { 
+			else {
 				console.log('ERROR: Able Player transcript is missing required parts');
 			}
 		}
@@ -390,22 +389,22 @@ var AblePlayerInstances = [];
 			this.skin = 'legacy';
 		}
 
-		// Size 
-		// width of Able Player is determined using the following order of precedence: 
-		// 1. data-width attribute 
+		// Size
+		// width of Able Player is determined using the following order of precedence:
+		// 1. data-width attribute
 		// 2. width attribute (for video or audio, although it is not valid HTML for audio)
 		// 3. Intrinsic size from video (video only, determined later)
 		if ($(media).data('width') !== undefined) {
 			this.playerWidth = parseInt($(media).data('width'));
 		}
 		else if ($(media)[0].getAttribute('width')) {
-			// NOTE: jQuery attr() returns null for all invalid HTML attributes 
+			// NOTE: jQuery attr() returns null for all invalid HTML attributes
 			// (e.g., width on <audio>)
-			// but it can be acessed via JavaScript getAttribute() 
+			// but it can be acessed via JavaScript getAttribute()
 			this.playerWidth = parseInt($(media)[0].getAttribute('width'));
 		}
-		else { 
-			this.playerWidth = null; 
+		else {
+			this.playerWidth = null;
 		}
 
 		// Icon type
@@ -429,9 +428,9 @@ var AblePlayerInstances = [];
 		else {
 			this.allowFullscreen = true;
 		}
-		// Define other variables that are used in fullscreen program flow 
-		this.clickedFullscreenButton = false; 
-		this.restoringAfterFullscreen = false;			
+		// Define other variables that are used in fullscreen program flow
+		this.clickedFullscreenButton = false;
+		this.restoringAfterFullscreen = false;
 
 		// Seek interval
 		// Number of seconds to seek forward or back with Rewind & Forward buttons
@@ -470,15 +469,15 @@ var AblePlayerInstances = [];
 		// Fallback
 		// The data-test-fallback attribute can be used to test the fallback solution in any browser
 		if ($(media).data('test-fallback') !== undefined && $(media).data('test-fallback') !== false) {
-			if ($(media).data('test-fallback') == '2') { 
-				this.testFallback = 2; // emulate browser that doesn't support HTML5 media 
+			if ($(media).data('test-fallback') == '2') {
+				this.testFallback = 2; // emulate browser that doesn't support HTML5 media
 			}
-			else { 
-				this.testFallback = 1; // emulate failure to load Able Player 
+			else {
+				this.testFallback = 1; // emulate failure to load Able Player
 			}
 		}
-		else { 
-			this.testFallback = false; 
+		else {
+			this.testFallback = false;
 		}
 
 		// Language
@@ -605,8 +604,8 @@ var AblePlayerInstances = [];
 				}
 			}
 		).
-		fail(function() { 
-			thisObj.provideFallback(); 
+		fail(function() {
+			thisObj.provideFallback();
 		});
 	};
 
@@ -630,9 +629,9 @@ var AblePlayerInstances = [];
 						// for playlists, recreatePlayer() is called from within cuePlaylistItem()
 					}
 					else {
-						thisObj.recreatePlayer().then(function() { 
+						thisObj.recreatePlayer().then(function() {
 							thisObj.initializing = false;
-							thisObj.playerCreated = true; // remains true until browser is refreshed		
+							thisObj.playerCreated = true; // remains true until browser is refreshed
 						});
 					}
 				});

diff --git a/scripts/browser.js b/scripts/browser.js
@@ -32,29 +32,29 @@
 
 	AblePlayer.prototype.browserSupportsVolume = function() {
 
-		// To test whether the browser supports changing the volume, 
-			// create a new audio element and try setting the volume to something other than 1. 
-			// Then, retrieve the current setting to see if it preserved it. 
+		// To test whether the browser supports changing the volume,
+			// create a new audio element and try setting the volume to something other than 1.
+			// Then, retrieve the current setting to see if it preserved it.
 
-			// Unfortunately, this doesn't work in iOS. In 2022, our tests yield the same results as reported here:  
+			// Unfortunately, this doesn't work in iOS. In 2022, our tests yield the same results as reported here:
 			// https://stackoverflow.com/questions/72861253/how-do-i-detect-if-a-browser-does-not-support-changing-html-audio-volume
 
-			// So, unfortunately we have to resort to sniffing for iOS  
-			// before testing for support in other browsers 
-			var audio, testVolume; 
- 
-			if (this.isIOS()) { 
-				return false; 
+			// So, unfortunately we have to resort to sniffing for iOS
+			// before testing for support in other browsers
+			var audio, testVolume;
+
+			if (this.isIOS()) {
+				return false;
 			}
 
-			testVolume = 0.9;  // any value between 0.1 and 0.9 
+			testVolume = 0.9;  // any value between 0.1 and 0.9
 			audio = new Audio();
       audio.volume = testVolume;
-			if (audio.volume === testVolume) { 
-				return true; 
-			} 
-			else { 
-				return false; 
+			if (audio.volume === testVolume) {
+				return true;
+			}
+			else {
+				return false;
 			}
 	};