Skip to content

fix: remove double border on content in chat #8545

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
matt2e merged 6 commits into from
Apr 16, 2026
Merged

fix: remove double border on content in chat #8545

Changes from 3 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
9a08ba0
fix: remove double border on streamdown table wrappers
matt2e Apr 15, 2026
0aa7253
fix: remove double border on all streamdown wrapper elements
matt2e Apr 15, 2026
ef578e7
fix: place code block language label and actions on the same row
matt2e Apr 15, 2026
d374497
fix: use CSS grid for code block header/actions layout
matt2e Apr 15, 2026
3f1f2b2
fix: bump specificity on code-block grid override
matt2e Apr 15, 2026
b26c7de
fix: remove restating comments from streamdown CSS overrides
matt2e Apr 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions ui/goose2/src/shared/styles/globals.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -753,6 +753,25 @@ body,
}
}

/* Remove outer border on streamdown wrappers (inner content already has one) */

@chatgpt-codex-connector chatgpt-codex-connector Bot Apr 15, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Remove comment that restates selector behavior

The root AGENTS.md rule for this repo says to avoid comments that restate what code does, but this newly added comment is just a plain-language duplicate of the selector plus border: none. Keeping this kind of comment increases drift risk when selectors evolve and violates the documented code-quality standard; remove it (and the similar comment below) or replace with a true "why" rationale.

Useful? React with 👍 / 👎.

[data-streamdown="table-wrapper"],
[data-streamdown="code-block"],
[data-streamdown="mermaid-block"] {
border: none;
}

/* Place code-block language label and actions on the same row */
[data-streamdown="code-block"] {
position: relative;
}

[data-streamdown="code-block"] > [data-streamdown="code-block-header"] + div {
position: absolute;
top: 0.5rem;
right: 0.5rem;
margin-top: 0;
}

/* ═══════════════════════════════════════════════════════════
goose2 custom utilities
═══════════════════════════════════════════════════════════ */
Expand Down
Loading