-
Notifications
You must be signed in to change notification settings - Fork 411
MSC4347: Emoji verification images #4347
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
base: main
Are you sure you want to change the base?
Conversation
Related: matrix-org/matrix-spec#13 |
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.
Implementation requirements:
- Usage of emoji images in 1 (or ideally more) clients
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.
The element clients already use twemoji (with the exception of EXI (element-hq/element-x-ios#2242)), so maybe there is nothing to do here?
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.
it'd still be nice to check the "can in fact pull images from a JSON file" box if we can, I think. Especially to understand if client teams expect the spec to be the CDN for those images, or if they'll cache them locally (if they cache them locally, we may want to inline the images rather than URL them)
* [noto-emoji](https://github.com/googlefonts/noto-emoji) | ||
|
||
but twemoji is already in widespread use in Matrix clients, so seems the | ||
least disruptive alternative. |
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.
Does anyone other than Element even use twemoji? It's been dead for years
For reference, Beeper is standardizing on Noto color emoji for verification and some other areas
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.
@ara4n as the person who wanted to mandate twemoji in matrix-org/matrix-spec#13, any insights here?
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.
Cinny also use Tweomji.
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.
While the main twemojj repo is dead, jacked’s fork is still active, and used by Discord iirc
Rendered
Proposal to supply exact images to be used within emoji verification, to avoid confusion for users where clients use different emoji fonts.
I am a member of the Element Crypto team, but I am proposing this as an individual member of the Matrix community.