Skip to content

[select] Simplify composite item value registry#5267

Closed
atomiks wants to merge 3 commits into
mui:masterfrom
atomiks:codex/simplify-select-tabs-registries
Closed

[select] Simplify composite item value registry#5267
atomiks wants to merge 3 commits into
mui:masterfrom
atomiks:codex/simplify-select-tabs-registries

Conversation

@atomiks

@atomiks atomiks commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Use Composite item metadata as the source of truth for Select item values.

Changes

  • Remove the per-item layout effect that writes into the Select value registry.
  • Derive the value array atomically from committed Composite map updates.
  • Preserve registration-order ownership for duplicate Tabs panels and add regression coverage.

Bundle size

Fresh sequential builds against master:

  • @base-ui/react: -21 B parsed, -8 B gzip
  • @base-ui/react/select: -21 B parsed, -11 B gzip
  • @base-ui/react/tabs: unchanged

@atomiks atomiks added type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. component: select Changes related to the select component. component: tabs Changes related to the tabs component. labels Jul 19, 2026
@pkg-pr-new

pkg-pr-new Bot commented Jul 19, 2026

Copy link
Copy Markdown

commit: ee9ef91

@code-infra-dashboard

code-infra-dashboard Bot commented Jul 19, 2026

Copy link
Copy Markdown

Bundle size

Bundle Parsed size Gzip size
@base-ui/react ▼-21B(0.00%) ▼-5B(0.00%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

@netlify

netlify Bot commented Jul 19, 2026

Copy link
Copy Markdown

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit ee9ef91
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/6a5e27db1a00590008bc8b13
😎 Deploy Preview https://deploy-preview-5267--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jul 20, 2026
…lect-tabs-registries

# Conflicts:
#	packages/react/src/select/item/SelectItem.tsx
#	packages/react/src/select/positioner/SelectPositioner.tsx
#	packages/react/src/tabs/panel/TabsPanel.tsx
#	packages/react/src/tabs/root/TabsRoot.tsx
#	packages/react/src/tabs/root/TabsRootContext.ts
@github-actions github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jul 20, 2026
@atomiks atomiks changed the title [select][tabs] Simplify composite item registries [select] Simplify composite item value registry Jul 20, 2026
@atomiks atomiks closed this Jul 20, 2026
@atomiks
atomiks deleted the codex/simplify-select-tabs-registries branch July 20, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: select Changes related to the select component. component: tabs Changes related to the tabs component. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant