[base-ui][useSelect] Keyboard operations on the activation button not working for HTML element different than "button"

[smellai]

Steps to reproduce

Link to live example: https://codesandbox.io/p/sandbox/boring-dawn-c5rxjv?file=%2Fsrc%2FDemo.tsx

Steps:

1. focus combobox with keyboard
2. try to activate it with spacebar or Enter
3. dropdown not showing up
4. change "div" to "button" in line 42 of Demo.tsx
5. it works as expected

Current behavior

pressing spacebar or Enter not activating the dropdown for the combobox

Expected behavior

pressing spacebar or Enter should open the combobox overlay

Context

I don't think "button" html element should be necessary as the role is overridden to "combobox" in any case.
It was working fine in @mui/base 5.0.0-beta.25

Your environment

npx @mui/envinfo

 System:
    OS: macOS 14.2.1
  Binaries:
    Node: 18.18.0
    npm: 10.2.4 
  Browsers:
    Chrome: 120.0.6099.199
    Edge: Not Found
    Safari: 17.2.1
  npmPackages:
    @mui/base: 5.0.0-beta.30 => 5.0.0-beta.30 
    @mui/core-downloads-tracker:  5.15.3 
    @mui/material: 5.15.3 => 5.15.3 
    @mui/private-theming:  5.15.3 
    @mui/styled-engine:  5.15.3 
    @mui/system:  5.15.3 
    @mui/types:  7.2.12 
    @mui/utils:  5.15.3 
    @types/react: 17.0.2 => 17.0.2 
    react: 17.0.2 => 17.0.2 
    react-dom: 17.0.2 => 17.0.2 
    typescript: 4.9.5 => 4.9.5 

Search keywords: keyboard select combobox

[smellai]

The issue persists in "@mui/base": "5.0.0-beta.31"

[ZeeshanTamboli]

It stopped working since version 5.0.0-beta.29. It might be related to #39675 which was released in 5.0.0-beta.29. @DiegoAndai may have better insight on this issue.

[DiegoAndai]

Thanks for the report @smellai! I opened a PR with the fix.

[smellai]

This issue is fixed (thanks!) but in the latest versions aria-activedescendant attribute was removed from comboboxes and tabindex="-1" added to the list elements. Is this expected? is the change documented somewhere?

[DiegoAndai]

in the latest versions aria-activedescendant attribute was removed from comboboxes and tabindex="-1" added to the list elements.

It is expected: https://github.com/mui/material-ui/blob/master/CHANGELOG.md#muibase500-beta29. The change should be seamless. Has it impacted you negatively? Let me know and I can help with the migration.

[smellai]

Yes, sorry I finally found the time to debug, will open a new issue