From c518ac8c73a63dd0d2cdfe79fe3298aa3ef513dc Mon Sep 17 00:00:00 2001 From: Caleb Allen Date: Wed, 19 Aug 2026 13:36:16 -0400 Subject: [PATCH] feat(now-playing): plain text instead of links in the phone mini player MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The bottom bar's title, artist and "Playing from X" all navigate. In a bar sized for a bar they are a few characters of text, which is not a touch target — and explore-link holds its navigation for one double-click interval and drops it if a second click arrives, a gesture that exists so double-clicking a row can play it and that means nothing on touch. Below the shell's phone breakpoint the three render as plain text. The words are unchanged: the source line still says where the queue came from, because dropping the link is the change and dropping the information would be a different and worse one. The cover art already carries the phone-only button that opens the full-screen Now Playing view, which is where the links live. This is in JS rather than in the stylesheet because what changes is the content, not its appearance — no CSS rule takes a click handler off an element. matchMedia is read in connectedCallback for the reason the reduce-motion query beside it already is, so a test can answer it first. Two smaller things. PHONE_QUERY moves out of track-list.ts into utils/breakpoints.ts: it was a private const when one component needed it, and a second reader is where a copy starts drifting from index.css. And `phone` joins geometryKey(), because crossing the breakpoint swaps a link for a bare string and the marquee travels a distance read from measuring it — the words being identical either side is not the same as the box measuring the same. Closes #61 --- .../src/components/now-playing/now-playing.ts | 38 +++- .../src/components/track-list/track-list.ts | 4 +- frontend/src/utils/breakpoints.ts | 23 +++ .../test/components/now-playing-phone.test.ts | 166 ++++++++++++++++++ 4 files changed, 225 insertions(+), 6 deletions(-) create mode 100644 frontend/src/utils/breakpoints.ts create mode 100644 frontend/test/components/now-playing-phone.test.ts diff --git a/frontend/src/components/now-playing/now-playing.ts b/frontend/src/components/now-playing/now-playing.ts index 4405695..0057f1d 100644 --- a/frontend/src/components/now-playing/now-playing.ts +++ b/frontend/src/components/now-playing/now-playing.ts @@ -13,6 +13,7 @@ import { isQueueSourceNavigable, navigateToQueueSource, } from '@utils/queue-source-link'; +import { PHONE_QUERY } from '@utils/breakpoints'; import { PlayerController } from '@store/controllers/player-controller'; import { creditStore } from '@store/credit-store'; import { QueueController } from '@store/controllers/queue-controller'; @@ -80,6 +81,19 @@ export class NowPlaying extends LitElement { private reduceMotionQuery?: MediaQueryList; + /** + * Phone width, from the shell's own breakpoint. + * + * This is in JS rather than in the stylesheet because what changes + * is the *content*, not its appearance: the title, artist and + * source render as plain text instead of as links, and no CSS rule + * can take a click handler off an element. + */ + @state() + private phone = false; + + private phoneQuery?: MediaQueryList; + /** Whether each field is actively mid-scroll (class toggle). */ @state() private titleScrolling = false; @@ -341,6 +355,12 @@ export class NowPlaying extends LitElement { this.reduceMotion = this.reduceMotionQuery?.matches ?? false; this.reduceMotionQuery?.addEventListener('change', this.handleReduceMotionChange); + // Same reasoning as above: looked up here, not at module load, + // so a test can install its own matchMedia first. + this.phoneQuery = window.matchMedia?.(PHONE_QUERY); + this.phone = this.phoneQuery?.matches ?? false; + this.phoneQuery?.addEventListener('change', this.handlePhoneChange); + this.resizeObserver = new ResizeObserver(() => { this.geometryDirty = true; this.requestUpdate(); @@ -364,6 +384,7 @@ export class NowPlaying extends LitElement { this.attachDragListeners(false); window.removeEventListener(SCROLL_CHANGE_EVENT, this.handleScrollModeEvent); this.reduceMotionQuery?.removeEventListener('change', this.handleReduceMotionChange); + this.phoneQuery?.removeEventListener('change', this.handlePhoneChange); this.resizeObserver?.disconnect(); this.stopScrollCycle('title'); this.stopScrollCycle('artist'); @@ -488,7 +509,7 @@ export class NowPlaying extends LitElement { @mouseleave=${this.handleTitleMouseLeave} @transitionend=${() => this.onScrollCycleEnd('title')} > - ${trackLink(track.title, track.album, track.releaseGroupMbid, track.recordingMbid) || track.title} + ${this.phone ? track.title : trackLink(track.title, track.album, track.releaseGroupMbid, track.recordingMbid) || track.title} this.onScrollCycleEnd('artist')} > - ${creditLink(creditStore.credits(track.recordingMbid), track.artist, track.artistMbid) || 'Unknown Artist'} + ${this.phone ? track.artist || 'Unknown Artist' : creditLink(creditStore.credits(track.recordingMbid), track.artist, track.artistMbid) || 'Unknown Artist'} ${describeQueueSource(this.queue.source) ? html` { + if (this.phone) return; if (!isQueueSourceNavigable(this.queue.source)) return; navigateToQueueSource( e.currentTarget as EventTarget, @@ -571,6 +593,10 @@ export class NowPlaying extends LitElement { this.reduceMotion = e.matches; }; + private handlePhoneChange = (e: MediaQueryListEvent): void => { + this.phone = e.matches; + }; + private shouldScroll(field: 'title' | 'artist'): boolean { const overflows = field === 'title' ? this.titleOverflows : this.artistOverflows; @@ -606,6 +632,12 @@ export class NowPlaying extends LitElement { track?.artist ?? '', this.shouldScroll('title') ? '1' : '0', this.shouldScroll('artist') ? '1' : '0', + // Crossing the breakpoint swaps a link for a bare string, + // and a link is not guaranteed to measure the same as the + // text inside it. The marquee travels a distance read from + // that measurement, so this belongs in the key even though + // the words are identical either side. + this.phone ? '1' : '0', ].join('\u0000'); } diff --git a/frontend/src/components/track-list/track-list.ts b/frontend/src/components/track-list/track-list.ts index df61e31..d278389 100644 --- a/frontend/src/components/track-list/track-list.ts +++ b/frontend/src/components/track-list/track-list.ts @@ -11,6 +11,7 @@ import { import { SelectionController } from '@utils/selection-controller'; import type { SelectionHost } from '@utils/selection-controller'; import { ViewLifecycleMixin } from '@utils/view-lifecycle'; +import { PHONE_QUERY } from '@utils/breakpoints'; import { ContextMenuController, contextMenuStyles, @@ -105,9 +106,6 @@ const ROW_CHROME_WIDTH = const ROW_HEIGHT = 33; const PHONE_ROW_HEIGHT = 52; -/** The shell's phone breakpoint, as `index.css` and every component - * stylesheet spells it. */ -const PHONE_QUERY = '(max-width: 599px)'; // Inline SVG paths for favorite icons — eliminates wa-icon shadow DOM // overhead (30-50 shadow roots during scroll). Font Awesome 6 paths. diff --git a/frontend/src/utils/breakpoints.ts b/frontend/src/utils/breakpoints.ts new file mode 100644 index 0000000..2c5b026 --- /dev/null +++ b/frontend/src/utils/breakpoints.ts @@ -0,0 +1,23 @@ +/** + * The shell's breakpoints, where JavaScript has to agree with CSS. + * + * A media query inside a shadow root is answered by the viewport, so a + * component normally states what it drops at phone width in its own + * stylesheet and needs nothing from here. This exists for the cases + * where the decision is not a style: `track-list` computes its grid in + * JS from the host width, and `now-playing` renders *different content* + * on a phone — a plain string instead of a link — which no stylesheet + * can express. + * + * One breakpoint, several expressions of it. It was a private const in + * track-list.ts when there was one; a second reader is where a copy + * would start drifting from index.css. + */ + +/** + * Phone width. 600px rather than the sidebar's 900px because 900 is a + * laptop: the answer there is a narrower sidebar, which is still a + * sidebar. Below this the shell drops the sidebar column entirely and + * bottom-nav takes over. + */ +export const PHONE_QUERY = '(max-width: 599px)'; diff --git a/frontend/test/components/now-playing-phone.test.ts b/frontend/test/components/now-playing-phone.test.ts new file mode 100644 index 0000000..027e5c5 --- /dev/null +++ b/frontend/test/components/now-playing-phone.test.ts @@ -0,0 +1,166 @@ +/** + * The mini player's links are a desktop affordance. + * + * `utils/explore-link.ts` makes every track and artist name navigate, + * and `utils/queue-source-link.ts` makes "Playing from X" navigate — in + * the bottom bar those are a few characters of text at a font size + * chosen for a bar, which is not a touch target. Worse, explore-link + * holds the navigation for one double-click interval and drops it if a + * second click arrives: a gesture that exists so double-clicking a row + * can play it, and which means nothing at all on touch. + * + * So below the shell's phone breakpoint the three render as plain text + * and the whole bar's cover art opens the full-screen Now Playing view, + * which is where the links live. + * + * The breakpoint is stubbed rather than emulated for the reason + * track-list-phone.test.ts states: this tier's viewport is fixed at + * 1280x800 by the runner, and the component reads matchMedia in + * connectedCallback precisely so a test can answer it first. + */ +import { describe, expect, it, beforeEach } from 'vitest'; + +import '@components/now-playing/now-playing'; +import { Events } from '../../src/events'; +import { emit, flush } from '@test/support/harness'; +import { fixture, shadow, shadowAll, text } from '@test/support/render'; +import type { TrackInfo } from '@store/player-store'; +import type { QueueTrack } from '@store/queue-store'; + +const TRACK: TrackInfo = { + fileName: 'ashes.mp3', + filePath: '/music/ashes.mp3', + trackLength: 215, + seekPosition: 0, + state: 'playing', + title: 'Ashes to Ashes', + artist: 'David Bowie', + album: 'Scary Monsters', + coverArt: '', + coverArtSmall: '', + coverArtMedium: '', + coverArtLarge: '', + trackChangeId: 1, + artistMbid: '', + releaseGroupMbid: '', + recordingMbid: '', +}; + +function queueTrack(n: number, title: string): QueueTrack { + return { + id: n, + audioFileId: n, + filePath: `/music/${n}.mp3`, + position: n, + title, + artist: 'David Bowie', + album: 'Scary Monsters', + coverArtPath: '', + artistMbid: '', + releaseGroupMbid: '', + recordingMbid: '', + }; +} + +/** Mount the bar with the phone breakpoint answering `matches`. */ +async function mountAt(phone: boolean) { + const real = window.matchMedia.bind(window); + + window.matchMedia = ((q: string) => + q.includes('max-width: 599px') + ? { + matches: phone, + media: q, + addEventListener() {}, + removeEventListener() {}, + } + : real(q)) as typeof window.matchMedia; + + try { + const el = await fixture('now-playing'); + + emit(Events.TrackChanged, { ...TRACK, trackChangeId: 20 }); + emit(Events.QueueChanged, { + tracks: [queueTrack(1, 'Ashes to Ashes')], + currentIndex: 0, + source: { type: 'album', id: 7, label: 'Scary Monsters' }, + }); + await flush(); + await el.updateComplete; + + return el; + } finally { + window.matchMedia = real; + } +} + +describe('the mini player on a phone', () => { + beforeEach(() => { + emit(Events.TrackChanged, { ...TRACK, trackChangeId: 1 }); + }); + + it('renders the title and artist as plain text', async () => { + const el = await mountAt(true); + + expect(shadowAll(el, '.explore-link').length).toBe(0); + + // The words are unchanged — this is about what they are, not about + // hiding them. A fix that dropped the text would pass an assertion + // about links alone. + expect(text(el, '[data-testid="now-playing-title"]')).toContain( + 'Ashes to Ashes', + ); + expect(text(el, '[data-testid="now-playing-artist"]')).toContain( + 'David Bowie', + ); + }); + + it('does not navigate from the source line', async () => { + const el = await mountAt(true); + const source = shadow(el, '[data-testid="now-playing-source"]'); + + expect(source?.classList.contains('navigable')).toBe(false); + + let navigated = false; + el.addEventListener('navigate', () => { + navigated = true; + }); + + source?.click(); + + expect(navigated).toBe(false); + }); + + it('still says where the queue came from', async () => { + const el = await mountAt(true); + + // Dropping the *link* is the change; dropping the information would + // be a different and worse one. + expect(text(el, '[data-testid="now-playing-source"]')).toBe( + 'Playing from Scary Monsters', + ); + }); + + it('leaves the desktop bar exactly as it was', async () => { + const el = await mountAt(false); + + expect(shadowAll(el, '.explore-link').length).toBeGreaterThan(0); + + const source = shadow(el, '[data-testid="now-playing-source"]'); + + expect(source?.classList.contains('navigable')).toBe(true); + + let detail: unknown; + el.addEventListener('navigate', (e) => { + detail = (e as CustomEvent).detail; + }); + + source?.click(); + + expect(detail).toEqual({ + view: 'explore-album-details', + localAlbumId: 7, + albumName: 'Scary Monsters', + }); + }); +});