style: fix all linting warnings (unused vars/imports)
This commit is contained in:
parent
48da88c12d
commit
5dcc688577
5 changed files with 9 additions and 18 deletions
|
|
@ -8,7 +8,6 @@ import {
|
|||
trackDataStore,
|
||||
formatTime,
|
||||
SVG_BIN,
|
||||
escapeHtml,
|
||||
} from './utils.js';
|
||||
import { lastFMStorage, waveformSettings } from './storage.js';
|
||||
import { showNotification, downloadTrackWithMetadata } from './downloads.js';
|
||||
|
|
|
|||
|
|
@ -1,6 +1,4 @@
|
|||
//js/lastfm.js
|
||||
import { delay, getTrackArtists } from './utils.js';
|
||||
|
||||
export class LastFMScrobbler {
|
||||
constructor() {
|
||||
this.API_KEY = '0ecf01914957b40c17030db822845a76';
|
||||
|
|
@ -25,8 +23,8 @@ export class LastFMScrobbler {
|
|||
this.sessionKey = data.key;
|
||||
this.username = data.name;
|
||||
}
|
||||
} catch (e) {
|
||||
console.error('Failed to load Last.fm session:', e);
|
||||
} catch {
|
||||
console.error('Failed to load Last.fm session');
|
||||
}
|
||||
}
|
||||
|
||||
|
|
@ -66,7 +64,7 @@ export class LastFMScrobbler {
|
|||
try {
|
||||
const { default: md5 } = await import('https://cdn.jsdelivr.net/npm/md5@2.3.0/+esm');
|
||||
return md5(signatureString);
|
||||
} catch (e) {
|
||||
} catch {
|
||||
console.error('MD5 library not available');
|
||||
throw new Error('MD5 library required for Last.fm');
|
||||
}
|
||||
|
|
|
|||
10
js/lyrics.js
10
js/lyrics.js
|
|
@ -1,11 +1,7 @@
|
|||
//js/lyrics.js
|
||||
import { getTrackTitle, getTrackArtists, buildTrackFilename, SVG_DOWNLOAD, SVG_CLOSE } from './utils.js';
|
||||
import { getTrackTitle, getTrackArtists, buildTrackFilename, SVG_CLOSE } from './utils.js';
|
||||
import { sidePanelManager } from './side-panel.js';
|
||||
|
||||
// Dictionary path for kuromoji
|
||||
// Using CDN - the kuroshiro-analyzer loaded from unpkg will use this as base for fetching dict files
|
||||
const KUROMOJI_DICT_PATH = 'https://cdn.jsdelivr.net/npm/kuromoji@0.1.2/dict/';
|
||||
|
||||
export class LyricsManager {
|
||||
constructor(api) {
|
||||
this.api = api;
|
||||
|
|
@ -190,7 +186,7 @@ export class LyricsManager {
|
|||
getRomajiMode() {
|
||||
try {
|
||||
return localStorage.getItem('lyricsRomajiMode') === 'true';
|
||||
} catch (e) {
|
||||
} catch {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
|
@ -543,7 +539,7 @@ export function openLyricsPanel(track, audioPlayer, lyricsManager, forceOpen = f
|
|||
romajiBtn.addEventListener('click', async () => {
|
||||
const amLyrics = sidePanelManager.panel.querySelector('am-lyrics');
|
||||
if (amLyrics) {
|
||||
const newMode = await manager.toggleRomajiMode(amLyrics);
|
||||
await manager.toggleRomajiMode(amLyrics);
|
||||
updateRomajiBtn();
|
||||
}
|
||||
});
|
||||
|
|
|
|||
|
|
@ -587,7 +587,7 @@ export function initializeSettings(scrobbler, player, api, ui) {
|
|||
try {
|
||||
const fbConfig = JSON.parse(fbConfigStr);
|
||||
saveFirebaseConfig(fbConfig);
|
||||
} catch (e) {
|
||||
} catch {
|
||||
alert('Invalid JSON for Firebase Config');
|
||||
return;
|
||||
}
|
||||
|
|
|
|||
6
js/ui.js
6
js/ui.js
|
|
@ -17,7 +17,7 @@ import {
|
|||
escapeHtml,
|
||||
} from './utils.js';
|
||||
import { openLyricsPanel } from './lyrics.js';
|
||||
import { recentActivityManager, backgroundSettings, trackListSettings, cardSettings } from './storage.js';
|
||||
import { recentActivityManager, backgroundSettings, cardSettings } from './storage.js';
|
||||
import { db } from './db.js';
|
||||
import { getVibrantColorFromImage } from './vibrant-color.js';
|
||||
import { syncManager } from './accounts/pocketbase.js';
|
||||
|
|
@ -70,7 +70,7 @@ export class UIRenderer {
|
|||
this.vibrantColorCache.set(url, null);
|
||||
this.resetVibrantColor();
|
||||
}
|
||||
} catch (e) {
|
||||
} catch {
|
||||
this.vibrantColorCache.set(url, null);
|
||||
this.resetVibrantColor();
|
||||
}
|
||||
|
|
@ -163,7 +163,6 @@ export class UIRenderer {
|
|||
}
|
||||
|
||||
createTrackItemHTML(track, index, showCover = false, hasMultipleDiscs = false) {
|
||||
const playIconSmall = SVG_PLAY;
|
||||
const trackImageHTML = showCover
|
||||
? `<img src="${this.api.getCoverUrl(track.album?.cover)}" alt="Track Cover" class="track-item-cover" loading="lazy">`
|
||||
: '';
|
||||
|
|
@ -630,7 +629,6 @@ export class UIRenderer {
|
|||
const title = document.getElementById('fullscreen-track-title');
|
||||
const artist = document.getElementById('fullscreen-track-artist');
|
||||
const nextTrackEl = document.getElementById('fullscreen-next-track');
|
||||
const lyricsContainer = document.getElementById('fullscreen-lyrics-container');
|
||||
const lyricsToggleBtn = document.getElementById('toggle-fullscreen-lyrics-btn');
|
||||
|
||||
const coverUrl = this.api.getCoverUrl(track.album?.cover, '1280');
|
||||
|
|
|
|||
Loading…
Reference in a new issue