Comment 60 for bug 39890

Revision history for this message
In , Albert Astals Cid (aacid) wrote :

(In reply to comment #32)
> The difference in character width is because of this change:
>
> @@ -863,7 +839,7 @@ void TextLine::coalesce(UnicodeMap *uMap) {
> word0->spaceAfter = gTrue;
> word0 = word1;
> word1 = word1->next;
> - } else if (word0->font == word1->font &&
> + } else if (word0->font[word0->len - 1] == word1->font[0] &&
> word0->underlined == word1->underlined &&
> fabs(word0->fontSize - word1->fontSize) <
> maxWordFontSizeDelta * words->fontSize &&
>
>
> Before, the words "Fumi" and "K" were being merged. Letters "F" and "K" are
> using the font BookmanOldStyle. Letters "umi" are using ArialMT. Because of
> this font check, the words are no longer being merged, making the width of the
> i different.

I understand why they are not merged (by code not, why the code is put there), but why the "i" has a different width that makes it overlap with the "K" when previously didn't?