# Image resizing doesn't seem to be working
**URL:** https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092
**Category:** Bug
**Created:** [March 21, 2020, 9:48pm UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092 "2020-03-21T21:48:44Z")
**Posts on this page:** 12
**Page:** 1
### Author:  [@southpaw](https://meta.discourse.org/u/southpaw)
#### Post date: [March 21, 2020, 9:48pm UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/1 "2020-03-21T21:48:44Z")
In [Discourse 2.3.0.beta3 Release Notes](https://meta.discourse.org/t/discourse-2-3-0-beta3-release-notes/110445) we are told:
> [@Discourse 2.3.0.beta3 Release Notes](https://meta.discourse.org/t/discourse-2-3-0-beta3-release-notes/110445/1):
>
> Sometimes, users do not want images to take up the full-width of the post stream, so Discourse supports resizing images.
> […]
> Example:
> ``
> […]
> Example (width at 200px, auto-height):
> ``
I’ve noticed recently that large images are displaying at full width, and the height is clipped.
For example:
Markup for that image is written as:
``
I’ve also tried
``
with the same full-width results.
When editing, the preview pane displays the image correctly:
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [March 23, 2020, 4:45am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/2 "2020-03-23T04:45:09Z")
Yeah our tall image cropping is a very very confusing feature.
I have a fix proposed here.
> [@\`min ratio to crop\` site setting should respect w/h ratio as defined in markdown](https://meta.discourse.org/t/min-ratio-to-crop-site-setting-should-respect-w-h-ratio-as-defined-in-markdown/138374/18):
>
> @zogstrip how do you feel about if crop cropped\_width, cropped\_height = ImageSizer.crop(original\_width, original\_height) if cropped\_width \< width width = cropped\_width img["width"] = width end if cropped\_height \< height height = cropped\_height img["height"] = height end end VS the existing: It is certainly less surprising than the current behavior and would make @awesomerobot happy, only real annoyance I can see is th…
@vinothkannans can you apply it, make sure tests pass and paste a bunch of examples here?
---
### Author:  [@southpaw](https://meta.discourse.org/u/southpaw)
#### Post date: [March 24, 2020, 2:31pm UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/4 "2020-03-24T14:31:51Z")
> [@sam](#):
>
> very very confusing
Does this mean there’s something I’m not understanding and doing wrong? And I don’t mean that to sound defensive, at all, simply that if it’s just that it’s confusing and I’ve not understood it, I’d love to know what I need to correct.
---
### Author:  [@awesomerobot](https://meta.discourse.org/u/awesomerobot)
#### Post date: [March 24, 2020, 3:48pm UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/5 "2020-03-24T15:48:48Z")
Large images are cropped based on the aspect ratio of the image as uploaded (there’s a setting called `min ratio to crop` that controls when they’re cropped)… this cropping applies even if the markdown dimensions fall below the cropping threshold.
So you’re not doing anything wrong other than being unaware of how the `min ratio to crop` threshold works (it would be a little crazy to expect everyone to understand that by simply looking at the image they’re about to upload, and the ratio setting is opaque to normal users!).
The fix being applied would ensure that the markdown dimensions are taken into consideration before cropping.
---
### Author:  [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [March 25, 2020, 5:10am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/6 "2020-03-25T05:10:10Z")
Not doing anything wrong. Just imagine a user uploaded an image that was 3 pixels wide and 50000 pixels tall. What would the topic look like, if that image was displayed in proportion? What if the user posted that image over and over? 🤔
---
### Author:  [@Stephen](https://meta.discourse.org/u/Stephen)
#### Post date: [March 25, 2020, 6:57am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/7 "2020-03-25T06:57:34Z")
That reminds me, have been meaning to mention that super tall very narrow images really do screw up the Lightbox:
Those dimensions demonstrate it perfectly 😉
---
### Author:  [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [March 25, 2020, 7:01am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/8 "2020-03-25T07:01:26Z")
Right, but imagine that _without_ cropping for height!
---
### Author:  [@southpaw](https://meta.discourse.org/u/southpaw)
#### Post date: [March 25, 2020, 11:41am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/9 "2020-03-25T11:41:52Z")
Thanks! You nailed the “not understanding” part perfectly! 😄
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [May 26, 2020, 7:35am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/10 "2020-05-26T07:35:28Z")
I think @nbianca fixed this issue and the fix is deployed!
---
### Author:  [@nbianca](https://meta.discourse.org/u/nbianca)
#### Post date: [May 27, 2020, 8:19am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/11 "2020-05-27T08:19:28Z")
Yes, I merged the PR two months ago, so it should work properly now. 😊
[https://github.com/discourse/discourse/pull/9224](https://github.com/discourse/discourse/pull/9224)
---
### Author:  [@southpaw](https://meta.discourse.org/u/southpaw)
#### Post date: [May 28, 2020, 5:32am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/12 "2020-05-28T05:32:14Z")
Hooray, and thank you! We are now able to have lovely, full-length smartphone screenshots in our Community!
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [May 30, 2020, 8:00am UTC](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092/13 "2020-05-30T08:00:19Z")
This topic was automatically closed after 4 days. New replies are no longer allowed.