-
Notifications
You must be signed in to change notification settings - Fork 12
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
Issue with "box-sizing: border-box;" #2
Comments
Hi, Glad you like it :) Can you make a reduced test case and put it on codepen or jsfiddle? |
Hi Joren, |
Another suggestion, more like feature request. I really like your responsive feature in the plugin. But there is one thing that you might consider to improve. In Bootstrap Grid there is a situation when at definite width the columns are no longer in one row. In codepen example above I used col-md-5 and col-md-7 grid classes (these are for screens with width more than 992px). When the screen size is reduced to 800px (for example) the columns are no longer in one row and actually there is no need in equal height anymore. It would be nice to have min-width setting in the plugin from which it starts working. |
The example on codepen doesn't seem to work in Chrome, using http://rawgit.com/ to serve your assets should do the job I think. Then about your suggestion. If you take a look at the advanced example, you can see how I do this. But it would be cool indeed if this was added to the core of the plugin. You can always add a new issue where we can discuss this further. |
Hi, |
This plugin basically does the same thing but better. I recently used it and it's really nice. So I won't put any more time in my plugin since there's a better one out there. Maybe later, but then a raw js version. Anyway, thanks for your time :) |
Hi,
I had this kind of an issue when I tried to use the plugin with Bootrap 3.0 and panels. Panel has 1px border on the top and bottom and as Bootrap uses "box-sizing: border-box;" the second panel was always 2px lower because of jQuery.height() function. Maybe you should consider on jQuery.outerHeight() .
Thank you.
P.S. By the way - very nice plugin!
The text was updated successfully, but these errors were encountered: