-
Notifications
You must be signed in to change notification settings - Fork 505
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
Horizontal line is not positioned correctly #87
Comments
FYI, this is also happening in critical areas other than my footer. The 'critical areas' examples were more difficult to add to the single page example as it is all knockout driven and has tons of css layout. |
interesting. In all three instances the style 'left' or 'right' needs to be -176px. We have elements of sizes 32x32, 51x14, 46x14. Why do they all needs to -176. I thought maybe that was a minimum value, but it wasn't. |
For the facebook image: .chardinjs-tooltip Somewhere, we should add 8 to the top. But not always. So when? For the first link in the footer For the facebook image: .chardinjs-tooltip Somewhere, we should subtract 22 from the left. But not always. So when? And it isn't always 22. So how is 26 calculated? If the width is 75 or greater, it doesn't have an issue. My image is 32. I need to subtract 43 more. 32 and 43 is 75. For the link, 75 holds true as well. However, if the text isn't max 200 (say it is only 195) then the 75 goes down to 70. I'm starting to find a pattern here. |
I have branched the code. I changed the javascript and everything appears to be working now. I guess I have to change the coffee file, though. |
I have a simple footer. Some text with some links. If I have help pointing at these links, however the horizontal white line is failing.
It is also failing if placed on an div that wraps an image.
See the issue:
Here is my simple html.
Note: I tried to apply https://github.com/heelhook/chardin.js/pull/65 (which I guess is the same as https://github.com/heelhook/chardin.js/pull/68 too). It helped the vertical position but not the horizontal position.
The text was updated successfully, but these errors were encountered: