Tagged: image, retina, testimonials
-
AuthorPosts
-
June 21, 2021 at 11:59 pm #1306831
Hey Guys,
Just discovered that in the ALB Testimonial element, regardless of the layout chosen, the image for each testimonial (if you insert an image) is being displayed as a background image in CSS, rather than a normal image.
I’m just wondering if this is strictly necessary, or could be refactored and improved?
For instance, because it is being served as a background image in CSS it means there is no srcset or imgsizes being created. As a result of this, you can’t serve a retina version of the image properly, nor can you lazy load it properly.
Can this element be improved to be fully retina/responsive image ready like you’ve done with other elements?
Thanks for considering.
June 23, 2021 at 3:00 pm #1307162Hey THP,
Thanks for your suggestion!
I am not sure why we have chose to use background image instead of image but I will forward your request to our devs and share this thread as reference :)
Best regards,
YigitJune 23, 2021 at 3:26 pm #1307165Thank Yigit, I really hope the devs can see the benefits of changing the structure of this element.
June 28, 2021 at 12:58 pm #1307730Hi Tim,
Wanted to inform you that we are going to update the element however due to a conflict with another change on the element, it will probably not be included in upcoming version but in the next one.
This thread was added as reference so we will keep you updated there :)
Best regards,
YigitJune 28, 2021 at 3:49 pm #1307767Hey Yigit,
Thanks for the update, much appreciated. Ok I will keep an eye out for it in the next couple of releases.
Thanks,
Tim
July 8, 2021 at 4:37 pm #1309280July 8, 2021 at 7:21 pm #1309291Hey Gunter,
That’s awesome, thanks for improving that element so quickly. That’s why I stick with Enfold, your response to suggested theme improvements is always so positive.
Thanks again.
July 9, 2021 at 1:28 pm #1309394 -
AuthorPosts
- The topic ‘Testimonial Image’ is closed to new replies.