[CLEANUP] even more ident/tabs

ableplayer · joedolson · Jun 15, 2025 · Nov 11, 2022 · Nov 11, 2022 · Nov 20, 2022
commit d442e839c92a7ff3e8ad4ea9d8f2bf93d45dae50
diff --git a/scripts/browser.js b/scripts/browser.js
@@ -33,29 +33,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.
+		// 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:
-			// https://stackoverflow.com/questions/72861253/how-do-i-detect-if-a-browser-does-not-support-changing-html-audio-volume
+		// 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;
+		// 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;
-			}
+		if (this.isIOS()) {
+			return false;
+		}
 
-			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;
-			}
+		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;
+		}
 	};
 
 	AblePlayer.prototype.nativeFullscreenSupported = function () {

diff --git a/scripts/buildplayer.js b/scripts/buildplayer.js
@@ -4,16 +4,16 @@
 
 		// create and inject surrounding HTML structure
 		// If IOS:
-		//	If video:
-		//	 IOS does not support any of the player's functionality
-		//	 - everything plays in its own player
-		//	 Therefore, AblePlayer is not loaded & all functionality is disabled
-		//	 (this all determined. If this is IOS && video, this function is never called)
-		//	If audio:
-		//	 HTML cannot be injected as a *parent* of the <audio> element
-		//	 It is therefore injected *after* the <audio> element
-		//	 This is only a problem in IOS 6 and earlier,
-		//	 & is a known bug, fixed in IOS 7
+		//  If video:
+		//   IOS does not support any of the player's functionality
+		//   - everything plays in its own player
+		//   Therefore, AblePlayer is not loaded & all functionality is disabled
+		//   (this all determined. If this is IOS && video, this function is never called)
+		//  If audio:
+		//   HTML cannot be injected as a *parent* of the <audio> element
+		//   It is therefore injected *after* the <audio> element
+		//   This is only a problem in IOS 6 and earlier,
+		//   & is a known bug, fixed in IOS 7
 
 		var thisObj, captionsContainer, prefsGroups, i;
 		thisObj = this;

diff --git a/scripts/control.js b/scripts/control.js
@@ -198,11 +198,11 @@
 	AblePlayer.prototype.getPlayerState = function () {
 
 		// Returns one of the following states:
-		//	'stopped' - Not yet played for the first time, or otherwise reset to unplayed.
-		//	'ended' - Finished playing.
-		//	'paused' - Not playing, but not stopped or ended.
-		//	'buffering' - Momentarily paused to load, but will resume once data is loaded.
-		//	'playing' - Currently playing.
+		// - 'stopped' - Not yet played for the first time, or otherwise reset to unplayed.
+		// - 'ended' - Finished playing.
+		// - 'paused' - Not playing, but not stopped or ended.
+		// - 'buffering' - Momentarily paused to load, but will resume once data is loaded.
+		// - 'playing' - Currently playing.
 
 		// Commented out the following in 3.2.1 - not sure of its intended purpose
 		// It can be useful to know player state even when swapping src
@@ -551,7 +551,7 @@
 					// Only update the aria live region if we have an update pending
 					// (from a seek button control) or if the seekBar has focus.
 					// We use document.activeElement instead of $(':focus') due to a strange bug:
-					// 	When the seekHead element is focused, .is(':focus') is failing and $(':focus') is returning an undefined element.
+					// When the seekHead element is focused, .is(':focus') is failing and $(':focus') is returning an undefined element.
 					updateLive = this.liveUpdatePending || this.seekBar.seekHead.is($(document.activeElement));
 					this.liveUpdatePending = false;
 					if (this.useChapterTimes) {

diff --git a/scripts/description.js b/scripts/description.js
@@ -674,8 +674,8 @@
 		//  and typically requires a computer reboot to make right again.
 		//  This has been observed frequently in macOS Big Sur, but also in Windows 10
 		//  To ignore user's voice preferences and always use the first voice, set the following var to true
-		//	This is for testing only; not recommended for production
-		// 	unless the voice select field is also removed from the Prefs dialog
+		//  This is for testing only; not recommended for production
+		//  unless the voice select field is also removed from the Prefs dialog
 		var useFirstVoice = false;
 
 		if (!this.speechEnabled) {

diff --git a/scripts/initialize.js b/scripts/initialize.js
@@ -281,7 +281,7 @@
 	};
 
 	// Initialize player based on data on page.
-	// This sets some variables, but does not modify anything.	Safe to call multiple times.
+	// This sets some variables, but does not modify anything. Safe to call multiple times.
 	// Can call again after updating this.media so long as new media element has the same ID.
 	AblePlayer.prototype.reinitialize = function () {
 

diff --git a/scripts/misc.js b/scripts/misc.js
@@ -50,7 +50,7 @@
 
 		// Takes seconds and converts to string of form hh:mm:ss
 		// If showFullTime is true, shows 00 for hours if time is less than an hour
-		//	 and show milliseconds	(e.g., 00:00:04.123 as in Video Track Sorter)
+		// and show milliseconds (e.g., 00:00:04.123 as in Video Track Sorter)
 		// Otherwise, omits empty hours and milliseconds (e.g., 00:04 as in timer on controller)
 
 		var dHours, dMinutes, dSeconds,
@@ -155,7 +155,7 @@
 		var attr = object.attr(attribute);
 
 		// For some browsers, `attr` is undefined; for others,
-		// `attr` is false.	 Check for both.
+		// `attr` is false. Check for both.
 		if (typeof attr !== typeof undefined && attr !== false) {
 			return true;
 		}

diff --git a/scripts/slider.js b/scripts/slider.js
@@ -2,9 +2,9 @@
 
 
 	// Events:
-	//	 startTracking(event, position)
-	//	 tracking(event, position)
-	//	 stopTracking(event, position)
+	// - startTracking(event, position)
+	// - tracking(event, position)
+	// - stopTracking(event, position)
 
 	window. AccessibleSlider = function(mediaType, div, orientation, length, min, max, bigInterval, label, className, trackingMedia, initialState) {
 
@@ -322,7 +322,7 @@
 		this.updateAriaValues(position, updateLive);
 	}
 
-	// TODO: Native HTML5 can have several buffered segments, and this actually happens quite often.	Change this to display them all.
+	// TODO: Native HTML5 can have several buffered segments, and this actually happens quite often. Change this to display them all.
 	AccessibleSlider.prototype.setBuffered = function (ratio) {
 		if (!isNaN(ratio)) {
 			this.buffered = ratio;