feat(web): mount TrackMenu in TrackRow + PlayerBar

This commit is contained in:
2026-04-30 20:31:56 -04:00
parent 8da4a3f9c0
commit 88ff997af7
4 changed files with 37 additions and 5 deletions
+2
View File
@@ -7,6 +7,7 @@
import { formatDuration } from '$lib/media/duration';
import { FALLBACK_COVER } from '$lib/media/covers';
import LikeButton from './LikeButton.svelte';
import TrackMenu from './TrackMenu.svelte';
const current = $derived(player.current);
@@ -59,6 +60,7 @@
</a>
</div>
<LikeButton entityType="track" entityId={current.id} />
<TrackMenu track={current} />
</div>
<!-- Center: seek row + transport row -->
+14
View File
@@ -51,6 +51,13 @@ vi.mock('$lib/api/likes', () => ({
unlikeEntity: vi.fn()
}));
vi.mock('$lib/api/quarantine', () => ({
flagTrack: vi.fn(),
unflagTrack: vi.fn(),
listMyQuarantine: vi.fn().mockResolvedValue([]),
createMyQuarantineQuery: vi.fn()
}));
vi.mock('@tanstack/svelte-query', async (orig) => {
const actual = (await orig()) as Record<string, unknown>;
return { ...actual, useQueryClient: () => ({}) };
@@ -174,4 +181,11 @@ describe('PlayerBar', () => {
expect(screen.getByText('1:05')).toBeInTheDocument();
expect(screen.getByText('4:05')).toBeInTheDocument();
});
test('renders the TrackMenu kebab button when a track is current', () => {
render(PlayerBar);
expect(
screen.getByRole('button', { name: /track actions for/i })
).toBeInTheDocument();
});
});
+3 -1
View File
@@ -3,6 +3,7 @@
import { formatDuration } from '$lib/media/duration';
import { playQueue, enqueueTrack, playRadio } from '$lib/player/store.svelte';
import LikeButton from './LikeButton.svelte';
import TrackMenu from './TrackMenu.svelte';
type PlayHandler = (tracks: TrackRef[], index: number) => void;
@@ -47,7 +48,7 @@
aria-label={track.title}
onclick={onRowClick}
onkeydown={onRowKey}
class="grid w-full cursor-pointer grid-cols-[32px_1fr_auto_auto_auto_auto] items-center gap-4 px-3 py-2 text-left text-sm odd:bg-surface/50 hover:bg-surface focus-visible:outline focus-visible:outline-2 focus-visible:outline-accent"
class="grid w-full cursor-pointer grid-cols-[32px_1fr_auto_auto_auto_auto_auto] items-center gap-4 px-3 py-2 text-left text-sm odd:bg-surface/50 hover:bg-surface focus-visible:outline focus-visible:outline-2 focus-visible:outline-accent"
>
<span class="text-right tabular-nums text-text-secondary">
{track.track_number ?? '—'}
@@ -67,5 +68,6 @@
onclick={onAddClick}
class="rounded p-1 text-text-secondary hover:text-text-primary"
>+</button>
<TrackMenu {track} />
<span class="tabular-nums text-text-secondary">{formatDuration(track.duration_sec)}</span>
</div>
+18 -4
View File
@@ -19,6 +19,13 @@ vi.mock('$lib/api/likes', () => ({
unlikeEntity: vi.fn()
}));
vi.mock('$lib/api/quarantine', () => ({
flagTrack: vi.fn(),
unflagTrack: vi.fn(),
listMyQuarantine: vi.fn().mockResolvedValue([]),
createMyQuarantineQuery: vi.fn()
}));
vi.mock('@tanstack/svelte-query', async (orig) => {
const actual = (await orig()) as Record<string, unknown>;
return { ...actual, useQueryClient: () => ({}) };
@@ -62,26 +69,26 @@ describe('TrackRow', () => {
test('clicking the row calls playQueue(tracks, index)', async () => {
render(TrackRow, { props: { tracks, index: 1 } });
await fireEvent.click(screen.getByRole('button', { name: /Freddie Freeloader/ }));
await fireEvent.click(screen.getByRole('button', { name: 'Freddie Freeloader' }));
expect(playQueue).toHaveBeenCalledWith(tracks, 1);
});
test('Enter on the row activates play', async () => {
render(TrackRow, { props: { tracks, index: 0 } });
await fireEvent.keyDown(screen.getByRole('button', { name: /So What/ }), { key: 'Enter' });
await fireEvent.keyDown(screen.getByRole('button', { name: 'So What' }), { key: 'Enter' });
expect(playQueue).toHaveBeenCalledWith(tracks, 0);
});
test('Space on the row activates play', async () => {
render(TrackRow, { props: { tracks, index: 0 } });
await fireEvent.keyDown(screen.getByRole('button', { name: /So What/ }), { key: ' ' });
await fireEvent.keyDown(screen.getByRole('button', { name: 'So What' }), { key: ' ' });
expect(playQueue).toHaveBeenCalledWith(tracks, 0);
});
test('onPlay prop overrides default playQueue', async () => {
const onPlay = vi.fn();
render(TrackRow, { props: { tracks, index: 1, onPlay } });
await fireEvent.click(screen.getByRole('button', { name: /Freddie Freeloader/ }));
await fireEvent.click(screen.getByRole('button', { name: 'Freddie Freeloader' }));
expect(onPlay).toHaveBeenCalledWith(tracks, 1);
expect(playQueue).not.toHaveBeenCalled();
});
@@ -99,4 +106,11 @@ describe('TrackRow', () => {
expect(playRadio).toHaveBeenCalledWith('t1');
expect(playQueue).not.toHaveBeenCalled();
});
test('renders the TrackMenu kebab button', () => {
render(TrackRow, { props: { tracks, index: 0 } });
expect(
screen.getByRole('button', { name: /track actions for/i })
).toBeInTheDocument();
});
});