base-ui
base-ui copied to clipboard
[dialog][popover] Fix focus restoration when focused element is hidden with CSS
- [x] I have followed (at least) the PR section of the contributing guide.
Fixes #3299. While conditional rendering was handled via .isConnected, display: none (e.g. React.Activity) was not.
https://stackblitz.com/edit/eldph4db-edlqv6im
pnpm add https://pkg.pr.new/mui/base-ui/@base-ui-components/react@3313
pnpm add https://pkg.pr.new/mui/base-ui/@base-ui-components/utils@3313
commit: 5d4e415
Bundle size report
| Bundle | Parsed size | Gzip size |
|---|---|---|
| @base-ui-components/react | 🔺+114B(+0.03%) | 🔺+18B(+0.01%) |
Check out the code infra dashboard for more information about this PR.
Deploy Preview for base-ui ready!
| Name | Link |
|---|---|
| Latest commit | 5d4e4158f3ce64ccf49b650ca73ec12ec03bab78 |
| Latest deploy log | https://app.netlify.com/projects/base-ui/deploys/69281a569d0f5d000861383a |
| Deploy Preview | https://deploy-preview-3313--base-ui.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify project configuration.