Skip to content

feat: added cds skills for distribution#561

Merged
haoruikun-cb merged 1 commit intomasterfrom
cds-skills-merge
Mar 30, 2026
Merged

feat: added cds skills for distribution#561
haoruikun-cb merged 1 commit intomasterfrom
cds-skills-merge

Conversation

@haoruikun-cb
Copy link
Copy Markdown
Contributor

What changed? Why?

Added cds-code and cds-design-to-code skill for distribution. Adapted from cds-playground.

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
old screenshot new screenshot

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

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 30, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/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

⚠️ Ignored Reviews (3)

Reviewer Reason
cb-ekuersch Bound to another issue (#554)
cb-ekuersch Bound to another issue (#554)
cb-ekuersch Review missing MFA

CODEOWNERS

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

@haoruikun-cb haoruikun-cb marked this pull request as ready for review March 30, 2026 17:29
@github-actions
Copy link
Copy Markdown
Contributor

@haoruikun-cb haoruikun-cb merged commit daaf755 into master Mar 30, 2026
26 checks passed
@haoruikun-cb haoruikun-cb deleted the cds-skills-merge branch March 30, 2026 17:56
adrienzheng-cb added a commit that referenced this pull request Mar 30, 2026
* feat: add indeterminate ProgressCircle (#501)

* feat: add indeterminate ProgressCircle, unify progress size and float-label math

- Add indeterminate prop to ProgressCircle (web & mobile): spinning state with
  configurable weight, default stroke ratio 0.11; animate full SVG on mobile,
  CSS keyframes on web; hide default content when indeterminate.
- Add getProgressSize(weight) in common and deprecate useProgressSize; use
  getProgressSize in ProgressBar and ProgressCircle on both platforms.
- Simplify ProgressBarWithFloatLabel (web & mobile): remove usePreviousValues,
  useIsoEffect, and imperative animation; use shared getEndTranslateX so float
  label trailing edge follows fill end (containerWidth * progress - textWidth);
  web uses useMotionProps + MotionBox, mobile animates translateX to target.
- ProgressBar/ProgressCircle: progress optional with default 0; add originX/
  originY in getProgressCircleParams; web ProgressCircle uses pathLength=1.
- Deprecate Spinner (web & mobile) in favor of indeterminate ProgressCircle.
- ProgressBar tests: update float-label position expectation (80) and accept
  transform none/translateX(0) for zero progress; iconSvgMap regenerated.

* update test and deprecation message

* add more button stories

* a11y fix

* remove 0.11

* add progressCircleSize prop

* update changelogs and package versions

* add progressCircleSize prop to IconButton

* fix removal version

* chore: deprecate CardGroup (#560)

* chore: deprecate CardGroup

* Update changelog

* feat: added cds skills for distribution (#561)

* chore: deprecate old card (#562)

* chore: deprecate old card

1. derpecated Card and related components and types
2. added derepcate-cds-api-skill

* update changelogs

* tweaks

---------

Co-authored-by: Hunter Copp <huntercolecopp@gmail.com>
Co-authored-by: Harry <ruikun.hao@coinbase.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants