gitmoji icon indicating copy to clipboard operation
gitmoji copied to clipboard

Add a custom font to be served by the gitmoji website

Open kevin-dedaniya-rcrm opened this issue 4 years ago • 18 comments

Original title: Adhesive Bandage(🩹) emoji not working properly on Ubuntu 18.04

kevin-dedaniya-rcrm avatar Nov 22 '20 10:11 kevin-dedaniya-rcrm

Yeah, this also happens for me, does anyone know a font that has support for this emoji?

vhoyer avatar Nov 23 '20 14:11 vhoyer

😱 Based on the spec it was added on 2019, so there should be some font that includes that as an emoji 😓

carloscuesta avatar Nov 23 '20 14:11 carloscuesta

Yeah, personally, I have a problem with emoji fonts on my system, so the gitmoji site does a fallback through the fonts and, for me, uses the Noto Color Emoji, and apparently, this file does not have the adhesive bandage emoji... :confused:. (I also didn't find any font that could render this emoji either, but I think I don't know how to search for it ...)

vhoyer avatar Nov 23 '20 15:11 vhoyer

Should a requirement for a new gitmoji to be added to the spec require a certain availability of the emoji on common fonts in OSs?

For example, if the default font with Ubuntu 20.04 LTS (gnome) does not support it, should that disqualify the proposed gitmoji? (this gitmoji would not apply in that case)

vegerot avatar Nov 24 '20 15:11 vegerot

Previously I was very up-to-date on the Emojis releases and I rejected some proposals due to incompatibility or being to new. However I did not thought about this for that specific case 😓 .

I don't think the rules should be based on the default font of any system. We should base the criteria on the emoji support that GitHub and GitLab have, because under the hood they convert emoji characters to an image:

Screenshot 2020-11-24 at 16 50 33

carloscuesta avatar Nov 24 '20 15:11 carloscuesta

Ok, I have to say that I simply updated my browser and now I have support for the missing emoji :eyes:

vhoyer avatar Nov 24 '20 19:11 vhoyer

actually, more than that, I also redownloaded Noto Color Emoji file and reinstalled onto my fonts and restarted my computer, I'm not really sure what part of it was a solution to this, but there it is.

Hey, @carloscuesta, what do think if we ship a fallback font for emojis alongside the website?

vhoyer avatar Nov 24 '20 19:11 vhoyer

We can try to serve a fallback font at the website, or just try the image fallback, I will take a look into it. However emojis not being rendered on the website is not a "big deal" compared to introducing emojis that are not supported by GitHub.

carloscuesta avatar Nov 24 '20 19:11 carloscuesta

Or even a default emoji font, this way we can choose colors with good contrast between the default font and the emoji background, which is not happening here, for example:

card card with the emoji text selected
image image

vhoyer avatar Nov 24 '20 19:11 vhoyer

Yes! Not a big fan though to depend on external fonts as the main resource for perf reasons but will take a look into it.

carloscuesta avatar Nov 24 '20 20:11 carloscuesta

I'm still considering options for this! I've found a font (which is not released yet) that we're going to implement it as soon as they came out!

Just for reference, here's what I found – https://sensa.co/emoji/ I think it is really beautiful 😍 . Signed up for the waitlist a week ago

As soon as they release the font I will work on this issue 😊

carloscuesta avatar Dec 10 '20 21:12 carloscuesta

They released a version of the font however it's missing a ton of emojis that we currently have on the website 😅

carloscuesta avatar Nov 03 '21 21:11 carloscuesta

shame :( should we wait more or maybe choose another font?

vhoyer avatar Nov 05 '21 15:11 vhoyer

Not sure to be honest, 😅 after thinking about it for some time, I don't know if using different emojis on the website vs the ones that are used in GitHub is a great idea, as it can cause some kind of confusions

carloscuesta avatar Nov 06 '21 11:11 carloscuesta

how about this then? https://github.com/github/g-emoji-element

vhoyer avatar Nov 08 '21 20:11 vhoyer

Yeah I remember I saw it, but I think I discarded it because they don't have SVG support?

carloscuesta avatar Nov 08 '21 21:11 carloscuesta

Why we need svg support? does those images have fixed tiny size? like 25x25?

vhoyer avatar Nov 10 '21 18:11 vhoyer

Hey! Mainly for quality and performance 😊

carloscuesta avatar Nov 15 '21 19:11 carloscuesta

Unsure if we still want to improve this, but I re-found this issue, so I can share thoughts following my latest open source emoji exploration.

TL;DR - OpenMoji or JoyPixels (free license)

But +1 to the "website maybe should use native emoji given native viewing on e.g. GitHub

connorjs avatar Sep 30 '23 21:09 connorjs

Thanks for bringing this up!

I believe that your point is quite fair and given that there's no usage of any font in GitHub sticking to the native one might be the best for our users

It woulf be confusing for me to have different emojis if we used a custom font.

I'll close thin issue 😍

carloscuesta avatar Oct 01 '23 10:10 carloscuesta