Skip to content

chore: deprecate old card#562

Merged
adrienzheng-cb merged 3 commits intomasterfrom
adrien/deprecate-old-card
Mar 30, 2026
Merged

chore: deprecate old card#562
adrienzheng-cb merged 3 commits intomasterfrom
adrien/deprecate-old-card

Conversation

@adrienzheng-cb
Copy link
Copy Markdown
Contributor

@adrienzheng-cb adrienzheng-cb commented Mar 30, 2026

  1. deprecated Card and related components and types
  2. added deprecate-cds-api skill

What changed? Why?

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

1. derpecated Card and related components and types
2. added derepcate-cds-api-skill
@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

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


/**
* @deprecated Use ContentCardHeaderProps instead. This will be removed in a future major release.
* @deprecationExpectedRemoval v9
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.

I think these should be v10

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.

probably need to tweak the instructions in the SKILL

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

should we say two major versions from now in that case

Copy link
Copy Markdown
Contributor Author

@adrienzheng-cb adrienzheng-cb Mar 30, 2026

Choose a reason for hiding this comment

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

updated to v10. kept the major version + 1 rule but agent will now confirm in case we are about to release a new version like now.

@adrienzheng-cb adrienzheng-cb marked this pull request as ready for review March 30, 2026 19:00
argument-hint: '<SymbolName or path> — replacement (e.g. Tabs) — [optional notes]'
---

# Deprecate CDS public API
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.

@adrienzheng-cb is it too early to begin adding a step for creating a migrator?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'll add that once I add the transform creation rule.

@github-actions
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@hcopp hcopp left a comment

Choose a reason for hiding this comment

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

Looks good

@adrienzheng-cb adrienzheng-cb merged commit ebc706f into master Mar 30, 2026
26 of 28 checks passed
@adrienzheng-cb adrienzheng-cb deleted the adrien/deprecate-old-card branch March 30, 2026 19:21
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

Development

Successfully merging this pull request may close these issues.

4 participants