linting cleanup

index.js

@@ -1,5 +1,4 @@
 // express
-// eslint-disable-next-line import/no-extraneous-dependencies
 const express = require('express');
 
 const app = express();

server/scripts/index.mjs

@@ -254,14 +254,14 @@ const loadData = (_latLon, haveDataCallback) => {
 
 const swipeCallBack = (direction) => {
 	switch (direction) {
-	case 'left':
+		case 'left':
-		btnNavigateNextClick();
+			btnNavigateNextClick();
-		break;
+			break;
 
-	case 'right':
+		case 'right':
-	default:
+		default:
-		btnNavigatePreviousClick();
+			btnNavigatePreviousClick();
-		break;
+			break;
 	}
 };
 
@@ -312,41 +312,41 @@ const documentKeydown = (e) => {
 
 	if (document.fullscreenElement || document.activeElement === document.body) {
 		switch (key) {
-		case ' ': // Space
+			case ' ': // Space
-			// don't scroll
+				// don't scroll
-			e.preventDefault();
+				e.preventDefault();
-			btnNavigatePlayClick();
+				btnNavigatePlayClick();
-			return false;
+				return false;
 
-		case 'ArrowRight':
+			case 'ArrowRight':
-		case 'PageDown':
+			case 'PageDown':
-			// don't scroll
+				// don't scroll
-			e.preventDefault();
+				e.preventDefault();
-			btnNavigateNextClick();
+				btnNavigateNextClick();
-			return false;
+				return false;
 
-		case 'ArrowLeft':
+			case 'ArrowLeft':
-		case 'PageUp':
+			case 'PageUp':
-			// don't scroll
+				// don't scroll
-			e.preventDefault();
+				e.preventDefault();
-			btnNavigatePreviousClick();
+				btnNavigatePreviousClick();
-			return false;
+				return false;
 
-		case 'ArrowUp': // Home
+			case 'ArrowUp': // Home
-			e.preventDefault();
+				e.preventDefault();
-			btnNavigateMenuClick();
+				btnNavigateMenuClick();
-			return false;
+				return false;
 
-		case '0': // "O" Restart
+			case '0': // "O" Restart
-			btnNavigateRefreshClick();
+				btnNavigateRefreshClick();
-			return false;
+				return false;
 
-		case 'F':
+			case 'F':
-		case 'f':
+			case 'f':
-			btnFullScreenClick();
+				btnFullScreenClick();
-			return false;
+				return false;
 
-		default:
+			default:
 		}
 	}
 	return false;

server/scripts/modules/hazards.mjs

@@ -3,7 +3,7 @@
 import STATUS from './status.mjs';
 import { json } from './utils/fetch.mjs';
 import WeatherDisplay from './weatherdisplay.mjs';
-import { registerDisplay, msg } from './navigation.mjs';
+import { registerDisplay } from './navigation.mjs';
 
 const hazardLevels = {
 	Extreme: 10,
@@ -14,7 +14,7 @@ const hazardModifiers = {
 	'Hurricane Warning': 2,
 	'Tornado Warning': 3,
 	'Severe Thunderstorm Warning': 1,
-}
+};
 
 class Hazards extends WeatherDisplay {
 	constructor(navId, elemId, defaultActive) {
@@ -160,7 +160,7 @@ const calcSeverity = (severity, event) => {
 	const baseSeverity = hazardLevels[severity] ?? 0;
 	const modifiedSeverity = hazardModifiers[event] ?? 0;
 	return baseSeverity + modifiedSeverity;
-}
+};
 
 // register display
 registerDisplay(new Hazards(0, 'hazards', true));

server/scripts/modules/navigation.mjs

@@ -238,30 +238,30 @@ const setPlaying = (newValue) => {
 // handle all navigation buttons
 const handleNavButton = (button) => {
 	switch (button) {
-	case 'play':
+		case 'play':
-		setPlaying(true);
+			setPlaying(true);
-		break;
+			break;
-	case 'playToggle':
+		case 'playToggle':
-		setPlaying(!playing);
+			setPlaying(!playing);
-		break;
+			break;
-	case 'stop':
+		case 'stop':
-		setPlaying(false);
+			setPlaying(false);
-		break;
+			break;
-	case 'next':
+		case 'next':
-		setPlaying(false);
+			setPlaying(false);
-		navTo(msg.command.nextFrame);
+			navTo(msg.command.nextFrame);
-		break;
+			break;
-	case 'previous':
+		case 'previous':
-		setPlaying(false);
+			setPlaying(false);
-		navTo(msg.command.previousFrame);
+			navTo(msg.command.previousFrame);
-		break;
+			break;
-	case 'menu':
+		case 'menu':
-		setPlaying(false);
+			setPlaying(false);
-		progress.showCanvas();
+			progress.showCanvas();
-		hideAllCanvases();
+			hideAllCanvases();
-		break;
+			break;
-	default:
+		default:
-		console.error(`Unknown navButton ${button}`);
+			console.error(`Unknown navButton ${button}`);
 	}
 };
 

server/scripts/modules/status.mjs

@@ -9,20 +9,20 @@ const STATUS = {
 
 const calcStatusClass = (statusCode) => {
 	switch (statusCode) {
-	case STATUS.loading:
+		case STATUS.loading:
-		return 'loading';
+			return 'loading';
-	case STATUS.loaded:
+		case STATUS.loaded:
-		return 'press-here';
+			return 'press-here';
-	case STATUS.failed:
+		case STATUS.failed:
-		return 'failed';
+			return 'failed';
-	case STATUS.noData:
+		case STATUS.noData:
-		return 'no-data';
+			return 'no-data';
-	case STATUS.disabled:
+		case STATUS.disabled:
-		return 'disabled';
+			return 'disabled';
-	case STATUS.retrying:
+		case STATUS.retrying:
-		return 'retrying';
+			return 'retrying';
-	default:
+		default:
-		return '';
+			return '';
 	}
 };
 

server/scripts/modules/utils/fetch.mjs

@@ -39,14 +39,14 @@ const fetchAsync = async (_url, responseType, _params = {}) => {
 	if (!response.ok) throw new Error(`Fetch error ${response.status} ${response.statusText} while fetching ${response.url}`);
 	// return the requested response
 	switch (responseType) {
-	case 'json':
+		case 'json':
-		return response.json();
+			return response.json();
-	case 'text':
+		case 'text':
-		return response.text();
+			return response.text();
-	case 'blob':
+		case 'blob':
-		return response.blob();
+			return response.blob();
-	default:
+		default:
-		return response;
+			return response;
 	}
 };
 
@@ -84,11 +84,11 @@ const delay = (time, func, ...args) => new Promise((resolve) => {
 
 const retryDelay = (retryNumber) => {
 	switch (retryNumber) {
-	case 1: return 1000;
+		case 1: return 1000;
-	case 2: return 2000;
+		case 2: return 2000;
-	case 3: return 5000;
+		case 3: return 5000;
-	case 4: return 10_000;
+		case 4: return 10_000;
-	default: return 30_000;
+		default: return 30_000;
 	}
 };
 

server/scripts/modules/utils/setting.mjs

@@ -33,12 +33,12 @@ class Setting {
 
 		// call the change function on startup
 		switch (type) {
-		case 'select':
+			case 'select':
-			this.selectChange({ target: { value: this.myValue } });
+				this.selectChange({ target: { value: this.myValue } });
-			break;
+				break;
-		case 'checkbox':
+			case 'checkbox':
-		default:
+			default:
-			this.checkboxChange({ target: { checked: this.myValue } });
+				this.checkboxChange({ target: { checked: this.myValue } });
 		}
 	}
 
@@ -129,12 +129,12 @@ class Setting {
 				const storedValue = JSON.parse(allSettings)?.[this.shortName];
 				if (storedValue !== undefined) {
 					switch (this.type) {
-					case 'boolean':
+						case 'boolean':
-						return storedValue;
+							return storedValue;
-					case 'select':
+						case 'select':
-						return storedValue;
+							return storedValue;
-					default:
+						default:
-						return null;
+							return null;
 					}
 				}
 			}
@@ -152,12 +152,12 @@ class Setting {
 		// update the state
 		this.myValue = newValue;
 		switch (this.type) {
-		case 'select':
+			case 'select':
-			this.selectHighlight(newValue);
+				this.selectHighlight(newValue);
-			break;
+				break;
-		case 'checkbox':
+			case 'checkbox':
-		default:
+			default:
-			this.element.checked = newValue;
+				this.element.checked = newValue;
 		}
 		this.storeToLocalStorage(this.myValue);
 
@@ -174,11 +174,11 @@ class Setting {
 
 	generate() {
 		switch (this.type) {
-		case 'select':
+			case 'select':
-			return this.generateSelect();
+				return this.generateSelect();
-		case 'checkbox':
+			case 'checkbox':
-		default:
+			default:
-			return this.generateCheckbox();
+				return this.generateCheckbox();
 		}
 	}
 }