[Avatar] Allow `Avatar.Image` to be supplied a `component` to render the image

[mririgoyen]

Feature request

Overview

The Avatar.Image primitive assumes I want to use an img without the ability to override that. I'd like to use the Next.js Image component instead for its optimizations. If I try to use the Next Image component instead of the primitive, the Fallback fails to work properly.

Examples in other libraries

Other libraries, such as Material UI, allow you to pass an optional component as a prop.

<Image
  alt={altText}
  className="aspect-square h-full w-full"
  component={NextImage} // Provide a React component or a string value of an HTML element, default would be "img"
  src={src}
/>

Who does this impact? Who is this for?

This would affect anyone utilizing the Next.js framework who wants to use it's image optimizations.

Other Notes

If there are any other primitives that make use of images, allowing overrides on them would be greatly appreciated.

[joaom00]

Did you try this?

<Avatar.Image asChild>
  <NextImage />
</Avatar.Image>

[mririgoyen]

Did you try this?

<Avatar.Image asChild>
  <NextImage />
</Avatar.Image>

This will throw type errors because none of the required props are being supplied to NextImage.

[zaunermax]

Did you try this?

<Avatar.Image asChild>
  <NextImage />
</Avatar.Image>

For me this does not throw type errors, it just outright does not work. Instead it renders the AvatarFallback instead.

[raffizulvian]

I faced a similar issue. My workaround was to also supply src to Avatar.Image and NextImage.

<Avatar.Image asChild src='image.svg'>
  <NextImage src='image.svg' alt='Some alt text' {...otherProps} />
</Avatar.Image>

It's a little bit redundant, but it could be wrapped inside a wrapper component.

I think it would be nice to make src not required if asChild is present or use a solution like other libraries as stated above by the author.

[trm217]

Another issue with having to supply the src on Avatar.Image, is that asset imports, which enables further build-time optimisation, is not possible as Avatar.Image would not accept it for the src value.

[akhan619]

I faced a similar issue. My workaround was to also supply src to Avatar.Image and NextImage.

<Avatar.Image asChild src='image.svg'>
  <NextImage src='image.svg' alt='Some alt text' {...otherProps} />
</Avatar.Image>

It's a little bit redundant, but it could be wrapped inside a wrapper component.

I think it would be nice to make src not required if asChild is present or use a solution like other libraries as stated above by the author.

This is not only redundant but highly inefficient. Just checked devtools and both the avatar and next-image components load the asset. So we end up loading the optimized image by next-image and then the avatar-image also loads the image. So, in the end there are 2 requests for the same image.

[akhan619]

The solution i am using currently is to not use the Avatar.Image at all and just use the NextImage and Avatar.Fallback . Just to give a barebones example:

<Avatar.Root>
    {imageSrc && <NextImage src={imageSrc} alt='Some alt text' {...otherProps} />}
    {!imageSrc && <Avatar.Fallback>John Doe</Avatar.Fallback>}
</Avatar.Root>

If imageSrc is not valid or there are errors loading the image then NextImage error handling and some state variables can be used to handle the case.

[DasOhmoff]

I am facing the same issue. Fixing this would make radix way more viable for the people

[jonathanlal]

agreed, would be a really nice feature

[mehrabmp]

Any update on this?