Skip to content

Conversation

@ascorbic
Copy link
Contributor

@ascorbic ascorbic commented Apr 28, 2025

Changes

The "simplified" responsive image styles simpified things a bit far, and was causing the images to not correctly maontain aspect ratio in some cases. This PR adds height: auto back in, which fixes it

Fixes #13709

Testing

Tested manually. This shows we need e2e tests for images.

Docs

@changeset-bot
Copy link

changeset-bot bot commented Apr 28, 2025

🦋 Changeset detected

Latest commit: a73f9a3

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

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Apr 28, 2025
@ascorbic ascorbic merged commit 2103991 into main Apr 29, 2025
14 checks passed
@ascorbic ascorbic deleted the resp-img-height branch April 29, 2025 10:15
@astrobot-houston astrobot-houston mentioned this pull request Apr 29, 2025
@robsimmons
Copy link

Thanks!

openscript pushed a commit to openscript/astro that referenced this pull request Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

experimentalLayout breaks between 5.7.4 and 5.7.7

3 participants