Skip to content

Show the active selector as pills instead of raw wrapped text - #909

Merged
ankit merged 3 commits into
worktree-editor-ui-redesignfrom
selector-pills-display
Sep 13, 2026
Merged

ankit merged 3 commits into
worktree-editor-ui-redesignfrom
selector-pills-display

Conversation

@ankit

@ankit ankit commented Sep 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • When not focused, a comma-separated selector now renders as individual pills (matching the dropdown list item's chip style) instead of plain wrapped text.
  • The underlying textarea stays fully editable — clicking the pills (or focusing in) reveals the raw text so manual editing works exactly as before.

Test plan

  • yarn typecheck / yarn lint / yarn test (260/260)

🤖 Generated with Claude Code

SAutocomplete gains an opt-in 'chips' mode: when not focused, a
comma-separated value renders as individual pills (matching the
dropdown list item's chip style) instead of plain wrapped text, while
the underlying textarea stays fully editable — clicking the pills (or
just tabbing/focusing in) reveals the raw text again so manual editing
still works exactly as before.
…aying

The chips were absolutely positioned over the textarea, so the pill's
height followed the textarea's auto-grow instead of the (potentially
taller, differently-wrapped) chip content, clipping the last row.
Rendering only one of the two (v-if/v-else) as a normal flex child
lets the container size to whichever is actually showing.
Left/bottom padding on the selector's pill display now matches top.
The .chip style was duplicated verbatim between the dropdown list item
and the new pill display — extracted into a shared SChip component
used by both.
@ankit
ankit merged commit ce19e1a into worktree-editor-ui-redesign Sep 13, 2026
3 checks passed
@ankit
ankit deleted the selector-pills-display branch September 13, 2026 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant