-
Notifications
You must be signed in to change notification settings - Fork 802
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
Can drop display:none #95
Labels
Comments
Is |
It is set via the style attribute yes. Isotope uses that to filter out elements. |
Thanks, should be resolved in the latest master. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We're combining this with http:https://isotope.metafizzy.co/ . Though that probably does not specifically matter.
In some situations, for some elements a set 'display: none' is dropped. Strangely not for all hidden elements.
I haven't gone through all the code so I don't see why it would sometimes not drop display:none.
But https://github.com/liabru/jquery-match-height/blob/master/jquery.matchHeight.js#L253 does not seem handy. I think it would be better if on line 235 you store the original display value in a variable and apply that on line 253 again.
The text was updated successfully, but these errors were encountered: