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

fixed tooltip position for charts placed on lower part of the page #2164

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

karjac14
Copy link

as per issue #2163, fixed by @RedScourge. This will fix the tooltip position of charts placed on lower part of the page where scrolling is needed.

as per issue novus#2163, fixed by @RedScourge. This will fix the tooltip position of charts placed on lower part of the page where scrolling is needed.
@fsbraun
Copy link

fsbraun commented Jul 16, 2019

Any chance this gets merged? Currently all tooltips are positioned incorrectly if the page is not scrolled to the top.

@karjac14
Copy link
Author

Any chance this gets merged? Currently all tooltips are positioned incorrectly if the page is not scrolled to the top.

unfortunately, most (if not all) of the PR are falling the automated CI test, because of an error on an existing code not on the actual proposed changes.

@fsbraun
Copy link

fsbraun commented Jul 18, 2019

Thanks @karjac14 for the reply! It seems that #2192 fixes the CI test. Maybe merge both??

@liquidpele
Copy link
Contributor

I'm not on this project anymore so I can't merge anything, the novus guys removed me. I created the nvd3/nvd3 repo though as a separate work to get it working with newer versions of D3. Feel free to help out :)

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.

3 participants