From aa8561ded403020f0292450e7a7af75eaf001e94 Mon Sep 17 00:00:00 2001 From: Ruslan Mustakov Date: Wed, 14 Mar 2018 19:38:34 +0700 Subject: Ceil dynamic font glyph size Fixes #15459. When oversampling is enabled, glyphs may have fractional size, but they are still rendered into integral pixels, which results in them taking more space than was anticiped by autowrapping algorithm. The solution here is to return ceiled width, which makes autowrapper consider characters a bit larger than they are, but it doesn't hurt the actual rendering and ensures there is enough space for the characters. --- scene/resources/dynamic_font.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scene/resources/dynamic_font.cpp b/scene/resources/dynamic_font.cpp index 26e29b3ccb..3d825b5a27 100644 --- a/scene/resources/dynamic_font.cpp +++ b/scene/resources/dynamic_font.cpp @@ -308,6 +308,9 @@ Size2 DynamicFontAtSize::get_char_size(CharType p_char, CharType p_next, const V ret.x += (delta.x >> 6) / oversampling; } } + + // ensures oversampled glyphs will have enough space when this value is used by clipping/wrapping algorithms + ret.x = Math::ceil(ret.x); return ret; } -- cgit v1.2.3