Skip to content

feat(dashboard): add terminalBackground field to DashboardTheme#4

Closed
davidgut1982 wants to merge 1 commit into
deploy/delegation-plus-rerankfrom
feat/terminal-background-theme
Closed

feat(dashboard): add terminalBackground field to DashboardTheme#4
davidgut1982 wants to merge 1 commit into
deploy/delegation-plus-rerankfrom
feat/terminal-background-theme

Conversation

@davidgut1982

Copy link
Copy Markdown
Owner

Wires the xterm.js terminal pane background color into the theme system. Previously hardcoded as #0d2626; now reads from DashboardTheme.terminalBackground with #000000 as default.

Users can override via ~/.hermes/dashboard-themes/*.yaml:

terminalBackground: "#1a0a2e"

Files Changed

  • web/src/themes/types.ts — added terminalBackground?: string field
  • web/src/themes/context.tsx — emits --theme-terminal-background CSS var
  • web/src/themes/presets.ts — default terminalBackground: "#000000"
  • web/src/pages/ChatPage.tsx — reads theme.terminalBackground instead of hardcoded value
  • web/package-lock.json — updated by npm install

Wires the xterm.js terminal pane background color into the theme
system. Previously hardcoded as #0d2626; now reads from
DashboardTheme.terminalBackground with #000000 as default.

Users can override via ~/.hermes/dashboard-themes/*.yaml:
  terminalBackground: "#1a0a2e"
@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown

⚠️ npm lockfile hash out of date

Checked against commit 2d6a247 (PR head at check time).

The hash = "sha256-..." line in these nix files no longer matches the committed package-lock.json:

Apply the fix

  • Apply lockfile fix — tick to push a commit with the correct hashes to this PR branch
  • Or run the Nix Lockfile Fix workflow manually (pass PR #4)
  • Or locally: nix run .#fix-lockfiles and commit the diff

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.

1 participant