Skip to content

Migration - Base UI based registry components#2424

Draft
SpicyPete wants to merge 8 commits intomasterfrom
pz/connect-exploration
Draft

Migration - Base UI based registry components#2424
SpicyPete wants to merge 8 commits intomasterfrom
pz/connect-exploration

Conversation

@SpicyPete
Copy link
Copy Markdown

@SpicyPete SpicyPete commented Apr 24, 2026

Summary

  • Registry recently moved from Radix-ui to base-ui
  • Bringing in the most recent base-ui based components
  • Shouldn't be any usage changes, a drop-in replacement

Todo

  • Integration tests passing
  • Waiting on some registry PR's which resolve some of the issues surfaced from this PR.

@SpicyPete SpicyPete requested review from a team, jvorcak and weeco and removed request for a team April 24, 2026 18:15
@SpicyPete SpicyPete marked this pull request as draft April 24, 2026 18:16
<Field data-invalid={fieldState.invalid}>
<FieldLabel required>Scopes</FieldLabel>
<MultiSelect
items={SCOPE_OPTIONS}
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.

How do item options get shared with multi select?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

{SCOPE_OPTIONS.map((option) => (
Just below this in the file we are mapping over SCOPE_OPTIONS again, so before we were passing them in twice, but only 1 was being used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants