Skip to content

fix(fork,timeline): use explicit sans-serif font stack - #482

Merged
Nagi-ovo merged 2 commits into
mainfrom
Nagi-ovo/issue-456-investigate
Mar 12, 2026
Merged

fix(fork,timeline): use explicit sans-serif font stack#482
Nagi-ovo merged 2 commits into
mainfrom
Nagi-ovo/issue-456-investigate

Conversation

@Nagi-ovo

@Nagi-ovo Nagi-ovo commented Mar 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fork UI (src/pages/content/fork/index.ts): Replaced font-family: inherit with explicit sans-serif stack on all 5 elements (button, confirm dialog, confirm buttons, indicator, delete button). inherit falls back to serif when parent containers lack font declarations.
  • Timeline preview panel (public/contentStyle.css): Added font-family to .timeline-preview-panel (was completely missing) and font-family: inherit to its search input (inputs don't auto-inherit).

Fixes #456

Test plan

  • Open a conversation with fork branches, verify fork button and indicator text renders in sans-serif
  • Click fork button, verify confirmation dialog uses sans-serif
  • Open timeline preview panel (history list), verify all text uses sans-serif
  • Test in Brave/Chrome/Firefox

🤖 Generated with Claude Code


Open with Devin

…erit

Fixes #456. Fork buttons/dialogs used `font-family: inherit` which falls
back to serif in browsers where the parent container lacks an explicit
font declaration. Timeline preview panel had no font-family at all.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

Use 'Google Sans', Roboto, Arial, sans-serif — the same stack used
throughout the codebase — instead of system font stack. This ensures
proper rendering for CJK, Arabic, and other non-Latin scripts since
Google Sans has broad multilingual coverage.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Nagi-ovo
Nagi-ovo merged commit 29936db into main Mar 12, 2026
1 check 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.

[Bug] Default serif font used in "Fork" and "History" pop-ups

1 participant