-
Notifications
You must be signed in to change notification settings - Fork 800
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
$.fn update not firing #60
Comments
What you're describing seems to be the correct approach and should work. |
I'm having the same problem. |
Same problem. |
- Doesn't work though. There seems to be a bug: liabru/jquery-match-height#60 Signed-off-by: Florian Ziegler <[email protected]>
Anyone have a work around for this? |
It seems to work with previous version 0.5.2. |
Can somebody please provide a jsfiddle that reproduces this? Thanks |
I'm having this issue too. Edit: It doesn't look like it's working in any callback function. So far tested with isotope and blazy. |
I think the problem here may be a misunderstanding of how the plugin works. When you call the Instead you need to create a new selection that includes the new element and remove the old one. Here is an example:
I hope that clears things up. If this still doesn't solve the problem you guys are having, please provide more information and an example. Thanks. |
Thanks @liabru, I have a page with boxes on it, consisting of text and images. I'm using Isotope to load all of the boxes on the page on load. The box container has a class of
I'm then using filtering in Isotope to filter all the items. It's not loading in anything new.
For some reason though, everything has a height of 6px. This might be that lazy loading hasn't taken effect yet however, even so, when the boxes do show, the Hope this helps. |
voltronik Change the line $.fn.matchHeight._update(); to jQuery.fn.matchHeight._update(); It worked for me in one instance the other issue I had was it was being called before all of the other elements were finished resizing so I set a delay. Hope that helps. |
@apeitup Changing $ to jQuery seemed to do the trick for some reason. I've still got some crazy overlapping and filtering issues with Isotope but i'll work those out in due course. Thanks! |
From my experience from isotope it needs to be called after the layout is set as it uses the dimensions of the item to position them on screen. So calling matchheight after could be causing the issues |
@apeitup Brilliant, thanks, i'll try that! |
I have a ajax call that loads new gallery items but sometimes it fails to set a proper height. I did try to do "remove" but i does't help. Any ideas why?
|
@liabru
|
@TrangVo are you applying to the |
@liabru |
Great, no problem! |
Hi,
Any help is much appreciated! |
I have a group which has content appended to it via AJAX. At the end of the images loading in the group, I show the content.
If I use:
$('.blocks').matchHeight(true);
it works perfectly fine.If I use:
$('.blocks').matchHeight(true);
outside of the $.ajax call and try utilizing$.fn.matchHeight._update();
to trigger an update when the content is shown, it doesn't fire.I have a few other areas where the content is dynamically adjusted and I'd like to have the matchHeight updated but as of now, I can't do that. I assume sprinkling
.matchHeight(true);
's everywhere is probably not the greatest option, so hopefully, I'm just missing something painfully obvious. 😄The text was updated successfully, but these errors were encountered: