Skip to content

fix(chat): make composer nav buttons readable in dark mode - #99

Merged
luoxuanzao merged 1 commit into
mainfrom
fix/nav-button-dark-contrast
Nov 12, 2026
Merged

luoxuanzao merged 1 commit into
mainfrom
fix/nav-button-dark-contrast

Conversation

@luoxuanzao

Copy link
Copy Markdown
Member

Summary

  • The new-tab, new-conversation, and chat-history buttons above the input (.qoderian-input-nav-btn) used --text-faint, Obsidian's dimmest text token, for their default icon color. In dark mode that measured 2.57:1 contrast against the workspace background (#666666 on #282828) — below the WCAG 3:1 minimum for graphical objects — so the icons were nearly invisible until hovered (hover jumps to --text-normal).
  • Switch the default color to --text-muted, the same token the other composer controls already use, raising contrast to 7.03:1 (#b3b3b3 on #282828). Hover stays --text-normal. One-line CSS change, no !important, consistent with sibling buttons.

Verification

  • npm run typecheck
  • npm run lint
  • npm run test (3611 passed)
  • npm run build
  • npm run release:check
  • npm run audit:prod (0 vulnerabilities)
  • Tested affected qodercli behavior against a real CLI when applicable — verified in real Obsidian (dark mode, legacy tab mode): the three buttons' computed contrast rose from 2.57:1 to 7.03:1 after the change; before/after screenshots captured.

Safety

  • No credentials, private vault content, internal URLs, or personal paths are included
  • User-visible changes are documented in CHANGELOG.md

The new-tab, new-conversation, and chat-history buttons above the input
used --text-faint, the dimmest text token, for their default icon color.
Against the dark workspace background that measured ~2.6:1 contrast, so
the icons were nearly invisible until hovered. Use --text-muted like the
other composer controls (~7:1), keeping the hover state at --text-normal.

Co-authored-by: QoderAI (Qwen 3.8 Max) <qoder_ai@qoder.com>
@luoxuanzao
luoxuanzao merged commit 6309de1 into main Nov 12, 2026
10 checks passed
@luoxuanzao
luoxuanzao deleted the fix/nav-button-dark-contrast branch November 12, 2026 08:30
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