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 resize changes only image width when img tag has assigned style="height..." #14461

Open
Acrophost opened this issue Jun 26, 2023 · 1 comment
Labels
package:image support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@Acrophost
Copy link

Acrophost commented Jun 26, 2023

📝 Provide detailed reproduction steps (if any)

  1. Open any demo which includes image resize plugin
  2. Insert an image
  3. Add a style="height:320px;" to img tag via devtools
  4. Resize the image

or (much worse case, since it touches IfW output)

  1. Open IfW demo: https://ckeditor.com/docs/ckeditor5/latest/features/converters/import-word/import-word.html
  2. Import a document with images
  3. Change image alignment to wrap text/break text
  4. Resize image

✔️ Expected result

Image will get resized both in width and height

❌ Actual result

Images with style="height..." defined get resized only in width

Screen.Recording.2023-06-26.at.12.33.17.mov

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@Acrophost Acrophost added type:bug This issue reports a buggy (incorrect) behavior. package:image labels Jun 26, 2023
@Acrophost Acrophost changed the title Image resize changes only image width when img tag has assigned height Image resize changes only image width when img tag has assigned style:height Jun 26, 2023
@Acrophost Acrophost changed the title Image resize changes only image width when img tag has assigned style:height Image resize changes only image width when img tag has assigned style="height..." Jun 26, 2023
@Witoso
Copy link
Member

Witoso commented Jun 26, 2023

It should be fixed along with the #14146 issue. @mmotyczynska interesting scenario to test out at the end ☝️.

@Witoso Witoso added the support:2 An issue reported by a commercially licensed client. label Jun 26, 2023
@martynawierzbicka martynawierzbicka added support and removed support:2 An issue reported by a commercially licensed client. labels Jun 26, 2023
@martynawierzbicka martynawierzbicka added support:2 An issue reported by a commercially licensed client. and removed support labels Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:image support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

3 participants