Skip to content

feat: SW-2591 add size prop to Checkbox and RadioGroup - #213

Open
boramyi-ts wants to merge 3 commits into
mainfrom
SW-2591-checkbox-radio
Open

feat: SW-2591 add size prop to Checkbox and RadioGroup#213
boramyi-ts wants to merge 3 commits into
mainfrom
SW-2591-checkbox-radio

Conversation

@boramyi-ts

@boramyi-ts boramyi-ts commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

SW-2591 — add size prop to Checkbox & RadioGroup

Jira: SW-2591

The box-control slice of the input-size harmonization (after toggles #211 and the input family #212). Adds a size prop to the two box controls on a 14 / 16 / 18 / 20-style ladder — but, per the design call, default stays today's 16px so nothing changes for existing usage:

size box
xs 14
sm 16
default 16 (unchanged)
lg 20

xs and lg are opt-in; sm/default are both 16 today. The check icon and the radio dot scale with the box.

  • Checkbox — CVA size variant; check icon steps 12 / 14 / 16.
  • RadioGroup — accepts size and provides it via context so a group is sized once; RadioGroupItem can override per item. The inner dot scales 6 / 8 / 10.

xs · default · lg

xs (14) default (16) lg (20)
Checkbox
Radio (16, unchanged)

Tests

New ExtraSmall / Large stories assert the rendered box (checkbox & radio 14px xs, 20px lg) + data-size; the radio story also verifies the group size flows to items. Checkbox + RadioGroup suites pass (9/9) incl. axe; typecheck + lint clean. Zephyr ids via zephyr_sync.

No visual change to existing checkboxes/radios (default = 16). This completes the input controls for SW-2591 except the pill/track groups (Switch, Slider), which need internal track/thumb changes and are separate follow-ups.

🤖 Generated with Claude Code

Box scale (SW-2591): xs 14 / sm 16 / default 16 / lg 20. default keeps
today's 16px so existing checkboxes/radios are unchanged; xs and lg are
opt-in. The check icon and radio dot scale with the box.

RadioGroup accepts size and provides it via context so a group can be
sized once; RadioGroupItem can still override per item.

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

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
ts-lib-ui-kit-storybook Ready Ready Preview Sep 10, 2026 4:46pm UTC

Request Review

@unblocked unblocked Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No issues found

About Unblocked

Unblocked has been set up to automatically review your team's pull requests to identify genuine bugs and issues.

📖 Documentation — Learn more in our docs.

💬 Ask questions — Mention @unblocked to request a review or summary, or ask follow-up questions.

👍 Give feedback — React to comments with 👍 or 👎 to help us improve.

⚙️ Customize — Adjust settings in your preferences.

@github-actions

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 96.94% (🎯 83%)
🟰 ±0%
24749 / 25530
🟢 Statements 96.94% (🎯 83%)
🟰 ±0%
24749 / 25530
🟢 Functions 94.61% (🎯 74%)
🟰 ±0%
1177 / 1244
🟢 Branches 90.66% (🎯 81%)
⬇️ -0.10%
4972 / 5484
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/components/ui/checkbox.tsx 100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
100%
🟰 ±0%
src/components/ui/radio-group.tsx 100%
🟰 ±0%
50%
⬇️ -50.00%
100%
🟰 ±0%
100%
🟰 ±0%
Generated in workflow #1233 for commit 22e8e8f by the Vitest Coverage Report Action

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