-
Notifications
You must be signed in to change notification settings - Fork 795
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
Ignore elements set to "display: none" #126
Comments
This actually an intended as a feature (no really!). Can you try something like |
Thanks for the response, Would that work when visibility changes at different breakpoints? What would be the use case for this "feature"? Seems matching the biggest height would be the expected outcome. |
Ah you're looking for breakpoint support? Take a look at #65 and +1 if that's what you need. I think there's some code there that someone gave that you can use on the existing version too. |
Not exactly, I'm looking for whatever elements are visible to match the highest visible height. So you could have:
And the end result should be both .item1 and .item3 being set to 500px. Currently only .item1 and .item2 (even with display:none) are set to a height of 500px and .item3 is given a blank style attribute. Thanks, |
The solution I gave earlier for that should work, but if you're dynamically modifying visibility then you will need to remove and reapply |
If an element in a group is set to display none, it can stop other elements from receiving the proper height.
Sometimes I hide elements at certain breakpoints, and it would be great if only visible elements would be considered to figure height and apply.
The text was updated successfully, but these errors were encountered: