Skip to content

feat(reference): Update astro:assets reference for new Astro 6.0 behavior#12639

Merged
florian-lefebvre merged 8 commits intov6from
feat/cropping
Nov 7, 2025
Merged

feat(reference): Update astro:assets reference for new Astro 6.0 behavior#12639
florian-lefebvre merged 8 commits intov6from
feat/cropping

Conversation

@Princesseuh
Copy link
Copy Markdown
Member

@Princesseuh Princesseuh commented Oct 29, 2025

Description (required)

In Astro 6, you can crop images and set fit and position without needing a layout. I'll admit, I don't fully understand how layouts work since I wasn't around when it was added, but hopefully I got it.

Related issues & labels (optional)

withastro/astro#14629

@netlify
Copy link
Copy Markdown

netlify bot commented Oct 29, 2025 •
edited
Loading

✅ Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit 1cb765d
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/690e034cac8b75000875b82d
😎 Deploy Preview https://deploy-preview-12639--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@astrobot-houston
Copy link
Copy Markdown
Contributor

astrobot-houston commented Oct 29, 2025 •
edited
Loading

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/guides/images.mdx Source changed, localizations will be marked as outdated.
en/guides/upgrade-to/v6.mdx Source changed, localizations will be marked as outdated.
en/reference/modules/astro-assets.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@Princesseuh Princesseuh mentioned this pull request Oct 29, 2025
Merged
@florian-lefebvre
Copy link
Copy Markdown
Member

florian-lefebvre commented Oct 29, 2025 •
edited
Loading

You need to update the upgrade guide as well, see #12322

@sarah11918
Copy link
Copy Markdown
Member

sarah11918 commented Oct 29, 2025

So based on this, if fit and position no longer require layout, then that "Responsive Images Properties" section probably isn't needed anymore, and layout itself could just be thrown up higher into the full list of image properties too, eh?

Like, this isn't true anymore:

Setting the layout property on an <Image /> or <Picture /> component creates a responsive image and enables additional property settings.

That was the reason to have the section, so I'd suggest we just move the layout API reference up too, and remove that section.

@sarah11918 sarah11918 added the 6.0 label Oct 29, 2025
@sarah11918 sarah11918 added this to the v6.0.0 docs milestone Oct 29, 2025
@Princesseuh
Copy link
Copy Markdown
Member Author

Princesseuh commented Nov 4, 2025

I removed the section entirely. I'll try to think of something to put in the upgrade guide

@sarah11918
Copy link
Copy Markdown
Member

sarah11918 commented Nov 4, 2025

It's not going to let me suggest fixes for the links that are broken now that there is no more section called "Responsive Images."

In general, use the following ideas to fix these:

  • occurrences of "responsive images" are just plain text, no link. They are a thing that exists, but they are (no longer) an Astro thing. So we use the phrase as regular terminology, but we don't document what they are.
  • "responsive image properties" no longer exist. They are not a thing. They should not even be mentioned.

In most cases, just removing the link should be fine. In a case where e.g. it says "image properties AND responsive image properties", it's now just "image properties."

As for the images.mdx guide page that now has a broken link, I would remove the link and update the text there to be something like:

With responsive image properties (layout, fit, position) applied to the <Image /> or <Picture /> components, ...

For the guide page, it's fine to be a little bit more explainy. These are generally known as properties you use when making images responsive.

If you have any doubt, when you update these, they will at least become changes I can make suggestions on top of!

sarah11918
sarah11918 reviewed Nov 4, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one small comment, and cleaning up all the now-broken anchor links, and this looks great!

Princesseuh and others added 2 commits November 4, 2025 17:02
@Princesseuh
fix: adjust for feedback
5eb5d4d
@Princesseuh @sarah11918
Update src/content/docs/en/reference/modules/astro-assets.mdx
d5c3b16 
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
florian-lefebvre
florian-lefebvre reviewed Nov 4, 2025
View reviewed changes
Princesseuh and others added 2 commits November 5, 2025 06:30
@Princesseuh
fix: update guide
5b9b9e8
@Princesseuh @florian-lefebvre
Apply suggestions from code review
e68803b 
Co-authored-by: Florian Lefebvre <contact@florian-lefebvre.dev>
florian-lefebvre
florian-lefebvre reviewed Nov 5, 2025
View reviewed changes
ArmandPhilippot
ArmandPhilippot reviewed Nov 5, 2025
View reviewed changes
sarah11918
sarah11918 reviewed Nov 5, 2025
View reviewed changes
@florian-lefebvre @ArmandPhilippot
Apply suggestions from code review
35a13d9 
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
ArmandPhilippot
ArmandPhilippot approved these changes Nov 7, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left a nit regarding the table of contents, otherwise the contents looks great to me!

sarah11918
sarah11918 reviewed Nov 7, 2025
View reviewed changes
sarah11918
sarah11918 reviewed Nov 7, 2025
View reviewed changes
sarah11918
sarah11918 approved these changes Nov 7, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@florian-lefebvre florian-lefebvre merged commit 7bee4c3 into v6 Nov 7, 2025
10 checks passed
@florian-lefebvre florian-lefebvre deleted the feat/cropping branch November 7, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sarah11918 sarah11918 sarah11918 approved these changes

@ArmandPhilippot ArmandPhilippot ArmandPhilippot approved these changes

@florian-lefebvre florian-lefebvre florian-lefebvre approved these changes

Assignees

@Princesseuh Princesseuh

Labels

6.0

Projects

None yet

Milestone

v6.0.0 docs

Development

Successfully merging this pull request may close these issues.

5 participants

@Princesseuh @astrobot-houston @florian-lefebvre @sarah11918 @ArmandPhilippot