Skip to content

WorkHub chat bubbles round at 12px instead of --radius-chat #4914

Description

@Astro-Han

What happened

WorkHub's conversation surface passes density="compact" into Astryx ChatMessage / ChatMessageList. ChatMessageBubble then paints --radius-container (12px) instead of --radius-chat (28px). The WorkHub composer still uses --radius-chat, so the bubble and the dock on the same surface disagree by more than 2x.

The main transcript does not. #3452 removed density="compact" from the main ChatMessageList for this pairing. WorkHub put it back in #3497.

@ARE404 this is the WorkHub conversation frame from that PR.

Left is the main transcript (28px). Right is WorkHub (12px). Same story viewport, light and dark separate.

Main transcript 28px vs WorkHub 12px, light

Main transcript 28px vs WorkHub 12px, dark

WorkHub on one surface: the user bubble is 12px, the composer under it is 28px.

WorkHub bubble 12px over composer 28px, light

WorkHub bubble 12px over composer 28px, dark

How to reproduce

  1. Open WorkHub with a turn that renders WorkHubMessageFrame (Storybook Product/WorkHubSubmittedWorkKeepsTargetMetadataInside).
  2. Read computed border-top-left-radius on .maka-chat-message-bubble-user.
  3. Compare with the WorkHub composer plate.

Expected: both 28px (--radius-chat), matching the main transcript.

Actual: user bubble 12px; composer 28px. List and message data-density are compact.

Main transcript, measured on the built Desktop renderer (dist-renderer, Electron turn-narrative fixture, CDP CSS.getComputedStyleForNode): user bubble, assistant bubble, and composer plate all 28px, data-density=balanced.

Environment

  • Maka commit: 07f6027ac (main)
  • Surface: Desktop / WorkHub
  • OS: macOS

Logs, screenshots, or additional context

apps/desktop/src/renderer/workhub-surface.tsx:

  • both ChatMessageList calls: density="compact"
  • WorkHubMessageFrame: ChatMessage density="compact" on the user and assistant rows

Astryx ChatMessageBubble maps density === 'compact' to border-radius: var(--radius-container).

Same fix as #3452: drop the compact density on WorkHub's chat primitives so the bubble stays on --radius-chat. Tracker: #3492.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions