Skip to content

refactor(tui): migrate dialog views to V2 themes#37516

Merged
jlongster merged 1 commit into
v2from
migrate-theme-dialogs
Jul 17, 2026
Merged

refactor(tui): migrate dialog views to V2 themes#37516
jlongster merged 1 commit into
v2from
migrate-theme-dialogs

Conversation

@jlongster

Copy link
Copy Markdown
Contributor

Summary

  • migrate ten dialog and lifecycle views from flat V1 colors to semantic V2 tokens
  • use elevated contextual tokens for dialog surfaces
  • preserve selected button contrast with paired focused action tokens

Verification

  • bun typecheck
  • bun test test/cli/tui/dialog-prompt.test.tsx test/cli/tui/dialog-select.test.tsx --timeout 30000 (8 passed)
  • git diff --check

OpenCode Drive screenshots will be attached in a follow-up comment.

@jlongster

Copy link
Copy Markdown
Contributor Author

OpenCode Drive visual verification

Captured from this branch with isolated simulated projects. Each migrated file is represented below.

Pair dialog

packages/tui/src/component/dialog-pair.tsx

Pair dialog

Status dialog

packages/tui/src/component/dialog-status.tsx

Status dialog

Project copy name

packages/tui/src/component/dialog-project-copy-name.tsx

Project copy name

Missing plugin route

packages/tui/src/component/plugin-route-missing.tsx

Missing plugin route

Reconnecting overlay

packages/tui/src/component/reconnecting.tsx

Reconnecting overlay

Startup loading

packages/tui/src/component/startup-loading.tsx

Startup loading

Help dialog

packages/tui/src/ui/dialog-help.tsx

Help dialog

Alert dialog

packages/tui/src/ui/dialog-alert.tsx

Alert dialog

Export result

packages/tui/src/ui/dialog-export-result.tsx

Export result

Confirm dialog

packages/tui/src/ui/dialog-confirm.tsx

Confirm dialog

@jlongster
jlongster merged commit 677526d into v2 Jul 17, 2026
10 checks passed
@jlongster
jlongster deleted the migrate-theme-dialogs branch July 17, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant