Skip to content

style: add dark mode inline code styling for message bubbles - #10861

Merged
alexhancock merged 3 commits into
aaif-goose:mainfrom
kursad-k:fix.darktheme-styling
Aug 3, 2026
Merged

style: add dark mode inline code styling for message bubbles#10861
alexhancock merged 3 commits into
aaif-goose:mainfrom
kursad-k:fix.darktheme-styling

Conversation

@kursad-k

Copy link
Copy Markdown
Contributor

Add transparent background and teal text color (#8cd3d6) for inline code blocks inside agent and user message bubbles in dark mode.

Summary

Refine inline code styling in dark-mode chat messages by removing the striped background and using a softer turquoise text color.

Testing

Manually tested user prompts and agent responses on Windows 11.

Screenshots/Demos (for UX changes)

Before:
image

After:
image

Add transparent background and teal text color (#8cd3d6) for inline
code blocks inside agent and user message bubbles in dark mode.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 90bd423955

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread ui/desktop/src/styles/main.css Outdated
Comment on lines +619 to +622
.dark :is(.agent-message-bubble, .user-message-bubble) .bg-inline-code {
background-color: transparent;
color: #8cd3d6;
padding: 0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Limit dark inline-code styling to inline code

When a dark-mode message contains a fenced or indented code block without a language-* class, MarkdownContent falls back to rendering that block as <pre><code class="... bg-inline-code ...">. This selector therefore also hits unlabeled block code inside both message bubbles, making the entire block use the inline teal color and clearing the code element's dark background/padding instead of only restyling inline snippets. Please exclude pre > code.bg-inline-code or target inline code more narrowly.

Useful? React with 👍 / 👎.

kursad added 2 commits August 1, 2026 17:05
Exclude fenced and indented code blocks from the dark-theme inline code override.
Remove per-line background stripes and replace orange text with a neutral cool gray while preserving the parent code block surface and padding.
@alexhancock
alexhancock self-requested a review August 3, 2026 16:26
@alexhancock
alexhancock added this pull request to the merge queue Aug 3, 2026
Merged via the queue into aaif-goose:main with commit 2db0e31 Aug 3, 2026
24 checks passed
michaelneale added a commit that referenced this pull request Aug 4, 2026
* origin/main: (42 commits)
  chore (codex-acp): migrate to @agentclientprotocol/codex-acp (#10923)
  style: add dark mode inline code styling for message bubbles (#10861)
  docs: clarify macOS sandbox feature was removed in post-v1.25.0 cleanup (#10900)
  fall back to static model list only for non-models payloads (#10189)
  Remove CLI project support (#10838)
  fix(anthropic): add claude-sonnet-5 and claude-fable-5 to known models list (#10865)
  fix: stdio extensions silently skipped when name missing or env: used in config (#10773)
  feat(desktop): show session metadata on sidebar chat hover (#10663)
  fix(serve): start scheduler at startup instead of first ACP connection (#10766)
  fix: respect disabled builtin extensions (developer) at session start (#10223)
  fix: parse PyPI requirements for OSV checks (#10510)
  fix(telegram): contain voice file extensions (#10456)
  Revert "feat(security): chunk command-classifier input with overlapping windows" (#10416) (#10870)
  docs: update Discord invite (#10863)
  fix(security): preserve denied tool request precedence (#10612)
  fix(hints): contain subdirectory hint discovery (#10545)
  chore(deps): bump pem from 3.0.6 to 4.0.0 (#10853)
  chore(deps): bump base64 from 0.22.1 to 0.23.0 (#10851)
  chore(deps): bump jsonwebtoken from 10.4.0 to 11.0.0 (#10850)
  chore(deps): bump astral-sh/setup-uv from 8.3.2 to 9.0.0 (#10847)
  ...
alexhancock added a commit that referenced this pull request Aug 4, 2026
* origin/main: (58 commits)
  Handle MCP tool list change notifications
  fix: restore final_output_tool when agent is recreated after LRU eviction (#10793)
  feat(ui): bring back make sidebar resizable with drag handle and persisted width (#10795)
  docs: document auto-injected GOOSE_SHELL flags (#10759)
  feat: surface output-token-limit info (#10831)
  fix(desktop): avoid O(n²) cloning during session load (#10665)
  feat: add interactive menu for single select elicitations (#10327)
  fix(acp): prefix child stderr log target so it passes the default goose=info filter (#10901)
  chore (codex-acp): migrate to @agentclientprotocol/codex-acp (#10923)
  style: add dark mode inline code styling for message bubbles (#10861)
  docs: clarify macOS sandbox feature was removed in post-v1.25.0 cleanup (#10900)
  fall back to static model list only for non-models payloads (#10189)
  Remove CLI project support (#10838)
  fix(anthropic): add claude-sonnet-5 and claude-fable-5 to known models list (#10865)
  fix: stdio extensions silently skipped when name missing or env: used in config (#10773)
  feat(desktop): show session metadata on sidebar chat hover (#10663)
  fix(serve): start scheduler at startup instead of first ACP connection (#10766)
  fix: respect disabled builtin extensions (developer) at session start (#10223)
  fix: parse PyPI requirements for OSV checks (#10510)
  fix(telegram): contain voice file extensions (#10456)
  ...

# Conflicts:
#	Cargo.lock
#	crates/goose/src/agents/extension_manager.rs
#	crates/goose/tests/mcp_replays/npx-y@modelcontextprotocol_server-everything@2026.1.14
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.

2 participants