Automattic / themes

Free WordPress themes made by Automattic for WordPress.org and WordPress.com.
https://themeshaper.com
GNU General Public License v2.0
852 stars 345 forks source link

Illustratr: lazy-loading image feature breaks layout on portfolio template #2154

Open cecilearkay opened 4 years ago

cecilearkay commented 4 years ago

Steps to replicate

1) Install Illustratr as a theme 2) Create portfolio projects 3) Set up a page with the Portfolio template 4) AT your site with the Business plan 5) Activate the lazy-loading images feature in Jetpack

Result

Images jump, get stuck together:

Annotation on 2020-06-19 at 13-48-44

Expected

Proper spacing between each element, like so:

Capture d’écran 2020-06-19 à 13 45 50

cecilearkay commented 4 years ago

Also reported in #22480724-hc EDIT: the theme is Qua, so it looks like it's widespread.

mikeicode commented 3 years ago

Reported (Illustratr): 27843924-hc

sophiegyo commented 2 years ago

I think this is related, although it doesn't look the same. Report: 4423014-zen

Issue: at the bottom of the portfolio projects in Illustratr, on the portfolio layout page, the images don't quite align. I realise Illustratr is primarily a masonry layout but since the other projects on the site all line up nicely in a grid, it seems odd that only the last ones would be misaligned. Judging from how the HTML has inline absolute positioning on each item, which looks like it readjusts via JS when the browser is resized, I'd hazard a guess this is still *lazy loading which has this effect.

Images: DPFZ0hx

VPTFBr0