@adobe/helix-html-pipeline 2.1.0 → 2.1.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [2.1.1](https://github.com/adobe/helix-html-pipeline/compare/v2.1.0...v2.1.1) (2022-06-02)
+
+
+### Bug Fixes
+
+* Lazy images ([#73](https://github.com/adobe/helix-html-pipeline/issues/73)) ([7442a5e](https://github.com/adobe/helix-html-pipeline/commit/7442a5e0720699d974332a2f5659fdddd27b0dc6))
+
 # [2.1.0](https://github.com/adobe/helix-html-pipeline/compare/v2.0.4...v2.1.0) (2022-06-02)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-html-pipeline",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "Helix HTML Pipeline",
   "main": "src/index.js",
   "types": "src/index.d.ts",

package/src/steps/create-pictures.js

@@ -18,7 +18,7 @@ const BREAK_POINTS = [
   { width: '750' },
 ];
 
-export function createOptimizedPicture(src, alt = '', eager = false) {
+export function createOptimizedPicture(src, alt = '') {
   const url = new URL(src, 'https://localhost/');
   const { pathname, hash = '' } = url;
   const props = new URLSearchParams(hash.substring(1));
@@ -53,7 +53,7 @@ export function createOptimizedPicture(src, alt = '', eager = false) {
       });
     }
     return h('img', {
-      loading: eager ? 'eager' : 'lazy',
+      loading: 'lazy',
       alt,
       type: v.type,
       src: srcset,
@@ -77,11 +77,9 @@ function isMediaImage(node) {
 export default async function createPictures({ content }) {
   const { hast } = content;
 
-  let first = true;
   visitParents(hast, isMediaImage, (img, parents) => {
     const { src, alt } = img.properties;
-    const picture = createOptimizedPicture(src, alt, first);
-    first = false;
+    const picture = createOptimizedPicture(src, alt);
 
     // check if parent has style and unwrap if needed
     const parent = parents[parents.length - 1];

package/src/steps/set-custom-response-headers.js

@@ -21,8 +21,8 @@ import { cleanupHeaderValue } from '@adobe/helix-shared-utils';
  */
 export default function setCustomResponseHeaders(state, req, res) {
   Object.entries(state.headers.getModifiers(state.info.path)).forEach(([name, value]) => {
-    // don't use `link` header for non-html pipeline
-    if (name !== 'link' || state.type === 'html') {
+    // only use `link` header for extensionless pipeline
+    if (name !== 'link' || (state.type === 'html' && state.info.selector === '')) {
       res.headers.set(name, cleanupHeaderValue(value));
     }
   });