search

radix-ui / design-system

Design system that was used by @modulz. It’s now archived and superceded by Radix Themes. (See https://github.com/radix-ui/themes)
https://design-system.modulz-deploys.com/
MIT License
2.28k stars 177 forks source link

Opening Sheet sets the scroll position to top #365

Open haneenmahd opened 2 years ago

haneenmahd commented 2 years ago

On Clicking the open sheet button in the test suite page, the scroll position is switched to top.

colmtuite commented 2 years ago

Ca you describe your setup please.

haneenmahd commented 2 years ago

This is a piece of screenshot from your https://design-system.modulz-deploys.com test suite.

Screenshot 2022-01-16 at 8 26 33 AM

On Clicking the open sheet button, sometimes the window's scroll position is reset to the top

colmtuite commented 2 years ago

But what is your setup? What browser are you using? Which OS?

haneenmahd commented 2 years ago

MacOS, Safari

colmtuite commented 2 years ago

Thanks. It seems to be linking to the heading, not the window top. But it's weird, since there is no link on the trigger. Might be a Safari bug. Thanks for reporting.