- This topic has 13 replies, 3 voices, and was last updated 4 years, 1 month ago by Tom.
-
AuthorPosts
-
March 1, 2020 at 3:35 pm #1181726Mathieu
Hi Tom,
It is a problem I had at various point while building my child theme (like here : https://generatepress.com/forums/topic/add-text-to-the-homepage-in-a-blog/)
There is masonry that is going over the content to various degrees. I fixed some by adding empty paragaphs but it’s not super clean. I tried clear: both, clear: left, clear:right, various padding, margin and br.
There’s nothing that really fixed it properly and permanently.
Here’s an example, on mobile : https://chefcuisto.com/cuisine/fast-food/ and a screenshot if you don’t see it : https://prnt.sc/ra1lml
Kindest regards,
March 1, 2020 at 4:32 pm #1181752LeoStaffCustomer SupportHi there,
Any chance you can disable all plugins except GP Premium to eliminate conflicts from other plugins first?
Please remove your custom CSS fix while we are debugging as well.
Let me know π
March 2, 2020 at 8:12 am #1182459MathieuHi Leo,
Unfortunately, I can’t disable every plugins on the live website but I did manage to reproduce it on a trimmed down version (http://nouveauchef.quebecblogue.com/category/type-de-recette/).
When I disable the lazyloading of the image (by Autoptimize), it is going away.
However, Masonry is repainting itself succesfully with the lazyloaded images everywhere else. It’s like there’s no triggering of masonry on the “taxonomy-description” class even if it is the main content element.
I can fix it easily myself by adding a class to the image that will be excluded from the lazyloading but it does seem to me that it is also something that should work.
I’ll wait for your take on that.
Kindest regards.
March 2, 2020 at 5:01 pm #1182859LeoStaffCustomer SupportCan you make sure to use our suggested setting for Autopimize here?
https://generatepress.com/fastest-wordpress-theme/Let me know π
March 3, 2020 at 4:39 am #1183326MathieuIt’s not an issue with autoptimize settings. I have a lazyloaded image (from autoptimize) in the taxonomy-description area but that doesn’t trigger a masonry repaint like it does elsewhere.
March 3, 2020 at 9:26 am #1183790TomLead DeveloperLead DeveloperHi there,
Does this PHP fix it?:
add_action( 'wp', function() { remove_action( 'generate_archive_title', 'generate_archive_title' ); add_action( 'generate_before_main_content', 'generate_archive_title', 1 ); } );
March 3, 2020 at 11:48 am #1183933MathieuNope. Same thing.
March 3, 2020 at 5:33 pm #1184189TomLead DeveloperLead DeveloperIs that code still active here?: http://nouveauchef.quebecblogue.com/category/type-de-recette/
If not, can you re-add it?
March 4, 2020 at 6:02 am #1184643MathieuIt wasn’t, now it is back. But the first line, remove_action doesn’t work as it is outputting the block twice.
I’ve commented line 33 of generatepress/archive.php for the time being.
//do_action('generate_archive_title');
March 4, 2020 at 8:37 am #1184958TomLead DeveloperLead DeveloperCan you try the updated code?: https://generatepress.com/forums/topic/problem-with-generatepress-masonry-collapsing-over-content/#post-1183790
March 4, 2020 at 9:23 am #1185003MathieuYes, it does solve the masonry problem. It does add some padding and push it to the right tho.
Also, work better wrapped in
if (is_archive()) { }
Otherwise it adds “Archive” to every page.
Thanks
March 4, 2020 at 7:34 pm #1185394TomLead DeveloperLead DeveloperYou can fix the margin with this:
.page-header { margin: 0; }
This looks like something we can improve in the plugin itself. I’ll see what I can do π
March 5, 2020 at 7:46 am #1186047MathieuThanks a lot again Tom π It was a small glitch but I am glad you made it works!
Have a nice day.
March 5, 2020 at 5:46 pm #1186513TomLead DeveloperLead DeveloperNo problem! You too π
-
AuthorPosts
- You must be logged in to reply to this topic.