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

emoji to emotes parsing is weird #6522

Closed
zeratax opened this issue Apr 13, 2018 · 2 comments
Closed

emoji to emotes parsing is weird #6522

zeratax opened this issue Apr 13, 2018 · 2 comments
Labels
A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@zeratax
Copy link

zeratax commented Apr 13, 2018

Description

Currently the way emojis are being replaced with the images for emojione is very unsatisfying.

It seems that only emojis created before Unicode 10 are being replaced using the 2.2 version of emojione.
Furthermore vs15 is being ignored.

Steps to reproduce

Sending an orange heart 🧡 will fall back to system default instead of the expected emojione image.
Meaning any phone below android 8 or ios11 will not display anything
and will also look inconsistent compared to e.g. purple heart 💜

If I send ❤︎ (U+2764 ❤ + U+FE0E VARIATION SELECTOR-15) it will be displayed as the emojione 2.2 version of the red heart ❤️ emoji instead of the Heavy Black Heart ❤︎ pictogram.

Version information

  • Platform: web

For the web app:

  • Browser: Chrome
  • OS: Arch Linux
  • URL: riot.im/develop
@lampholder lampholder added T-Defect P2 S-Minor Impairs non-critical functionality or suitable workarounds exist A-Timeline type:composer labels Apr 16, 2018
@turt2live
Copy link
Member

I think this is effectively solved by #9281 because emojione won't interfere for much longer.

@zeratax
Copy link
Author

zeratax commented May 30, 2020

vs15 is btw still being ignored fwiw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

3 participants