Skip to content

fix(app): prevent message action controls overlapping text#1845

Open
kgarg2468 wants to merge 1 commit into
different-ai:devfrom
kgarg2468:kgarg/fix-session-copy-button-overlap
Open

fix(app): prevent message action controls overlapping text#1845
kgarg2468 wants to merge 1 commit into
different-ai:devfrom
kgarg2468:kgarg/fix-session-copy-button-overlap

Conversation

@kgarg2468
Copy link
Copy Markdown

Summary

  • prevent session message action controls from overlapping message text
  • reserve bottom space in non-nested message frames for the absolute-positioned copy/revert/fork action bar

Why

  • The message action bar is positioned at the bottom-right of the message frame. On short or tightly wrapped messages, the copy button could sit on top of the final line of text.

Issue

Scope

  • Layout-only change in the session transcript message frame classes.

Out of scope

  • No changes to copy behavior, message rendering semantics, transcript sync, or nested transcript rows.

Testing

Ran

  • pnpm --filter @openwork/app typecheck
  • pnpm --filter @openwork/app build
  • PATH="$HOME/.opencode/bin:$PATH" pnpm --filter @openwork/app test:e2e
  • git diff --check

Result

  • pass: typecheck
  • pass: production build
  • pass: app e2e suite
  • pass: whitespace check
  • note: I also ran bun test apps/app/scripts/session-render-state.test.ts; it currently fails 3 existing session-render-state expectations in untouched files, unrelated to this layout-only change.

CI status

  • pass: pending GitHub Actions
  • code-related failures: none known
  • external/env/auth blockers: none known

Manual verification

  1. Confirmed the action bar remains absolutely positioned at the message frame bottom-right.
  2. Confirmed non-nested message frames now reserve bottom padding so controls render in reserved space instead of over the last line.
  3. Confirmed nested transcript rows are unchanged because they do not render this action bar.

Evidence

  • N/A: no screenshot captured locally. This is a CSS layout reservation change validated by typecheck, build, and e2e.

Risk

  • Low. This only adds reserved bottom space to non-nested message frames that already render the hover action bar.

Rollback

  • Revert apps/app/src/react-app/domains/session/surface/message-list.tsx to remove the reserved action space classes.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-landing Ready Ready Preview, Comment, Open in v0 May 19, 2026 7:39am

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 19, 2026

@kgarg2468 is attempting to deploy a commit to the Different AI Team on Vercel.

A member of the Team first needs to authorize it.

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.

Copy button overlaps message text in session chat bubbles

1 participant