# Common image uploads sizes cropped

**URL:** https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849
**Category:** Bug
**Created:** [May 26, 2016, 4:38pm UTC](https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849 "2016-05-26T16:38:40Z")
**Posts on this page:** 6
**Page:** 1

<div class="post-metadata">

### Author: ![DeanMarkTaylor](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/deanmarktaylor/32/102462_2.png) [@DeanMarkTaylor](https://meta.discourse.org/u/DeanMarkTaylor)
#### Post date: [May 26, 2016, 4:38pm UTC](https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849/1 "2016-05-26T16:38:40Z")

</div>

It’s very typical for users to upload images which are screenshots taken on mobile.

Currently the recent changes prevent these mobile screenshots from displaying in a reasonable way.

Cropping to the top of the image is undesirable, unexpected and considered “broken” by users.

## Expected

 ![](https://global.discourse-cdn.com/meta/original/3X/e/d/ed33f1af3ae7729de02e69c002ada69c03dbe585.png)

## Actual

 ![](https://global.discourse-cdn.com/meta/original/3X/5/7/57e1b41137f635bd7e6ab4a7efe6d818ddbe9ab8.png)

## Sample image here on meta

 ![](https://global.discourse-cdn.com/meta/original/3X/9/3/93e76274c75e5ae0828ae0a9d60203fc49797d63.png)

## Additional notes

Please also consider adding a visual indicator on the smaller image when it has been cropped.

---

<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, 4:47pm UTC](https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849/2 "2016-05-26T16:47:57Z")

</div>

I [noticed this earlier today](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/15) and just pushed a fix 😉

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

* * *

> [@DeanMarkTaylor](#):
>
> ### Additional notes
> 
> Please also consider adding a visual indicator on the smaller image when it has been cropped.

I’m open for suggestions 😉

---

<div class="post-metadata">

### Author: ![cpradio](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/cpradio/32/4970_2.png) [@cpradio](https://meta.discourse.org/u/cpradio)
#### Post date: [May 26, 2016, 5:05pm UTC](https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849/3 "2016-05-26T17:05:31Z")

</div>

> [@zogstrip](#):
>
> I’m open for suggestions 😉

Maybe the crop font-awesome icon?

> **[Font Awesome](https://fontawesome.com/)**
>
> The internet's icon library + toolkit. Used by millions of designers, devs, & content creators. Open-source. Always free. Always awesome.

I’m just unsure if you know at the time the lightbox runs that it was cropped versus resized…

---

<div class="post-metadata">

### Author: ![DeanMarkTaylor](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/deanmarktaylor/32/102462_2.png) [@DeanMarkTaylor](https://meta.discourse.org/u/DeanMarkTaylor)
#### Post date: [May 26, 2016, 5:07pm UTC](https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849/4 "2016-05-26T17:07:51Z")

</div>

> [@DeanMarkTaylor](#):
>
> Please also consider adding a visual indicator on the smaller image when it has been cropped.

> [@zogstrip](#):
>
> I’m open for suggestions

Perhaps something like this which is always displayed on a cropped image:

 ![](https://global.discourse-cdn.com/meta/original/3X/c/5/c5a3c12ef8205a605e96d64fff3a7f3ddcbe3240.png)

* * *

Please also note that for extremely tall images clicking the image and then showing full-screen is a poor solution…

… mainly because you are unable to scroll and you are still limited by the height of the users screen.

Note the result of clicking the image in the first post of the feature request:

> [@Feature Request: Tall and Thin Image Handling](https://meta.discourse.org/t/feature-request-tall-and-thin-image-handling/43573/1):
>
> 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: 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’ (…

---

<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, 5:09pm UTC](https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849/5 "2016-05-26T17:09:08Z")

</div>

> [@DeanMarkTaylor](#):
>
> Please also note that for extremely tall images clicking the image and then showing full-screen is a poor solution…
> 
> … mainly because you are unable to scroll and you are still limited by the height of the users screen.
> 
> Note the result of clicking the image in the first post of the feature request:

If you click on the image once it’s full-screen, the image will be “real size” and you’ll be able to scroll 😉

---

<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 26, 2016, 7:21pm UTC](https://meta.discourse.org/t/common-image-uploads-sizes-cropped/44849/6 "2016-05-26T19:21:32Z")

</div>


