[useScrollLock] overflow-y: scroll support no longer working

[oliviertassinari]

Steps to reproduce

Steps:

1. Go to https://master--base-ui.netlify.app/components/react-dialog/

Screen.Recording.2024-09-14.at.16.36.39.mov

Current behavior

Doesn't restore the value, it's overridden, so it introduces a layout shift.

Expected behavior

Correctly restore the previous values. For example https://github.com/mui/material-ui/blob/012c2e899b9a8977bb326746f43ef19a5f4f01a7/packages/mui-material/src/Modal/ModalManager.ts#L143 from https://mui.com/base-ui/react-modal/, it works.

Context

The support for overflow-y: scroll is needed to solve bugs like mui/material-ui#40749, until we can rely on #79.

Your environment

npx @mui/envinfo

  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: -

[vladmoroz]

Fixed in #604

[github-actions]

This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.