marigold-ui/marigold

Can we move `FocusScope` into `<Popover>`? (used in Select and Menu)

sarahgm opened this issue · 1 comments

sarahgm commented

<FocusScope> is used in <Modal>, <Select>and <Menu>
<FocusScope> comes from react-aria

what <FocusScope> does:

  • restore focus back to the trigger when menu is closed
sarahgm commented

Question: Why? Because Popover is not used in <Menu>. It's just used in <Select>. And whats with the <Modal>