Skip to content

Commit

Permalink
fix: always produce image sizes attribute when PhotoAlbum sizes i…
Browse files Browse the repository at this point in the history
…s present
  • Loading branch information
igordanchenko committed Apr 11, 2023
1 parent f196c6c commit ed6fd6f
Showing 1 changed file with 9 additions and 8 deletions.
17 changes: 9 additions & 8 deletions src/components/renderers/PhotoRenderer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -40,15 +40,16 @@ function srcSetAndSizes<T extends Photo = Photo>(photo: T, layout: PhotoLayout,
.sort((first, second) => first.width - second.width)
.map((image) => `${image.src} ${image.width}w`)
.join(", ");
}

if (layoutOptions.sizes) {
sizes = (layoutOptions.sizes.sizes || [])
.map(({ viewport, size }) => `${viewport} ${calculateSizesValue(size, layout, layoutOptions)}`)
.concat(calculateSizesValue(layoutOptions.sizes.size, layout, layoutOptions))
.join(", ");
} else {
sizes = `${Math.ceil((layout.width / layoutOptions.containerWidth) * 100)}vw`;
}
// always produce image `sizes` attribute when PhotoAlbum `sizes` is present (use case: NextJS image)
if (layoutOptions.sizes) {
sizes = (layoutOptions.sizes.sizes || [])
.map(({ viewport, size }) => `${viewport} ${calculateSizesValue(size, layout, layoutOptions)}`)
.concat(calculateSizesValue(layoutOptions.sizes.size, layout, layoutOptions))
.join(", ");
} else if (srcSet) {
sizes = `${Math.ceil((layout.width / layoutOptions.containerWidth) * 100)}vw`;
}

return { srcSet, sizes };
Expand Down

0 comments on commit ed6fd6f

Please sign in to comment.