-
Notifications
You must be signed in to change notification settings - Fork 97
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
How to make the modal overflow on vfm--fixed element correctly? #337
Comments
I implemented an long content scroll modal example like bootstrap: |
Thank you! Would you add this example to the documentaion? |
Yeah I'll consider to add a new use case inside of https://vue-final-modal.org/use-cases/playground/. |
The example was added to the use-cases page: |
Hi Hunter, Thank you for your previous update. I have another question related to user experience. I noticed that in the example you updated, if the mouse is clicked inside the modal and then dragged to the gray background before being released, the modal will be closed. However, it seems that the original |
@johnson0903 Thanks for your feedback. |
@johnson0903 The issue was fixed in version v4.1.3 |
@hunterliu1003 I am wondering if it's because the click.self directive written in the following code block of the example is causing the issue: |
Version
vue-final-modal: 4.0.7
vue: 3.2.37
Reproduction Link
https://codesandbox.io/p/sandbox/vue-final-modal-scroll-9vkef0
What is Expected?
I want to make the modal content scrollable and the scrollbar should appear on vfm--fixed element, such as Bootstrap Scrolling long content Modal Example.
What is actually happening?
I add
overflow-y: auto
on vfm--fixed element to make the scrollbar appear on the right side , but it caused 2 problems.First, the overlay height is not equal to the long content. Maybe can make it appear outside the vfm container to solve this problem.
Second, when release the mouse after drag or click on the scroll bar, the modal was closed.
Any suggestion will be helpful !
The text was updated successfully, but these errors were encountered: