Image block: Incorrect image width with aspect ratio in block editor

[ajlende]

Description

In the post editor, if you have an image block with an image that is smaller than the width of the post container and you specify an aspect ratio without selecting a width or height dimension, the image is incorrectly scaling up to fill the width of the container.

Step-by-step reproduction instructions

Screenshots use https://github.com/WordPress/wordpress-develop/blob/62b286f9b2ebc7bb609827a09fcf5136cac5fa01/tests/phpunit/data/images/a2-small.jpg for testing.

1. Use the following image block content or create a block includes an aspect ratio (and scale), but no width or height.

<!-- wp:image {"id":443,"aspectRatio":"3/4","scale":"cover","sizeSlug":"full","linkDestination":"none"} -->
<figure class="wp-block-image size-full"><img src="http://wp.local/wp-content/uploads/2023/06/a2-small.jpg" alt="" class="wp-image-443" style="aspect-ratio:3/4;object-fit:cover"/></figure>
<!-- /wp:image -->

3. See that the image fills the width of the editor instead of the width of the image.

Screenshots, screen recording, code snippet

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

[draganescu]

This does not happen if the image is picked from the media library 🤔

[Mamaduka]

Was this resolved via #52286?

[ajlende]

Was this resolved via #52286?

No, that PR exclusively changed the frontend, not the editor. I took a break from this last week, but I'm back on it now.

[ajlende]

This one is way more frustrating than it should be. The solution should be as easy as adding fit-content for width and height for the resizable container.

The problem is that re-resizable doesn't support fit-content. It looks like the style is being overridden when they render the component. It overrides custom styles, passing in fit-content appends px resulting in fit-contentpx, undefined results in 'auto', and null crashes the component.