1090692 - Advance width of Emoji in RTL text is doubled (Firefox for OS X)

Status: RESOLVED DUPLICATE of bug 923007

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

Description

[Ryusei Yamaguchi]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:33.0) Gecko/20100101 Firefox/33.0
Build ID: 20141011015303

Steps to reproduce:

Display an emoji in RTL (right-to-left) text (e.g. "قط

Comment 1

[Ryusei Yamaguchi]

Attachment: rtl_emoji_bug.txt

It doesn't seems to be able to submit messages that contain astral code points to the bugzilla, so I attach the original post.

Comment 2

[Ryusei Yamaguchi]

Attachment: rtl_emoji_bug.txt

correct the encoding specification

Comment 3

[Jonathan Kew [:jfkthame]]

I can reproduce this (on OS X 10.9) using Firefox 33, but not with current Nightly; as such, it looks like it's already fixed on trunk. It would be helpful to identify exactly when it got fixed, so we can consider whether something should be uplifted to earlier releases.

Comment 4

[Jonathan Kew [:jfkthame]]

I'm guessing it may have been bug 923007 that fixed this, but have not verified that.

Comment 5

[:Gijs (he/him)]

The 'progression' pushlog for this fix is:

https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=818f353b7aa6&tochange=1735ff2bb23e

which corroborates the guess of bug 923007. That means this will be fixed in Firefox 35. Jonathan, is that change low-risk enough to request uplift to 34? :-)

Comment 6

[Jonathan Kew [:jfkthame]]

IMO, this is a relatively minor cosmetic flaw, so I think I'd be happier leaving that one to ride the trains and get its full period of aurora and beta testing. I believe the patch to be OK (otherwise I wouldn't have written it!), but we're dealing here with our interaction with the dark internals of Core Text behavior, and potentially could disrupt the shaping of other Apple fonts if something unexpected happens.

Comment 7

[:Gijs (he/him)]

(In reply to Jonathan Kew (:jfkthame) from comment #6)
> IMO, this is a relatively minor cosmetic flaw, so I think I'd be happier
> leaving that one to ride the trains and get its full period of aurora and
> beta testing. I believe the patch to be OK (otherwise I wouldn't have
> written it!), but we're dealing here with our interaction with the dark
> internals of Core Text behavior, and potentially could disrupt the shaping
> of other Apple fonts if something unexpected happens.

Makes sense. Marking as a duplicate, then...