chore: 🤖 restore flyout content focus handlers#924
Conversation
🦋 Changeset detectedLatest commit: d6d628a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Claude finished @punkbit's task in 58s —— View job Code Review✅ Looks great! No issues found — nice work! Summary: This PR correctly restores A quick check confirms all other Radix ♿ Accessibility — positive impact: restoring these hooks allows consumers to prevent or customize the default focus-trap behavior on open/close, which is important for embedded/inline flyout use cases where auto-focusing may conflict with the surrounding context. |
📚 Storybook Preview Deployed✅ Preview URL: https://click-4gi8ljjl3-clickhouse.vercel.app Built from commit: |
Why?
Restore focus handlers in flyout's props type definition. These focus management props were inadvertently removed during the decoupling from Radix UI types, e.g. needs a few more iterations/passes to get right. They are now explicitly added back to provide parity with the
Dialogcomponent and support common accessibility use cases.How?
Preview?
N/A