1430591 - Fitzpatrick modifier is not correctly applied with Apple Color Emoji

Status: UNCONFIRMED

(Core :: Layout: Text and Fonts, defect, P3)

Description

[İsmail Dönmez]

Attachment: fitzpatrick.html

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:59.0) Gecko/20100101 Firefox/59.0
Build ID: 20100101

Steps to reproduce:

- Set font.name-list.emoji to "Apple Color Emoji" and make sure the font is available.
- Go to https://eosrei.github.io/emojione-color-font/full-demo.html
- Scroll to Pale Emojis - Fitzpatrick Skin Type 1-2



Actual results:

See screenshot: https://screenshots.firefox.com/QYU44iNuL0e3oShf/eosrei.github.io


Expected results:

Here is the correct rendering with EmojiOne Mozilla font: https://screenshots.firefox.com/ZH8WHGyczyjQEzvd/eosrei.github.io

Comment 1

[Makoto Kato [:m_kato]]

That emoji code point can render another font such as sans-serif's default (Osaka, Times etc), gfxFontGroup::FindFontForChar won't select eFontPrefLang_Emoji since WhichPrefFontSupportsChar isn't called.

And FindFontForChar won't check whether current font in list is color font.  When I check Blink and WebKit, they ignore monochrome font when emoji presentation.

Comment 2

[Jonathan Kew [:jfkthame]]

Is the Apple Color Emoji font legitimately available for non-macOS platforms? If not, this seems pretty low priority.

Comment 3

[Jonathan Kew [:jfkthame]]

(I assume, though haven't inspected in detail, that Apple Color Emoji uses the 'morx' table to implement the fitzpatrick modifiers and other emoji sequences, and we don't support 'morx' on any platform except macOS, via the Core Text backend.)

Comment 4

[İsmail Dönmez]

(In reply to Jonathan Kew (:jfkthame) from comment #2)
> Is the Apple Color Emoji font legitimately available for non-macOS
> platforms? If not, this seems pretty low priority.

I don't see a problem using it on Linux on Mac hardware.

Comment 5

[Makoto Kato [:m_kato]]

(In reply to Jonathan Kew (:jfkthame) from comment #2)
> Is the Apple Color Emoji font legitimately available for non-macOS
> platforms? If not, this seems pretty low priority.

I can reproduce this on Android 8.1 with Noto Color Emoji.