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

Switched the scrolling behavior to center the tooltip vertically #132

Closed
wants to merge 3 commits into from

Conversation

jlovison
Copy link

Changed the scroll behavior to center the popover in the browser window, to minimize issues with popover placement and/or a higher z-index persistent navbar.

See issues #114 and #125

@jlovison
Copy link
Author

@LostCrew I cut down on the variable use for half the window height, but not having much experience in coffeescript (and not having the compilation tools on my development machine), I didn't want to reduce down to a single line, as that animate call is technically a dictionary of CSS statements, and I could anticipate potential hiccups with the compiled JS.

@sorich87
Copy link
Owner

@kromem, install node.js (http://nodejs.org/). Run npm install -d command in the library directory to install the compilation tools and other dependencies and grunt build to compile.

@ghost ghost assigned LostCrew Aug 22, 2013
@LostCrew
Copy link
Contributor

LostCrew commented Sep 1, 2013

@sorich87 @kromem integrated in v0.6.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants