# 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:** 1
**Showing post:** 2

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@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")

</div>

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?

---

_[View the full topic](https://meta.discourse.org/t/image-resizing-doesnt-seem-to-be-working/145092)._
