# Feature Request: Tall and Thin Image Handling

**URL:** https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573
**Category:** Feature
**Created:** [May 2, 2016, 10:34pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573 "2016-05-02T22:34:14Z")
**Posts on this page:** 14
**Page:** 1

<div class="post-metadata">

### Author: ![fearlessfrog](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/fearlessfrog/32/119653_2.png) [@fearlessfrog](https://meta.discourse.org/u/fearlessfrog)
#### Post date: [May 2, 2016, 10:34pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/1 "2016-05-02T22:34:14Z")

</div>

I think this is a feature request. We have users putting images in posts, and sometimes those images are quite tall, i.e. height much greater than the width. Discourse provides an image preview in the post, but uses the image height within the post height to determine the aspect ratio. An example image looks like this:

 ![](https://global.discourse-cdn.com/meta/original/3X/1/d/1d7879267413aed2e3dd212bc8226c6e93874c1d.jpg)

Ideally we’d like to present this image so that the image width takes the full post width, but perhaps shows a ‘More..’ link at the bottom. The ‘image viewer overlay’ (not sure what type, mfp css classes) struggles with this format too, in that it squishes the file info at the bottom.

So yep, probably a bit unusual but thought I’d throw that out there anyway. If anyone has any simple custom CSS then we could give that a go too.

---

<div class="post-metadata">

### Author: ![lisetest](https://avatars.discourse-cdn.com/v4/letter/l/57b2e6/32.png) [@lisetest](https://meta.discourse.org/u/lisetest)
#### Post date: [May 7, 2016, 9:58pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/2 "2016-05-07T21:58:09Z")

</div>

we have the same problem-- I’d love to see this addressed

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [May 7, 2016, 10:23pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/3 "2016-05-07T22:23:59Z")

</div>

I am also interested in this @zogstrip, I think we need special handling of very tall images. Maybe the thumbnail crops them at a certain height to prevent crazy super-mega-tall thumbnails with bizarre height to width ratios, as in, way too tall?

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [May 16, 2016, 9:33pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/4 "2016-05-16T21:33:49Z")

</div>

@zogstrip is working on this but it turned out to be harder than we expected. 💪

---

<div class="post-metadata">

### Author: ![Falco](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/falco/32/179432_2.png) [@Falco](https://meta.discourse.org/u/Falco)
#### Post date: [May 17, 2016, 12:01am UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/5 "2016-05-17T00:01:48Z")

</div>

I really like the way [RES](http://redditenhancementsuite.com/) handles this.

The comics at /r/polandball are a textbook example of such tall images.

---

<div class="post-metadata">

### Author: ![zogstrip](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/zogstrip/32/512781_2.png) [@zogstrip](https://meta.discourse.org/u/zogstrip)
#### Post date: [May 23, 2016, 2:21pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/6 "2016-05-23T14:21:17Z")

</div>

Took me longer than anticipated because I went the wrong way trying to solve this. Left it on the backburner for a few days before going back to it with another approach 😉

There’s now a new site setting: `crop_tall_images` which is enabled by default and will generate a crop of tall images instead of generating a thin thumbnail.

[https://github.com/discourse/discourse/commit/667dd54a23b705d4286ac0b2a8d398c13675963b](https://github.com/discourse/discourse/commit/667dd54a23b705d4286ac0b2a8d398c13675963b)

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [May 23, 2016, 2:25pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/7 "2016-05-23T14:25:35Z")

</div>

Why ever disable this? Does it really need a site setting? It seems like such a net positive that I can’t imagine it ever needing it to be disabled?

---

<div class="post-metadata">

### Author: ![zogstrip](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/zogstrip/32/512781_2.png) [@zogstrip](https://meta.discourse.org/u/zogstrip)
#### Post date: [May 23, 2016, 2:28pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/8 "2016-05-23T14:28:44Z")

</div>

Might be a bit of imagineering on my part but there might be art/design/photo-heavy communities who might not want to crop their images and instead increased the maximum height.

It’s only 2 `if`s so it’s an easy change if we need to remove it.

_I know, **3** “might” in the same sentence is definitely leaning towards imageerining…_

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [May 23, 2016, 2:31pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/9 "2016-05-23T14:31:17Z")

</div>

I suggest we wait for that request rather than adding another site setting Because We Can.

---

<div class="post-metadata">

### Author: ![zogstrip](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/zogstrip/32/512781_2.png) [@zogstrip](https://meta.discourse.org/u/zogstrip)
#### Post date: [May 23, 2016, 2:45pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/10 "2016-05-23T14:45:37Z")

</div>

[https://github.com/discourse/discourse/commit/32d253d484327e4880c1097d03747f0919edebcb](https://github.com/discourse/discourse/commit/32d253d484327e4880c1097d03747f0919edebcb)

&nbsp;

 ![](https://global.discourse-cdn.com/meta/original/3X/1/c/1c8ab37ad0c3f5cc858ca94af9f76b6d49b08390.gif)

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [May 23, 2016, 3:24pm UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/11 "2016-05-23T15:24:16Z")

</div>

Let’s see if it works!

 ![](https://global.discourse-cdn.com/meta/original/3X/8/9/89fc4eed2d6c1561b41eda01b4bb6dfd021652db.jpeg)

Yes! Amazing! Well done @zogstrip 💋 🇫🇷 🐸 🐩

---

<div class="post-metadata">

### Author: ![erlend\_sh](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/erlend_sh/32/119475_2.png) [@erlend\_sh](https://meta.discourse.org/u/erlend_sh)
#### Post date: [May 26, 2016, 10:37am UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/14 "2016-05-26T10:37:27Z")

</div>

Something’s not right in the top image of this post:

> [@With the new right gutter, where should "Reply as Linked Topic" go?](https://meta.discourse.org/t/with-the-new-right-gutter-where-should-reply-as-linked-topic-go/44815/12):
>
> We actually have a button there already on mobile. It’s used to toggle the formatting button on/off. Here’s a quick mockup of what I’m proposing. We flip the position of these two buttons (maybe repurposing the gear icon to a vertical ellipsis), and add “Reply as linked topic” as an option in the top-right menu.

I’m confused now. Is this intended behaviour? I guess I’m clamouring for this now?

> [@zogstrip](#):
>
> Might be a bit of imagineering on my part but there might be art/design/photo-heavy communities who might not want to crop their images and instead increased the maximum height.

---

<div class="post-metadata">

### Author: ![zogstrip](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/zogstrip/32/512781_2.png) [@zogstrip](https://meta.discourse.org/u/zogstrip)
#### Post date: [May 26, 2016, 10:43am UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/15 "2016-05-26T10:43:55Z")

</div>

I’m actually tweaking the ratio used to trigger the crop. Will change it so that iPhone 6 screenshot **don’t** crop 😉

[https://github.com/discourse/discourse/commit/ebd4b4577168644ff1132af85215bfa2034ecdbb](https://github.com/discourse/discourse/commit/ebd4b4577168644ff1132af85215bfa2034ecdbb)

---

<div class="post-metadata">

### Author: ![erlend\_sh](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/erlend_sh/32/119475_2.png) [@erlend\_sh](https://meta.discourse.org/u/erlend_sh)
#### Post date: [May 26, 2016, 10:44am UTC](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/16 "2016-05-26T10:44:41Z")

</div>


