diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-06 16:27:48 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-06 16:27:48 -0500 |
| commit | 22f6f46b51a7cd071f3d0683b6e3a522f2050e3e (patch) | |
| tree | 7c36a66863e83938a6e5dd4947c9e1506dd4e264 /modules/system-utils.el | |
| parent | d7c3afffd8592fb57a59b9f2b00bbf8a6d086982 (diff) | |
| download | dotemacs-22f6f46b51a7cd071f3d0683b6e3a522f2050e3e.tar.gz dotemacs-22f6f46b51a7cd071f3d0683b6e3a522f2050e3e.zip | |
docs(specs): review and respond fancy music-player UI spec (READY)
I reviewed the fancy music-player UI spec, then dispositioned all seven findings: five accept, two modify, none rejected. Two were blockers.
The progress bar had a chosen rendering and cadence but no data source: a stream has no duration, and for a local file the elapsed position isn't wired. The playing-time display is off and the mpv IPC helper is send-only. I narrowed it to mpv's percent-pos over the socket the seek commands already use, with streams rendering "on air" and no fill. That avoids reviving the EMMS timer I turned off on purpose.
The named row renderer, cj/music--track-description, is also the header's now-playing source and EMMS's description function, so glyph and meta would have leaked into both. I split it into a pure name-only helper the header and the row share, with the glyph, aligned meta, and image added on the row alone.
The rest tightened scope: favicon-at-creation applies to search-created stations only, fetched art is validated before caching, meta right-aligns with :align-to so it survives a resize, the placeholder is a shipped vinyl SVG with per-station tinting left to vNext, and the nerd-icons dependency gets a require and a TTY fallback. I flipped DRAFT to READY.
Diffstat (limited to 'modules/system-utils.el')
0 files changed, 0 insertions, 0 deletions
