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.

Sign up for GitHub

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

Jump to bottom

[popup] data-content|html not properly aligned in modal when body of page have scrollbar #1899

Open
dutrieux opened this issue Feb 26, 2021 · 3 comments
Open

[popup] data-content|html not properly aligned in modal when body of page have scrollbar #1899

dutrieux opened this issue Feb 26, 2021 · 3 comments
Labels
state/has-pr An issue which has a related PR open type/bug Any issue which is a bug or PR which fixes a bug
Milestone
2.9.x

Comments

@dutrieux
Copy link
Contributor

Bug Report

The popup (data-content or data-html) is not properly aligned when in modal and the body of page have previously scrollbar.

Testcase

https://jsfiddle.net/dutrieux/npjsc5d8/

Screenshot (if possible)

popup

Version

2.8.7
@dutrieux dutrieux added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels Feb 26, 2021
@lubber-de lubber-de removed the state/awaiting-triage Any issues or pull requests which haven't yet been triaged label Feb 26, 2021
@ivangomes
Copy link

Testcase presented no problem for me.

macOS Big Sur 13.3
Google Chrome:
Version 89.0.4389.72 (Official Build) (x86_64)

@dutrieux
Copy link
Contributor Author

dutrieux commented Mar 3, 2021

Indeed the problem does not exist on mac ! On mac I try too Firefox and Safari.

But the problem exist on Windows 10 (Chrome & Firefox) and Ubuntu (Chromium & Firefox)

@lubber-de lubber-de mentioned this issue Mar 19, 2021
Draft
@lubber-de
Copy link
Member

Fixed by #1918

@lubber-de lubber-de added state/has-pr An issue which has a related PR open and removed state/awaiting-investigation Anything which needs more investigation labels Mar 19, 2021
@lubber-de lubber-de added this to the 2.8.x milestone Mar 19, 2021
@lubber-de lubber-de modified the milestones: 2.8.x, 2.9.x Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
state/has-pr An issue which has a related PR open type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Milestone
2.9.x
Development

No branches or pull requests
3 participants
@dutrieux @ivangomes @lubber-de