Skip to content

feat(storage): tiles show where the preview is, like a library card - #51

Merged
DarrellVS merged 1 commit into
devfrom
storage-playhead
Sep 23, 2026
Merged

DarrellVS merged 1 commit into
devfrom
storage-playhead

Conversation

@DarrellVS

Copy link
Copy Markdown
Owner

Storage Saver tiles had no playback position. The 2px playhead line from library cards is now a shared composable (usePreviewPlayhead), moved out of ClipCard.vue unchanged, and both tiles use it. The tiles also draw GoodBit pips like the library.

Driven: the line grows while a preview plays and sits under the pointer while scrubbing. The screens, features, primitives and infiniteScroll specs: 27 passed. npm run check 849.

🤖 Generated with Claude Code

Storage Saver's tiles played on hover and scrubbed, but drew nothing
saying how far in they were: a video moving under the pointer with no
position, on the screen where the position is what decides a delete.

The two pixel playhead line under a library card's picture is now
`composables/clips/usePreviewPlayhead.ts`, lifted out of `ClipCard.vue`
unchanged (read once per frame while playing, driven by the pointer while
scrubbing, drawn as a transform), and both tiles use it. The storage tile
also draws a clip's GoodBit pips the way a card does.

Driven: the line grows while a preview plays and sits under the pointer
while scrubbing. `screens`, `features`, `primitives` and `infiniteScroll`
specs: 27 passed, so the card itself is unchanged. `npm run check` 849.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@DarrellVS DarrellVS added this to the 3.5.0 milestone Sep 23, 2026
@DarrellVS DarrellVS added the enhancement New feature or request label Sep 23, 2026
@DarrellVS DarrellVS self-assigned this Sep 23, 2026
@DarrellVS
DarrellVS merged commit 0d996dd into dev Sep 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant