From 552895da60ebb30d81712a7c81718cda6ab73ef6 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Sat, 4 Mar 2023 20:54:25 +0100 Subject: [PATCH] LibGfx: Skip old-style emoji lookup for fonts that have color bitmaps Ultimately, we should find a way to route all emoji access through the font code, but for now, this patch adds a special case for fonts that are known to have embedded color bitmaps so we can test them. --- Userland/Libraries/LibGfx/Font/ScaledFont.cpp | 6 ++++-- Userland/Libraries/LibGfx/Painter.cpp | 3 ++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/Userland/Libraries/LibGfx/Font/ScaledFont.cpp b/Userland/Libraries/LibGfx/Font/ScaledFont.cpp index d28b0092c5..ccdc85529e 100644 --- a/Userland/Libraries/LibGfx/Font/ScaledFont.cpp +++ b/Userland/Libraries/LibGfx/Font/ScaledFont.cpp @@ -107,8 +107,10 @@ float ScaledFont::glyph_width(u32 code_point) const template static float glyph_or_emoji_width_impl(ScaledFont const& font, CodePointIterator& it) { - if (auto const* emoji = Emoji::emoji_for_code_point_iterator(it)) - return font.pixel_size() * emoji->width() / emoji->height(); + if (!font.has_color_bitmaps()) { + if (auto const* emoji = Emoji::emoji_for_code_point_iterator(it)) + return font.pixel_size() * emoji->width() / emoji->height(); + } return font.glyph_width(*it); } diff --git a/Userland/Libraries/LibGfx/Painter.cpp b/Userland/Libraries/LibGfx/Painter.cpp index 7b3cb903ee..24463739c5 100644 --- a/Userland/Libraries/LibGfx/Painter.cpp +++ b/Userland/Libraries/LibGfx/Painter.cpp @@ -1421,8 +1421,9 @@ void Painter::draw_glyph_or_emoji(FloatPoint point, Utf8CodePointIterator& it, F ++it; }; + // NOTE: We don't check for emoji auto font_contains_glyph = font.contains_glyph(code_point); - auto check_for_emoji = Unicode::could_be_start_of_emoji_sequence(it, font_contains_glyph ? Unicode::SequenceType::EmojiPresentation : Unicode::SequenceType::Any); + auto check_for_emoji = !font.has_color_bitmaps() && Unicode::could_be_start_of_emoji_sequence(it, font_contains_glyph ? Unicode::SequenceType::EmojiPresentation : Unicode::SequenceType::Any); // If the font contains the glyph, and we know it's not the start of an emoji, draw a text glyph. if (font_contains_glyph && !check_for_emoji) {