2 Commits

Include renames

Author	SHA1	Message	Date
HoneyandClaude Sonnet 4.6	920e385a8f	fix: break static circular dep in notify.js that caused 'Failed to fetch' after ui split ... notify.js statically imported navigate from views/views.js, creating a new circular chain: navigation.js → ui/ui.js → settings.js → downloads.js → notify.js → views/views.js → navigation.js. This chain did not exist in the old monolithic ui.js and caused module initialisation failures in some browsers. Fix: make navigate a lazy dynamic import inside _notifNavigate() body, which is the only call site. Also replace unnecessary dynamic import('../utils.js') calls with the already-statically-imported toast function. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-07 19:49:29 +02:00
HoneyandClaude Sonnet 4.6	0663b1c923	refactor: split ui/ui.js into 5 focused modules ... settings.js — settings modal, romaji overrides, sidebar, theme, nav mode modals.js — context menu, playlist-add modal, confirm dialog, edit-playlist modal metadata.js — song info/edit modal, MusicBrainz/iTunes autofill, saveMetadata, deleteSong notify.js — notification panel, bell badge, _addNotif, _notifNavigate downloads.js — download modal, Spotify download, jobs panel, startArtistSortFix ui.js — barrel re-export; all external imports unchanged player.js left unsplit: shared private state (_nativePlaying, _playGen, _queueGen, etc.) spans every section — splitting would require exporting internal state. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-07 14:00:38 +02:00