From 58aa9b8955e6c640400066e9cacb2d8de526f058 Mon Sep 17 00:00:00 2001 From: Federico Jaramillo Martinez Date: Tue, 14 Jul 2026 23:28:06 +0200 Subject: [PATCH] fix: use standard chat styling for tool images --- .changeset/show-tool-images-outside-events.md | 2 +- src/client/src/components/ChatView.image.test.ts | 2 +- src/client/src/components/shared.ts | 8 ++------ 3 files changed, 4 insertions(+), 8 deletions(-) diff --git a/.changeset/show-tool-images-outside-events.md b/.changeset/show-tool-images-outside-events.md index 86686e4..387debe 100644 --- a/.changeset/show-tool-images-outside-events.md +++ b/.changeset/show-tool-images-outside-events.md @@ -2,4 +2,4 @@ "@jmfederico/pi-web": patch --- -Keep tool-result images visible as compact, clearly labeled output outside collapsed event groups while retaining technical execution details and final message metadata. +Keep tool-result images visible in clearly labeled standard chat cards outside collapsed event groups while retaining technical execution details and final message metadata. diff --git a/src/client/src/components/ChatView.image.test.ts b/src/client/src/components/ChatView.image.test.ts index fc787bc..b810478 100644 --- a/src/client/src/components/ChatView.image.test.ts +++ b/src/client/src/components/ChatView.image.test.ts @@ -27,7 +27,7 @@ describe("ChatView image rendering", () => { // Direct rendering keeps this node-environment test focused on the dedicated // tool-image presentation without introducing a component-wide DOM shim. - it("renders tool images as compact labeled output with final metadata", () => { + it("renders tool images as labeled standard messages with final metadata", () => { const message: ChatLine = { role: "tool", parts: [{ type: "image", mimeType: "image/png", data: "QUJD" }], diff --git a/src/client/src/components/shared.ts b/src/client/src/components/shared.ts index 36cb1b3..cbdf86c 100644 --- a/src/client/src/components/shared.ts +++ b/src/client/src/components/shared.ts @@ -281,10 +281,9 @@ export const chatStyles = css` .dot { width: 8px; height: 8px; border-radius: 50%; background: currentColor; opacity: .45; flex: 0 0 auto; } .activity-dock.active .dot { animation: pulse 1s ease-in-out infinite; opacity: 1; } .msg { max-width: 100%; min-width: 0; box-sizing: border-box; margin: 0 0 14px; padding: 12px; border: 1px solid var(--pi-border); border-radius: 10px; background: var(--pi-surface); overflow: visible; } - .msg.assistant { background: var(--pi-surface); } + .msg.assistant, .msg.tool-image-output { background: var(--pi-surface); } .msg.user { border-color: var(--pi-accent-border); background: var(--pi-selection-bg); } .msg.tool { border-color: var(--pi-warning-border); background: var(--pi-warning-surface); color: var(--pi-warning); } - .msg.tool-image-output { padding: 0; border: 0; background: transparent; color: var(--pi-text); } .msg.tool-execution-shell { padding: 0; border: 0; background: transparent; color: var(--pi-text); } .msg.system { color: var(--pi-danger); } .msg.bash { border-color: var(--pi-success); background: var(--pi-success-bg); } @@ -322,14 +321,11 @@ export const chatStyles = css` .msg-header { display: flex; align-items: center; justify-content: space-between; gap: 10px; min-height: 22px; margin-bottom: 8px; } .msg > .msg-header { position: sticky; top: -26px; z-index: 4; margin: -12px -12px 8px; padding: 7px 10px 6px; border-radius: 9px 9px 0 0; border-bottom: 1px solid color-mix(in srgb, var(--pi-border-muted) 35%, transparent); background: var(--pi-surface); box-shadow: 0 8px 18px var(--pi-shadow-soft); } .msg.user > .msg-header { border-bottom-color: color-mix(in srgb, var(--pi-accent-border) 35%, transparent); background: var(--pi-selection-bg); } - .msg.assistant > .msg-header .label { color: var(--pi-text-secondary); } + .msg.assistant > .msg-header .label, .msg.tool-image-output > .msg-header .label { color: var(--pi-text-secondary); } .msg.user > .msg-header .label { color: var(--pi-accent); } .msg.tool > .msg-header { border-bottom-color: color-mix(in srgb, var(--pi-warning-border) 35%, transparent); background: var(--pi-warning-surface); } .msg.bash > .msg-header { border-bottom-color: color-mix(in srgb, var(--pi-success) 35%, transparent); background: var(--pi-success-bg); } .msg.skill > .msg-header { border-bottom-color: color-mix(in srgb, var(--pi-purple-border) 35%, transparent); background: var(--pi-purple-surface); } - .msg.tool-image-output > .msg-header { position: static; min-height: 18px; margin: 0 0 6px; padding: 0; border: 0; border-radius: 0; background: transparent; box-shadow: none; } - .msg.tool-image-output > .msg-header .label { color: var(--pi-muted); text-transform: none; } - .msg.tool-image-output .chat-image { margin-top: 0; } .group-msg > .msg-header { position: sticky; top: -26px; z-index: 4; margin: -10px 0 8px; padding: 7px 0 6px; border-bottom: 1px solid color-mix(in srgb, var(--pi-border-muted) 35%, transparent); background: var(--pi-bg); } .msg-header-trailing { min-width: 0; flex: 1 1 auto; display: inline-flex; align-items: center; justify-content: flex-end; gap: 8px; } .msg-actions { flex: 0 0 auto; display: inline-flex; gap: 6px; opacity: 0; transition: opacity .12s ease; }