🦋 Changeset detected

Latest commit: 203aa8c

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Hey there! I initially added the option to replace the background with a solid color in @astrojs/image (#4642).
The intent was to allow to replace the transparent background on an image regardless of the output format. With this PR, formats that support transparency cannot anymore be transformed to have a solid background and no alpha channel.

The initial behavior (allowing replacing alpha with color for all formats) is useful because such images have a smaller filesize. This is why I added this feature and how I'm using it in my projects (I have a source of truth as transparent PNG and generate an optimized image in webp without alpha at build time). Since webp supports transparency, this PR increased the size of my assets.

If that's OK, I will soon make a PR that restores the initial behavior, assuming that when the user passes a background property to the component, they want to set the background to a specific color.
I'll also take the opportunity to fix the tests where the rgba variants don't use rgba in the fixture at the moment (which lead to a typescript warning)

Pinging @tony-sull to see if that would be ok. (here is a preview of the changes beeb@7a78ce6 )

@beeb changing the current behavior would be a breaking change, I'd definitely recommend starting up an RFC discussion before you dive too far into a PR to get some feedback on the idea 🙌

Erika opened a discussion not long ago to kickstart a cleanup of @astrojs/image, that might actually be a good place to add this feedback

Personally I've always been torn on exposing so many sharp-specific props through our components - a general "make @astrojs/image easier to use" RFC might be the right time to revisit whether we even want all the extra complexity of image transformations beyond the basics of dimensions and image format 🤔

Cheers @tony-sull , I'll pitch in on the discussion thread. Note that this PR #4800 was a breaking change to the behavior I implemented. But it was never documented as such.