fix: added missing CSS unit when positioning the window offset to `top`

willmcpo / body-scroll-lock

Body scroll locking that just works with everything 😏

MIT License

4.06k stars 340 forks source link

fix: added missing CSS unit when positioning the window offset to `top` #253

Open emeraldsanto opened 2 years ago

emeraldsanto commented 2 years ago

Description

This PR simply adds the missing px unit to the calculation of how much to offset from top. Without this, certain browsers will simply interpret the value as 0 which makes the page scroll to the top.

topolanekmartin commented 2 years ago

On mobile iOS it does not attach top property on body element. This causes the scroll is not returned back to its previous position after scroll is enabled.