Skip to content

Migrate shadcn/ui from Radix to Base UI#115

Merged
overtrue merged 4 commits intorustfs:mainfrom
nakrovati:refactor/migrate-shadcn-to-baseui
May 10, 2026
Merged

Migrate shadcn/ui from Radix to Base UI#115
overtrue merged 4 commits intorustfs:mainfrom
nakrovati:refactor/migrate-shadcn-to-baseui

Conversation

@nakrovati
Copy link
Copy Markdown
Contributor

@nakrovati nakrovati commented May 9, 2026

Pull Request

Description

Migrate shadcn/ui from Radix to Base UI.

Reasons:

  1. Radix is no longer being developed.
  2. Base UI is recommended as the default component library. Incidentally, I found the Base UI API to be superior, for example, in terms of handling the closing of Dialog windows and handling the indeterminate state of elements.
  3. In the current configuration, Base UI and Radix were used simultaneously, which is certainly allowed, but having two libraries increases the cognitive load on developers and AI agents when writing code, since they have to think about the APIs of two component libraries at once; moreover, during migration, we can reduce the bundle size by removing Radix (and upon removing radix-ui, pnpm-lock.yaml was reduced by 1,250 lines by removing @radix-ui/* primitives).

Changes in the PR:

  1. Migrate shadcn/ui from Radix to Base UI.
  2. Fix TypeScript errors (due to the migration) and a few minor ESLint errors.
  3. Update pnpm-lock.yaml for PNPM 11.
  4. Add .editorconfig.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvements
  • Security fix

Testing

  • Unit tests added/updated
  • Manual testing completed
pnpm test:run

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • TypeScript types are properly defined
  • All commit messages are in English (Conventional Commits)
  • All existing tests pass
  • No new dependencies added, or they are justified

Related Issues

Closes #

Screenshots (if applicable)

Additional Notes

@nakrovati nakrovati changed the title Refactor/migrate shadcn to baseui Migrate shadcn/ui from Radix to Base UI May 9, 2026
@overtrue overtrue merged commit bb3422d into rustfs:main May 10, 2026
11 checks passed
@overtrue
Copy link
Copy Markdown
Collaborator

Thanks!

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