Skip to content

Fix chat demo role styles so markdown bold is visible#241

Merged
ryanoneill merged 1 commit intomainfrom
fix/chat-demo-role-styles
Mar 10, 2026
Merged

Fix chat demo role styles so markdown bold is visible#241
ryanoneill merged 1 commit intomainfrom
fix/chat-demo-role-styles

Conversation

@ryanoneill
Copy link
Owner

Summary

  • Remove BOLD modifier from User and Assistant role styles in the chat markdown demo
  • The role style sets the base content style for message text. The header (username) line adds BOLD separately. With BOLD already on the base style, markdown **bold** was visually identical to plain text — no way to tell bold from non-bold.

Test plan

  • cargo run --example chat_markdown_demo --features "compound-components,markdown" — type **Hello** how are you doing? and verify "Hello" appears bold while the rest does not

🤖 Generated with Claude Code

Remove BOLD modifier from User and Assistant role styles. The role
style is used as the base content style — the header (username) line
already adds BOLD separately. With BOLD on the base style, markdown
**bold** text was visually identical to plain text.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.71%. Comparing base (5bf5c49) to head (16c4240).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #241   +/-   ##
=======================================
  Coverage   91.71%   91.71%           
=======================================
  Files         148      148           
  Lines       13052    13052           
=======================================
  Hits        11970    11970           
  Misses       1082     1082           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ryanoneill ryanoneill merged commit 11fd519 into main Mar 10, 2026
17 checks passed
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