Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Image Size: Inconsistent behavior between rendered size and intrinsic size #51177

Open
hanneslsm opened this issue Jun 1, 2023 · 6 comments
Open
Labels
[Block] Image Affects the Image Block [Feature] Media Anything that impacts the experience of managing media [Type] Bug An existing feature does not function as intended

Comments

@hanneslsm
Copy link

Description

It's possible to resize the width of a image, even if the content width should constrain it. The website doesn't display the wider image, the editor does.
Also, should't be the percentages calculated from the content width, not the image size? If I have a 620px content width, selecting 50% I'd expect the image to be 310px then.

Step-by-step reproduction instructions

  1. Change the size of an image with the handles
  2. The new size is not correctly displayed on the website.

Screenshots, screen recording, code snippet

Screen.Recording.2023-06-01.at.17.43.23.mp4

Environment info

  • Custom Theme creating with Create Block Theme plugin

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

@skorasaurus skorasaurus added [Block] Image Affects the Image Block [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed [Status] Needs More Info Follow-up required in order to be actionable. and removed [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed labels Jun 5, 2023
@skorasaurus
Copy link
Member

Thank you for reporting.

It's possible to resize the width of a image, even if the content width should constrain it.

This is a known issue; reported at #12168

Also, should't be the percentages calculated from the content width, not the image size? If I have a 620px content width, selecting 50% I'd expect the image to be 310px then.

On the front end or within the editor?

@hanneslsm
Copy link
Author

hanneslsm commented Jun 6, 2023

This is a known issue; reported at #12168

It amazes me how often this bug is reported and it's still not fixed.

On the front end or within the editor?

Isn't the idea that the editor displays the same as the front end? When the container div constrains the content and image to 620px, this should be the 100%. There is no value in using the original image size as 100% - or even giving the information to the user on how much the original image size is, because it will not be displayed as such anyway.
Let's take this scenario: As a user I'm uploading a image which I have taken on my iPhone. I know it has some thousand pixels, but I don't know the exact value - also when uploading it to my page I do not really care if its 1234px or 4321px or anything else - it should be displayed as 620px - and that's the 100% I care about. When clicking 50%, I'd expect it to be half of the content size: 310px.

@hanneslsm
Copy link
Author

hanneslsm commented Jun 6, 2023

This also relates to the discussion here: #48618
I think the whole experience around the image size settings is too technical.
To improve the UX we should think about: What does the user want? What do they expect? Can everything else be automatically handled by the software?

@github-actions
Copy link

Help us move this issue forward. This issue is being marked stale since it has no activity after 15 days of requesting more information. Please add info requested so we can help move the issue forward. Note: The triage policy is to close stale issues that need more info and no response after 2 weeks.

@github-actions github-actions bot added the [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. label Jun 22, 2023
@hanneslsm
Copy link
Author

@skorasaurus this issue has the label "needs more info" and is now being marked as stale. What more info are needed?

@skorasaurus skorasaurus added [Feature] Media Anything that impacts the experience of managing media and removed [Status] Needs More Info Follow-up required in order to be actionable. [Status] Stale Gives the original author opportunity to update before closing. Can be reopened as needed. labels Jun 22, 2023
@skorasaurus
Copy link
Member

@skorasaurus this issue has the label "needs more info" and is now being marked as stale. What more info are needed?

Sorry, the stale label and comment is automatically applied when the 'needs more info' label is applied; I forgot to remove the label.

@jordesign jordesign added the [Type] Bug An existing feature does not function as intended label Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Image Affects the Image Block [Feature] Media Anything that impacts the experience of managing media [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

3 participants