Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ship SBIX emoji fallback font #9900

Closed
jryans opened this issue May 30, 2019 · 7 comments
Closed

Ship SBIX emoji fallback font #9900

jryans opened this issue May 30, 2019 · 7 comments

Comments

@jryans
Copy link
Collaborator

jryans commented May 30, 2019

We're seeing some configurations (#9891 and elsewhere in chat) that don't support COLR fonts, leading to a bad emoji experience, especially if you also don't have a native font either.

We should be able to remedy this by also shipping a SBIX variant of our emoji font, since that format has wider support.

@lampholder
Copy link
Member

Are we actually moving forward with this? Is the first step for Matthew to fork the mozilla script to make it also generate SBIX? Is that something somebody other than Matthew could do?

@ara4n
Copy link
Member

ara4n commented May 31, 2019

i've already forked the script to fix mozilla/twemoji-colr#50. i would be working on it right now if i hadn't been hijacked. someone else could go and add the thing to the script to blat out a PNG and change the TTX to point to the the PNG in a SBIX, probably cribbing off https://github.com/RoelN/ChromaCheck/blob/master/src/chromacheck-sbix.ttx as an example of what SBIX TTX looks like.

@ara4n
Copy link
Member

ara4n commented May 31, 2019

and yes, we should absolutely move fwd on it.

@jryans
Copy link
Collaborator Author

jryans commented May 31, 2019

Is your patch in mozilla/twemoji-colr#50 literally the only change so far? (Should we fork the font into our org and continue from there?)

@ara4n
Copy link
Member

ara4n commented May 31, 2019

yup and yup.

@ara4n
Copy link
Member

ara4n commented Jun 2, 2019

should be fixed by matrix-org/matrix-react-sdk#3050

@jryans
Copy link
Collaborator Author

jryans commented Jun 3, 2019

matrix-org/matrix-react-sdk#3050 has merged to develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants