Skip to content

Commit

Permalink
Readd support for kerning in DynamicFont
Browse files Browse the repository at this point in the history
DynamicFont kerning was removed in
a9b7843.

This readds kerning support and fixes it at the same time.

Co-authored-by: follower <[email protected]>
  • Loading branch information
Calinou and follower committed Jun 7, 2021
1 parent 6580166 commit c5ed95a
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 0 deletions.
16 changes: 16 additions & 0 deletions scene/resources/dynamic_font.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,18 @@ const Pair<const DynamicFontAtSize::Character *, DynamicFontAtSize *> DynamicFon
return Pair<const Character *, DynamicFontAtSize *>(chr, const_cast<DynamicFontAtSize *>(this));
}

float DynamicFontAtSize::_get_kerning_advance(const DynamicFontAtSize *font, CharType p_char, CharType p_next) const {
float advance = 0.0;

if (p_next) {
FT_Vector delta;
FT_Get_Kerning(font->face, FT_Get_Char_Index(font->face, p_char), FT_Get_Char_Index(font->face, p_next), FT_KERNING_DEFAULT, &delta);
advance = (delta.x / 64.0) / oversampling;
}

return advance;
}

Size2 DynamicFontAtSize::get_char_size(CharType p_char, CharType p_next, const Vector<Ref<DynamicFontAtSize>> &p_fallbacks) const {
if (!valid) {
return Size2(1, 1);
Expand All @@ -244,13 +256,15 @@ Size2 DynamicFontAtSize::get_char_size(CharType p_char, CharType p_next, const V

Pair<const Character *, DynamicFontAtSize *> char_pair_with_font = _find_char_with_font(p_char, p_fallbacks);
const Character *ch = char_pair_with_font.first;
DynamicFontAtSize *font = char_pair_with_font.second;
ERR_FAIL_COND_V(!ch, Size2());

Size2 ret(0, get_height());

if (ch->found) {
ret.x = ch->advance;
}
ret.x += _get_kerning_advance(font, p_char, p_next);

return ret;
}
Expand Down Expand Up @@ -332,6 +346,8 @@ float DynamicFontAtSize::draw_char(RID p_canvas_item, const Point2 &p_pos, CharT
advance = ch->advance;
}

advance += _get_kerning_advance(font, p_char, p_next);

return advance;
}

Expand Down
1 change: 1 addition & 0 deletions scene/resources/dynamic_font.h
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,7 @@ class DynamicFontAtSize : public Reference {

const Pair<const Character *, DynamicFontAtSize *> _find_char_with_font(CharType p_char, const Vector<Ref<DynamicFontAtSize>> &p_fallbacks) const;
Character _make_outline_char(CharType p_char);
float _get_kerning_advance(const DynamicFontAtSize *font, CharType p_char, CharType p_next) const;
TexturePosition _find_texture_pos_for_glyph(int p_color_size, Image::Format p_image_format, int p_width, int p_height);
Character _bitmap_to_character(FT_Bitmap bitmap, int yofs, int xofs, float advance);

Expand Down

0 comments on commit c5ed95a

Please sign in to comment.