Uploaded Emoji are a bit desaturated?


(Ionuț Staicu) #1

Hi guys. I tried to upload a new emoji but both size and colors seemed to be a bit off. So i tried the following:

  • downloaded an existing emoji (that is bundled into Discourse)
  • tried to re-upload

After that, i opened both images in new tabs (so there is no css involved) and I have this result:

(left image is the uploaded one)


(Régis Hanol) #2

We downsize uploaded emoji to make sure they fit in a 60x60 square.

Maybe one of the Image Magick option is altering the emoji?


(Ionuț Staicu) #3

But… Why? Since the default (apple) emoji are 64x64 and emoji one (i think) are 74x74, it wouldn’t make more sense to have them resized to the current emoji scheme base size? :confused:


(Jeff Atwood) #4

This might be related to Image Magick messing up black and white images too.

Maybe Image could do with a little less Magicks


(Jens Maier) #5

What, like ImageScience? :smirk:

In all seriousness, tho, I’m guessing that the -thumbnail operation is a bad choice. It focuses filesize over everything else, and while this may be fine for actual thumbnails, it’s not desireable for emojis (which are small enough anyways).


(Régis Hanol) #6

Thanks @iamntz for reporting that issue. I updated the code so that we only downsize to 100x100 using the resize option instead of thumbnail. :beetle:

https://github.com/discourse/discourse/commit/a87083fa7641267339e06a77773601594570d0fd


(Jeff Atwood) #9

@zogstrip can you verify we don’t have any lingering issues here? Maybe upload emoji to test?

Here’s one:


(Régis Hanol) #10

Here’s another 2.

They look fine to me.


(Régis Hanol) #11