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

Doesn't render compound emoji correctly. Example: 🧘‍♂️ #1371

Closed
dwt opened this issue Mar 29, 2018 · 3 comments
Closed

Doesn't render compound emoji correctly. Example: 🧘‍♂️ #1371

dwt opened this issue Mar 29, 2018 · 3 comments

Comments

@dwt
Copy link

dwt commented Mar 29, 2018

Wire version: Version 3.1.2822
Wire for web version: Wire für Web Version 2018.03.12.1141
Operating system: Darwin crest.fritz.box 17.4.0 Darwin Kernel Version 17.4.0: Sun Dec 17 09:19:54 PST 2017; root:xnu-4570.41.2~1/RELEASE_X86_64 x86_64
Which antivirus software do you have installed: none

What steps will reproduce the problem?

  1. Get a message with 🧘‍♂️
  2. Look at it
  3. Wonder about the strange rendering

What is the expected result?
🧘‍♂️

What is the actual result?
🧘‍♀️ ♂

Please provide any additional information below. Attach a screenshot if
possible.
bildschirmfoto 2018-03-29 um 09 36 17

@maximbaz
Copy link
Contributor

It's a bug in Chromium (the engine behind Wire Desktop), or in fact a regression. I found this issue back in July, I confirmed that the issue was fixed in November, but now the issue is back again 😞

Emojis look fine in Firefox.


In fact, this is how your bug report looks in Chromium as of today:

image

And this is how it looks in Firefox:

image

@gregor
Copy link
Contributor

gregor commented Mar 29, 2018

@maximbaz Please keep in mind that our desktop apps are based on electron. These do not include the latest Chromium version at this point in time.. You can always see those version numbers at https://github.com/wireapp/wire-desktop/wiki/Builds

@gregor
Copy link
Contributor

gregor commented Jun 1, 2018

Hopefully we can soon ship desktop versions based on newer versions of electron. Our current internal builds are based on 1.8.7 as the 2.x branch still sees some compatibility issues. This will at least bring us Chrome 59.

@gregor gregor closed this as completed Jun 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants