Skip to content

Fix missing px of top style - #265

Open
liumingye wants to merge 2 commits into
willmcpo:masterfrom
liumingye:patch-1
Open

Fix missing px of top style#265
liumingye wants to merge 2 commits into
willmcpo:masterfrom
liumingye:patch-1

Conversation

@liumingye

Copy link
Copy Markdown

Comment thread src/bodyScrollLock.js Outdated
Co-authored-by: Alex Barnsley <8069294+alexbarnsley@users.noreply.github.com>
@riddla

riddla commented Jan 12, 2023

Copy link
Copy Markdown

This fix works for me on iOS 👍

@6180design 6180design left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOPE - Cache didn't update, this fix doesn't seem to work on an iPhone 11 with iOS 16.1.2.
Once the bodyscrolllock is called, the page jumps up top the top of the page.
The only thing working seems to be downgrading to Version 3.1.5

6180design

This comment was marked as duplicate.

@rick-liruixin

Copy link
Copy Markdown

They stopped the repairs. I had to do it myself, in the same way, with a new version of typeScript. And fix these problems for everyone to use. add react hooks、vue3 example

npm i body-scroll-lock-upgrade

repair log,Refer to the releases page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants