Skip to content

CDS-1665: Refresh checkbox component with 4px corner radius#506

Closed
samcartersmith wants to merge 4 commits intocoinbase:masterfrom
samcartersmith:pr/477-checkbox-fix
Closed

CDS-1665: Refresh checkbox component with 4px corner radius#506
samcartersmith wants to merge 4 commits intocoinbase:masterfrom
samcartersmith:pr/477-checkbox-fix

Conversation

@samcartersmith
Copy link
Copy Markdown
Contributor

  • Set default borderRadius to 100 (4px) for Checkbox on web and mobile
  • Consumers can still override via borderRadius prop

Made-with: Cursor

What changed? Why?

Added 100 border radius to checkbox to more align with CDS styling and request from consumer.

Root cause (required for bugfixes)

UI changes

iOS Old iOS New
old screenshot new screenshot
Android Old Android New
old screenshot new screenshot
Web Old Web New
Screenshot 2026-03-13 at 9 21 05 AM Screenshot 2026-03-13 at 9 25 30 AM

Testing

How has it been tested?

  • Unit tests
  • Interaction tests
  • Pseudo State tests
  • Manual - Web
  • Manual - Android (Emulator / Device)
  • Manual - iOS (Emulator / Device)

Testing instructions

  1. Open CDS docs site or storybook
  2. Open Checkbox
  3. Observe visual change

Illustrations/Icons Checklist

Required if this PR changes files under packages/illustrations/** or packages/icons/**

  • verified visreg changes with Terran (include link to visreg run/approval)
  • all illustration/icons names have been reviewed by Dom and/or Terran

Change management

type=routine
risk=low
impact=sev5

automerge=false

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Mar 16, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 1
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1
CODEOWNERS 🟡 See below

🟡 CODEOWNERS

Code Owner Status Calculation
ui-systems-eng-team 🟡 0/1
Denominator calculation
Additional CODEOWNERS Requirement
Show calculation
Sum 0
0
From CODEOWNERS 1
Sum 1

#### 📘 Misc

- Internal: update figma code connect config and some mapping files. [[#304](https://github.com/coinbase/cds/pull/304)]

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.

@samcartersmith can you undo this removal?

#### 🐞 Fixes

- Internal: update figma code connect config and some mapping files. [[#304](https://github.com/coinbase/cds/pull/304)]
- Refresh Checkbox component with 4px corner radius. [[#477](https://github.com/coinbase/cds/pull/477)]
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.

Same here with it changing past changelog

@hcopp
Copy link
Copy Markdown
Contributor

hcopp commented Mar 17, 2026

Closing in favor of #509

@hcopp hcopp closed this Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants