feat(storage): rename a clip on its tile, and refresh when you say so - #52
Merged
Merged
Conversation
A clip could not be renamed from Storage Saver, which is the moment somebody finds the clip worth keeping and wants to say what it is. The tile's name is now `ClipNameInput`, the field a library card uses: it saves on Enter or on leaving it, never per keystroke. Picking is the corner box's job alone, so a click on the name edits it rather than ticking the clip. **The list does not refresh on save.** A named clip is one somebody wants, so it no longer belongs here, but taking it away the moment Enter is pressed would pull the tile from under somebody fixing a typo. The row is patched in place so the new name shows, and a toast says why the clip will leave, with a "Refresh list" button that takes it away when the reader is done. Clearing a name back to nothing says nothing. Driven: renaming a tile keeps all 36 in place with the new name showing, the toast carries the button, and pressing it leaves 35. `screens.spec.ts` 9 passed; `npm run check` 849. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rename a clip on its Storage Saver tile, using the same field a library card uses (saves on Enter or blur, never per keystroke). The list is not refreshed on save, so a typo fix is not cut off. The tile shows the new name at once, and a toast explains that named clips leave this list, with a Refresh list button. Picking is the corner box only.
Driven: a rename keeps all 36 tiles with the new name shown; the toast carries the button; pressing it leaves 35.
screens.spec.ts9 passed;npm run check849.🤖 Generated with Claude Code