Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The no-scroll class is not given correctly #580

Open
1 of 3 tasks
wayheming opened this issue Sep 4, 2023 · 1 comment · May be fixed by #581
Open
1 of 3 tasks

The no-scroll class is not given correctly #580

wayheming opened this issue Sep 4, 2023 · 1 comment · May be fixed by #581

Comments

@wayheming
Copy link

jquery-confirm version:
v3.3.4

I'm submitting a ... (check one with "x")

  • bug report
  • feature request
  • support request

Current behavior:

The no-scroll class is not given correctly due to the fact that the height is incorrectly calculated. We need to use the .outerHeight() function instead of the .height() function as we do in the _updateContentMaxHeight function.
https://a.supportally.com/i/urzsk5

With .height(): https://a.supportally.com/v/iwjyIm (I can't scroll even though the content is there)
With .outerHeight(): https://a.supportally.com/v/Ay4WIg

@seredniy
Copy link

seredniy commented Sep 4, 2023

I have this issue too

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 a pull request may close this issue.

2 participants