Skip to content

Respect sectionTitleFontSize for schematic section labels#2366

Merged
seveibar merged 2 commits into
tscircuit:mainfrom
rushabhcodes:fix/schematic-section-title-font-size
May 31, 2026
Merged

Respect sectionTitleFontSize for schematic section labels#2366
seveibar merged 2 commits into
tscircuit:mainfrom
rushabhcodes:fix/schematic-section-title-font-size

Conversation

@rushabhcodes
Copy link
Copy Markdown
Contributor

What changed

  • Added sectionTitleFontSize support to schematic section labels.
  • The label now uses the provided size instead of always falling back to 0.18.
  • Added a focused test that checks the emitted schematic text font size and updated the snapshot.

Why

  • This lets callers control section title emphasis in schematic layouts without changing the default behavior for existing sections.

Validation

  • BUN_UPDATE_SNAPSHOTS=1 bun test tests/components/primitive-components/schematic-section.test.tsx

Copilot AI review requested due to automatic review settings May 31, 2026 17:26
@vercel
Copy link
Copy Markdown

vercel Bot commented May 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tscircuit-core-benchmarks Ready Ready Preview, Comment May 31, 2026 5:28pm

Request Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@seveibar seveibar merged commit d81af55 into tscircuit:main May 31, 2026
12 checks passed
@seveibar
Copy link
Copy Markdown
Contributor

Cc @Sang-it

@rushabhcodes rushabhcodes deleted the fix/schematic-section-title-font-size branch May 31, 2026 17:36
@tscircuitbot
Copy link
Copy Markdown
Contributor


Thank you for your contribution! 🎉

PR Rating: ⭐⭐
Impact: Minor

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


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.

4 participants