mirror of
https://github.com/withastro/astro.git
synced 2024-12-23 21:53:55 -05:00
944d24e9ee
* Update Picture.astro the image variable of getPicture contains a width and height property, which we usually require. In this case, the image is wrapped in a picture tag and the img tag itself should not have a width and height property as this will break the responsiveness of the image provided by the picture tag. * added changeset * fixing SSG picture tests that were expecting img dimensions Co-authored-by: Tony Sullivan <tony.f.sullivan@outlook.com>
77 lines
1.9 KiB
Text
77 lines
1.9 KiB
Text
---
|
|
import { getPicture } from '../dist/index.js';
|
|
import { warnForMissingAlt } from './index.js';
|
|
import type { ImgHTMLAttributes, HTMLAttributes } from './index.js';
|
|
import type { ImageMetadata, OutputFormat, TransformOptions } from '../dist/index.js';
|
|
|
|
interface LocalImageProps
|
|
extends Omit<HTMLAttributes, 'src' | 'width' | 'height'>,
|
|
Omit<TransformOptions, 'src'>,
|
|
Pick<astroHTML.JSX.ImgHTMLAttributes, 'loading' | 'decoding'> {
|
|
src: ImageMetadata | Promise<{ default: ImageMetadata }>;
|
|
/** Defines an alternative text description of the image. Set to an empty string (alt="") if the image is not a key part of the content (it's decoration or a tracking pixel). */
|
|
alt: string;
|
|
sizes: HTMLImageElement['sizes'];
|
|
widths: number[];
|
|
formats?: OutputFormat[];
|
|
}
|
|
|
|
interface RemoteImageProps
|
|
extends Omit<HTMLAttributes, 'src' | 'width' | 'height'>,
|
|
TransformOptions,
|
|
Pick<ImgHTMLAttributes, 'loading' | 'decoding'> {
|
|
src: string;
|
|
/** Defines an alternative text description of the image. Set to an empty string (alt="") if the image is not a key part of the content (it's decoration or a tracking pixel). */
|
|
alt: string;
|
|
sizes: HTMLImageElement['sizes'];
|
|
widths: number[];
|
|
aspectRatio: TransformOptions['aspectRatio'];
|
|
formats?: OutputFormat[];
|
|
background: TransformOptions['background'];
|
|
}
|
|
|
|
export type Props = LocalImageProps | RemoteImageProps;
|
|
|
|
const {
|
|
src,
|
|
alt,
|
|
sizes,
|
|
widths,
|
|
aspectRatio,
|
|
fit,
|
|
background,
|
|
position,
|
|
formats = ['avif', 'webp'],
|
|
loading = 'lazy',
|
|
decoding = 'async',
|
|
...attrs
|
|
} = Astro.props as Props;
|
|
|
|
if (alt === undefined || alt === null) {
|
|
warnForMissingAlt();
|
|
}
|
|
|
|
const { image, sources } = await getPicture({
|
|
src,
|
|
widths,
|
|
formats,
|
|
aspectRatio,
|
|
fit,
|
|
background,
|
|
position,
|
|
});
|
|
|
|
delete image.width;
|
|
delete image.height;
|
|
---
|
|
|
|
<picture {...attrs}>
|
|
{sources.map((attrs) => <source {...attrs} {sizes} />)}
|
|
<img {...image} {loading} {decoding} {alt} />
|
|
</picture>
|
|
|
|
<style>
|
|
img {
|
|
content-visibility: auto;
|
|
}
|
|
</style>
|