Problem/Motivation

We noticed a problem after the update to Blazy v. 8.x-2.10-rc1: often, but not always, there is additional blank space under the loaded images, as big as the images themselves.
When the problem appears, the DIV element containing the PICTURE element has a padding-bottom and the picture has default positioning, so the padding is added under the image.
Sometimes, even reloading the same page that was showing the problem, the padding-bottom is not present and the images are shown correctly.
We use the Media module for the images.

Thank you

CommentFileSizeAuthor
#4 3274730-ratio-leak-4.patch1.32 KBgausarts
Screenshot from 2022-04-12 10-04-43.png663.41 KBtrickfun
Screenshot from 2022-04-12 09-57-16.png663.96 KBtrickfun

Comments

trickfun created an issue. See original summary.

gausarts’s picture

gausarts commented

Thank you.

Is it using aspect ratio Fluid?
If not, try Fluid and see.

If yes, worse than I thought :)

trickfun’s picture

trickfun commented

Fluid works fine!
Thank you

gausarts’s picture

gausarts commented
Component: JavaScript » Regression
Status: Active » Needs review
Related issues: +#3257511: Optimization pre Blazy 3
StatusFileSize
new 3274730-ratio-leak-4.patch1.32 KB

Thank you for detailed screenshots. I could reproduce the OP easily.

Attached to remove the data-ratios leak, only relevant if Fluid is selected under Aspect ratio option.

This is no issue if you choose to decouple lazy loader via `No lazyload: lazy`.
No issue for non Responsive image, either.

With the patch, leaving Aspect ratio empty should not leak the confusing padding, previously only added if Fluid was chosen.

  • gausarts committed 45657ac on 8.x-2.x 
    Issue #3274730 by gausarts, trickfun: Padding bottom under the image
gausarts’s picture

gausarts commented
Status: Needs review » Fixed

Committed. Thank you for contribution.

trickfun’s picture

trickfun commented

Now it works!!
Thank you

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.