-
Notifications
You must be signed in to change notification settings - Fork 5
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
create SBIX font #1
base: master
Are you sure you want to change the base?
Conversation
The only remaining problem I can see with this is that some ligatures inexplicably don't take effect - e.g.
However, this is broken on both COLR & sbix, and also seems to be Chrome specific (Safari gets it right, somehow). |
(should this not be contributed upstream rather than here?) |
judging by how rapidly mozilla#50 is getting merged, i'm not holding my breath for them to accept an invasive change like this for implementing sbix... |
yup :| Do we already ship this? |
yup, it’s the fallback font used for old freetype browsers in riot-web |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should merge this then - seems to work well in practice.
fixes element-hq/element-web#9895 VS-16 list taken from https://unicode.org/emoji/charts/emoji-variants.html will need to be kept in sync with latest emoji
No description provided.