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

Relative positioning - popovers do not scroll with content #144

Closed
bobmash opened this issue Feb 10, 2016 · 3 comments
Closed

Relative positioning - popovers do not scroll with content #144

bobmash opened this issue Feb 10, 2016 · 3 comments

Comments

@bobmash
Copy link

@bobmash bobmash commented Feb 10, 2016

Hi,

Great plugin, but I have an issue with using the popover within a scrolled div container - they do not scroll relatively with the content / trigger element.

As the popovers are absolute positioned, they do not scroll with the content when they are open, they remain static; is there a way of achieving this perhaps through relative positioning etc.?

Thanks,
Liam

@phc-itt

This comment has been minimized.

Copy link

@phc-itt phc-itt commented Feb 19, 2016

Hi!
Currently I have problems like you to use them on the bootstrap's modal.
When the bootstrap's modal is opened. they will roll off document.body; This has led to the modal popup will not be open on the modal scroll is rolled.

@jeremyVignelles

This comment has been minimized.

Copy link
Contributor

@jeremyVignelles jeremyVignelles commented Mar 24, 2016

Hi,
Same problem here. I hope to have time to fix and publish a PR.
My guess is that this line is causing the issue:

https://github.com/sandywalker/webui-popover/blob/master/src/jquery.webui-popover.js#L348

I think there should be a "container" option to let you specify the scrolling content.

@jeremyVignelles

This comment has been minimized.

Copy link
Contributor

@jeremyVignelles jeremyVignelles commented Mar 24, 2016

Added a JSFiddle to reproduce my issue : https://jsfiddle.net/1x21rj9e/
And the fixed version : https://jsfiddle.net/1x21rj9e/1/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.