Comparison with Drupal Core lazy loading, future of this module

Created on 30 November 2022, about 2 years ago
Updated 20 September 2024, 5 months ago

This module's description mentions at the top "There is also an initiative to get this feature into core" -- that initiative at https://www.drupal.org/project/drupal/issues/3167034#comment-13845238 📌 Leverage the 'loading' html attribute to enable lazy-load by default for images in Drupal core Fixed has been merged way back in September 2020. We're looking at "should we use this module?" and have a few questions:

1. What benefits does this module have that Drupal Core is not doing?
2. Future of this module: I see it is marked as minimally maintained and feature complete. If there's not a significant benefit compared to Drupal Core, I suppose this module could be marked as deprecated?
3. Bonus: update the module description to note that #3167034 is now merged and with answers to the above.

Thanks in advance for any guidance.

💬 Support request
Status

Fixed

Version

1.1

Component

Documentation

Created by

🇩🇪Germany Ken Hawkins

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇫🇷France prudloff Lille

    I did some tests and I think this module can be deprecated:

    • template_preprocess_image() now does the same thing as native_lazy_loading_preprocess_image()
    • native_lazy_loading_preprocess_image_formatter() and native_lazy_loading_preprocess_responsive_image_formatter() don't seem necessary anymore to add the width and height attributes. (template_preprocess_image_formatter() adds them for non-responsive images. For responsive images, I am not sure which function adds them.)
    • _responsive_image_build_source_attributes() adds the width and height attributes on source tags so native_lazy_loading_preprocess_responsive_image() is not needed anymore.
  • Merge request !3Make module obsolete → (Merged) created by prudloff
  • Status changed to Needs review 6 months ago
  • 🇫🇷France prudloff Lille
  • Status changed to Fixed 6 months ago
  • 🇫🇷France GaëlG Lille, France
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024