Skip to content

Conversation

@nikaro
Copy link

@nikaro nikaro commented Oct 6, 2025

  • I have read CONTRIBUTING.md.
  • I have created a discussion that was approved by a maintainer (for new features).

Relates: #755

I gave a try at vibecoding this features. But as i said in the issue i'm not competent enough to review it properly. So feel free to take over it.

In its current state it works, but since markdown colors are hardcoded it cannot be changed by the theme, and the text contrast is not right (edit: vibefixed).

Here are some screenshots (the asciinema record did not kept the colors...):
Capture d’écran 2025-10-06 à 21 29 17
Capture d’écran 2025-10-06 à 21 29 52
Capture d’écran 2025-10-06 à 21 29 57

@charmcli
Copy link
Contributor

charmcli commented Oct 6, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@nikaro
Copy link
Author

nikaro commented Oct 6, 2025

I have read the Contributor License Agreement (CLA) and hereby sign the CLA.

nikaro and others added 3 commits October 13, 2025 20:56
Added structured Markdown color support with theme-based customization:
- Added MarkdownColors and ChromaColors structs to Theme
- Added helper functions for color fallback and hex conversion
- Updated NewCharmtoneTheme() with comprehensive Markdown colors
- Updated NewCharmtoneLightTheme() with appropriate light theme colors
- Refactored buildStyles() to use theme colors with fallbacks

Also adjusted the FgMuted color in the light theme to make model name and
execution time text more readable. Reordered the foreground color
progression to be more logical:

- FgMuted: charmtone.Iron (medium gray)
- FgHalfMuted: charmtone.BBQ (dark)
- FgSubtle: charmtone.Charcoal (medium dark)

This enables proper theme adaptation for Markdown content, ensuring consistent
visual presentation when switching between dark and light themes.

💘 Generated with Crush
Co-Authored-By: Crush <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants