Skip to content

[composite] Change focus to sync#4484

Draft
atomiks wants to merge 6 commits intomui:masterfrom
atomiks:codex/composite-sync-focus
Draft

[composite] Change focus to sync#4484
atomiks wants to merge 6 commits intomui:masterfrom
atomiks:codex/composite-sync-focus

Conversation

@atomiks
Copy link
Copy Markdown
Contributor

@atomiks atomiks commented Mar 30, 2026

Fixes #828

This removes Composite's extra focus microtask now that floating focus return is already async, and preserves radio group selection behavior when arrow navigation moves focus synchronously.

Changes

  • Focus the next composite item synchronously during keyboard navigation.
  • Keep radio group arrow navigation selecting the focused radio in the same event turn.
  • Remove obsolete flushMicrotasks() noise from the Composite root tests and add direct coverage for synchronous focus.

@atomiks atomiks added type: bug It doesn't behave as expected. component: composite labels Mar 30, 2026 — with ChatGPT Codex Connector
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 30, 2026

commit: 1f204fb

@mui-bot
Copy link
Copy Markdown

mui-bot commented Mar 30, 2026

Bundle size report

Bundle Parsed size Gzip size
@base-ui/react 🔺+262B(+0.06%) 🔺+54B(+0.04%)

Details of bundle changes


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

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 30, 2026

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit 1f204fb
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/69ca51af7bcbaf000877c6bb
😎 Deploy Preview https://deploy-preview-4484--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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 Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: composite PR: out-of-date The pull request has merge conflicts and can't be merged. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Composite] Change focus to sync

2 participants