fix(web): restore bold weight in rendered Markdown#25694

open2
main
fix/issue-25396
opened2d agobyJJulius Marminge
View on GitHubTwo-way sync enabledPR last imported 2d ago

Problem

Sending Regular text and **bold text** reduced bold from the rich text composer's weight 700 to weight 600 in rendered messages. The global Markdown rule also applied with Settings → Beta → New design disabled, making emphasis difficult to distinguish in some fonts.

Change

Restore .chat-markdown strong to weight 700 in both designs. This also restores bold in other surfaces using the shared Markdown renderer.

Scope and approval

Fixes https://github.com/pingdotgg/t3code/issues/25395 GitCafe: https://git.cafe/pingdotgg/t3code/issues/25396

Triaged bug via the automated issue flow; GitCafe mirror #25396.

Verification

  • Reproduced in the real web dev app: user and assistant message bold computed to 600 with New design on and off, while composer bold computed to 700. The fixed messages compute to 700.
  • vp run check:changed passed formatting and affected typechecks after merging main. No tests added for this purely visual change.
  • Reproduction transcript.
  • Native Windows desktop font rendering and shell could not be checked on this Linux VM. Web and desktop share the changed stylesheet; native mobile uses a separate renderer and is unaffected.

Evidence

FixtureBeforeAfterDiff0DiffT
C1 · 1440×900 · light · new on · InterBefore
0604547a3a
After
5f0a0cf558
Diff0
0604547a3a → 5f0a0cf558
DiffT
0604547a3a → 5f0a0cf558
C2 · 390×844 · dark · new on · InterBefore
0604547a3a
After
5f0a0cf558
Diff0
0604547a3a → 5f0a0cf558
DiffT
0604547a3a → 5f0a0cf558
C3 · 1440×900 · dark · new off · InterBefore
0604547a3a
After
5f0a0cf558
Diff0
0604547a3a → 5f0a0cf558
DiffT
0604547a3a → 5f0a0cf558
C4 · 390×844 · light · new off · InterBefore
0604547a3a
After
5f0a0cf558
Diff0
0604547a3a → 5f0a0cf558
DiffT
0604547a3a → 5f0a0cf558
C5 · 1440×900 · light · new on · Roboto FlexBefore
0604547a3a
After
5f0a0cf558
Diff0
0604547a3a → 5f0a0cf558
DiffT
0604547a3a → 5f0a0cf558
P1 · 1440×900 · light · new off · workspace Markdown previewBefore
0604547a3a
After
5f0a0cf558
Diff0
0604547a3a → 5f0a0cf558
DiffT
0604547a3a → 5f0a0cf558

Diff0 shows every changed RGB pixel; DiffT uses a 16/255 maximum-channel threshold. Both amplify the absolute difference ×4. Static typography is shown with settled stills; paired stability frames have zero differing pixels.

Linux Chrome 156, DPR 2, 100% scale; desktop CSS viewport 1440×900 is captured by the preview service at 1280×800, narrow CSS viewport 390×844 at 780×1688. Inter Variable and Roboto Flex were verified loaded. CSS motion suppression was used because the preview tools do not expose reduced-motion emulation.

Visible differences: bold glyphs are heavier and slightly wider; the user bubble and its regular prefix move left by 1.5 CSS pixels; the preview table column shifts by 5.08 CSS pixels as bold-cell width changes. No wrapping or clipping.

Full evidence and reproducible fixture/driver · Fixture matrix.

Independent proof audit: all six pairs sufficient; C3 rebuilt at the exact head with zero pixel difference. Both isolated code reviewers returned MERGE.

Implemented by Claude Opus 5.5 through the Claude harness; triaged, verified, and reviewed by GPT-6.1-Sol through Codex and Claude Opus 5.5 through Claude in T3 Code.

Checks

22 successful · 9 skipped · 1 neutral
All checks passed
5f0a0cf

Added 2 commits

  • fix(web): bold text in messages renders at weight 700, matching the composer074ef89
  • Merge remote-tracking branch 'origin/main' into fix/issue-253965f0a0cf
Synced from GitHub

Approvability

Verdict: Needs Human Approval

Macroscope's review found this PR requires human approval — This focused CSS fix restores bold Markdown text to weight 700 across shared rendered-Markdown surfaces, changing the product’s default typography and potentially text layout. The visual behavior change should receive human review.

You can add or adjust custom eligibility rules. Learn more.

Synced from GitHub

Review in Change Stack →

No actionable comments were generated in the recent review.

External image Recent review info
External image Run configuration
External image Commits

Reviewing files that changed from the base of the PR and between d73fd1279a9cbf3ef4677b88b4ad82f0a55583c7 and 5f0a0cf55824f0ecff00a65216422a7d3a937cd2.

External image Files selected for processing: 1
  • apps/web/src/index.css

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 3 remain after this review.

Walkthrough

The .chat-markdown strong rule now uses font weight 700 instead of 600.

Changes: Rendered Markdown bold

Layer / File(s)Summary
Set rendered bold weight
apps/web/src/index.css
The .chat-markdown strong rule changes from font weight 600 to 700.

Priority: External image Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix · Severity of issue fixed: Low

Possibly related PRs

Suggested reviewers: t3dotgg, macroscopeapp

Merge Risk: External image Minimal · up to 5f0a0

No actionable merge-blocking risk remains.

Pre-merge checks: Passed 4
Check nameStatusExplanation
Title checkPassedThe title is concise, follows the repository's conventional format, and clearly identifies the main change: restoring bold weight in rendered Markdown.
Description checkPassedThe description includes all required sections. It explains the problem, change, approval basis, verification steps, limitations, and UI evidence with before/after screenshots.
Linked Issues checkPassedIssue #25395 requires clearly distinguishable Markdown bold with consistent emphasis between the composer and rendered messages, including when New design is disabled. The PR changes `.chat-markdown s…
Out of Scope Changes checkPassedThe whole-PR diff contains one CSS declaration change in apps/web/src/index.css. The change supports issue #25395 by restoring Markdown bold weight. No unrelated code or behavior change is present.
Full details: Linked Issues check

Explanation

Issue #25395 requires clearly distinguishable Markdown bold with consistent emphasis between the composer and rendered messages, including when New design is disabled. The PR changes .chat-markdown strong in apps/web/src/index.css from font-weight: 600 to font-weight: 700. This directly restores the requested weight for the shared Markdown renderer. The PR reports successful formatting and affected typechecks. No additional automated test is required by the issue for this visual CSS change.

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

Acknowledged: this changes the shared rendered-Markdown bold weight from 600 to 700. The Verification section includes six before/after pairs and an independent proof audit; wider bold glyphs shift the user bubble by 1.5 CSS pixels and the preview table column by 5.08 CSS pixels. Leaving the human approval decision to the maintainer.