Skip to content

fix(session-ui): align code copy tooltip with v2 style#34459

Merged
Brendonovich merged 1 commit into
devfrom
code-tooltip-v2
Jun 30, 2026
Merged

fix(session-ui): align code copy tooltip with v2 style#34459
Brendonovich merged 1 commit into
devfrom
code-tooltip-v2

Conversation

@opencode-agent

Copy link
Copy Markdown
Contributor

Summary

  • Restyle markdown code-block copy tooltip to match TooltipV2 tokens in the new layout
  • Preserve the legacy tooltip appearance under body:not([data-new-layout])

Validation

  • bun typecheck from packages/session-ui
  • bun typecheck from packages/app
  • git diff --check
  • Production session timeline benchmark before/after (timeline/session-timeline-benchmark.spec.ts)
    • before: passed, completionObservedMs 64107.8, rowReplaced false, markdownReplaced false
    • after: passed, completionObservedMs 64372.9, rowReplaced false, markdownReplaced false
  • Real-app before/after screenshot + pixel diff captured with mocked backend fixture; changed region limited to tooltip (1297 / 189440 pixels, 0.685%)

Notes

  • Chromium needed cached runtime libs via LD_LIBRARY_PATH=/home/gangprompt/services/opencord/agent-root/output/browser-libs/usr/lib for local Playwright runs.

Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com>
@opencode-agent opencode-agent Bot marked this pull request as ready for review June 29, 2026 19:25
@Brendonovich Brendonovich merged commit 1982d98 into dev Jun 30, 2026
9 checks passed
@Brendonovich Brendonovich deleted the code-tooltip-v2 branch June 30, 2026 09:01
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.

2 participants