base-ui icon indicating copy to clipboard operation
base-ui copied to clipboard

[core] Refactor popup component style hooks to `data-popup-open`, `data-open`

Open atomiks opened this issue 1 year ago • 1 comments

Closes https://github.com/mui/base-ui/issues/717

Style hooks refer to themselves or closely related components:

  • Trigger receives data-popup-open
  • Backdrop, Popup, Positioner and child components receive data-open
  • Menu data-radioitem -> data-radio-item-{checked,unchecked}
  • Menu data-checkboxitem -> data-checkbox-item-{checked,unchecked}
  • Components that don't need to read open remove the state
  • React.useMemo the ownerState for missing components
  • Make data-entering/data-exiting consistent

atomiks avatar Oct 10 '24 05:10 atomiks

Netlify deploy preview

https://deploy-preview-731--base-ui.netlify.app/

Generated by :no_entry_sign: dangerJS against 7acdd950b805f1bd11dc2dc190631f622ce68b32

mui-bot avatar Oct 10 '24 05:10 mui-bot