Merge pull request #184 from blacksigkill/fix/linting

fix all linting (js/html/css)

eslint.config.js

@@ -4,7 +4,7 @@ import prettierConfig from 'eslint-config-prettier';
 
 export default [
     {
-        ignores: ['**/dist/**', '**/node_modules/**', '**/legacy/**', '**/bin/**', '**/www/**'],
+        ignores: ['**/dist/**', '**/node_modules/**', '**/legacy/**', '**/bin/**', '**/www/**', '**/public/lib/**', '**/public/neutralino.js'],
     },
     js.configs.recommended,
     prettierConfig,

js/app.js

@@ -1,5 +1,4 @@
 //js/app.js
-import { LosslessAPI } from './api.js';
 import { MusicAPI } from './music-api.js';
 import {
     apiSettings,

js/music-api.js

@@ -76,12 +76,12 @@ export class MusicAPI {
         return api.getArtist(cleanId);
     }
 
-    async getPlaylist(id, provider = null) {
+    async getPlaylist(id, _provider = null) {
         // Playlists are always Tidal for now
         return this.tidalAPI.getPlaylist(id);
     }
 
-    async getMix(id, provider = null) {
+    async getMix(id, _provider = null) {
         // Mixes are always Tidal for now
         return this.tidalAPI.getMix(id);
     }
@@ -173,10 +173,4 @@ export class MusicAPI {
     get settings() {
         return this._settings;
     }
-
-    // Extract stream URL from manifest (Tidal only)
-    extractStreamUrlFromManifest(manifest) {
-        // This is only available for Tidal
-        return this.tidalAPI.extractStreamUrlFromManifest(manifest);
-    }
 }

js/qobuz-api.js

@@ -115,7 +115,7 @@ export class QobuzAPI {
     }
 
     // Get track details
-    async getTrack(id) {
+    async getTrack(_id) {
         // Qobuz doesn't have a direct track endpoint
         // Track metadata comes from search/album endpoints
         // For playback, use getStreamUrl directly

js/settings.js

@@ -1837,7 +1837,7 @@ export function initializeSettings(scrobbler, player, api, ui) {
                         for (const storeName of stores) {
                             try {
                                 await db.performTransaction(storeName, 'readwrite', (store) => store.clear());
-                            } catch (e) {
+                            } catch {
                                 // Store might not exist, continue
                             }
                         }

js/ui.js

@@ -2339,7 +2339,7 @@ export class UIRenderer {
         }
     }
 
-    async renderPlaylistPage(playlistId, source = null, provider = null) {
+    async renderPlaylistPage(playlistId, source = null, _provider = null) {
         this.showPage('playlist');
 
         // Reset search input for new playlist

scripts/dev-runner.js

@@ -1,6 +1,5 @@
 import fs from 'fs';
 import { spawn } from 'child_process';
-import path from 'path';
 
 const CONFIG_FILE = 'neutralino.config.json';
 const DEV_CONFIG_FILE = 'neutralino.config.dev.json';

styles.css

@@ -5244,6 +5244,7 @@ body:has(#fullscreen-cover-overlay:not([style*='display: none'])) .now-playing-b
     max-width: 400px;
     overflow: hidden;
     text-overflow: ellipsis;
+    will-change: transform, opacity;
 }
 
 /* Genius i love genius brah!! */
@@ -5969,27 +5970,6 @@ textarea:focus {
     font-size: 0.95rem;
 }
 
-/* Custom Tooltip */
-#custom-tooltip {
-    position: fixed;
-    background: var(--card);
-    color: var(--card-foreground);
-    padding: 0.5rem 0.75rem;
-    border-radius: var(--radius);
-    border: 1px solid var(--border);
-    box-shadow: var(--shadow-lg);
-    font-size: 0.85rem;
-    pointer-events: none;
-    z-index: 10000;
-    opacity: 0;
-    transition: opacity 0.1s ease;
-    white-space: nowrap;
-    max-width: 400px;
-    overflow: hidden;
-    text-overflow: ellipsis;
-    will-change: transform, opacity;
-}
-
 #custom-tooltip.visible {
     opacity: 1;
 }