asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #15711 from poke1024/fix-rtl-descent

Browse source 
Fixes wrong vertical font layout in RichTextLabel
This commit is contained in:
Rémi Verschelde 2018-02-14 16:28:16 +01:00 committed by GitHub
commit f1ee573ebe
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 24 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
scene/gui/rich_text_label.cpp
View file

@ -121,6 +121,8 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
if (p_mode == PROCESS_CACHE) { if (p_mode == PROCESS_CACHE) {
l.offset_caches.clear(); l.offset_caches.clear();
l.height_caches.clear(); l.height_caches.clear();
l.ascent_caches.clear();
l.descent_caches.clear();
l.char_count = 0; l.char_count = 0;
l.minimum_width = 0; l.minimum_width = 0;
} }
@ -140,6 +142,8 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
//line height should be the font height for the first time, this ensures that an empty line will never have zero height and successive newlines are displayed //line height should be the font height for the first time, this ensures that an empty line will never have zero height and successive newlines are displayed
int line_height = cfont->get_height(); int line_height = cfont->get_height();
int line_ascent = cfont->get_ascent();
int line_descent = cfont->get_descent();
int nonblank_line_count = 0; //number of nonblank lines as counted during PROCESS_DRAW int nonblank_line_count = 0; //number of nonblank lines as counted during PROCESS_DRAW
@ -169,16 +173,22 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
case ALIGN_FILL: l.offset_caches.push_back((p_width - margin) - used /*+spaces_size*/); break; \ case ALIGN_FILL: l.offset_caches.push_back((p_width - margin) - used /*+spaces_size*/); break; \
} \ } \
l.height_caches.push_back(line_height); \ l.height_caches.push_back(line_height); \
l.ascent_caches.push_back(line_ascent); \
l.descent_caches.push_back(line_descent); \
l.space_caches.push_back(spaces); \ l.space_caches.push_back(spaces); \
} \ } \
y += line_height + get_constant(SceneStringNames::get_singleton()->line_separation); \ y += line_height + get_constant(SceneStringNames::get_singleton()->line_separation); \
line_height = 0; \ line_height = 0; \
line_ascent = 0; \
line_descent = 0; \
spaces = 0; \ spaces = 0; \
spaces_size = 0; \ spaces_size = 0; \
wofs = begin; \ wofs = begin; \
align_ofs = 0; \ align_ofs = 0; \
if (p_mode != PROCESS_CACHE) { \ if (p_mode != PROCESS_CACHE) { \
lh = line < l.height_caches.size() ? l.height_caches[line] : 1; \ lh = line < l.height_caches.size() ? l.height_caches[line] : 1; \
line_ascent = line < l.ascent_caches.size() ? l.ascent_caches[line] : 1; \
line_descent = line < l.descent_caches.size() ? l.descent_caches[line] : 1; \
} \ } \
if (p_mode == PROCESS_POINTER && r_click_item && p_click_pos.y >= p_ofs.y + y && p_click_pos.y <= p_ofs.y + y + lh && p_click_pos.x < p_ofs.x + wofs) { \ if (p_mode == PROCESS_POINTER && r_click_item && p_click_pos.y >= p_ofs.y + y && p_click_pos.y <= p_ofs.y + y + lh && p_click_pos.x < p_ofs.x + wofs) { \
if (r_outside) *r_outside = true; \ if (r_outside) *r_outside = true; \
@ -254,6 +264,12 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
const CharType *cf = c; const CharType *cf = c;
int fh = font->get_height(); int fh = font->get_height();
int ascent = font->get_ascent(); int ascent = font->get_ascent();
int descent = font->get_descent();
line_ascent = MAX(line_ascent, ascent);
line_descent = MAX(line_descent, descent);
fh = MAX(fh, line_ascent + line_descent); // various fonts!
Color color; Color color;
bool underline = false; bool underline = false;
@ -280,6 +296,8 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
lh = 0; lh = 0;
if (p_mode != PROCESS_CACHE) { if (p_mode != PROCESS_CACHE) {
lh = line < l.height_caches.size() ? l.height_caches[line] : 1; lh = line < l.height_caches.size() ? l.height_caches[line] : 1;
line_ascent = line < l.ascent_caches.size() ? l.ascent_caches[line] : 1;
line_descent = line < l.descent_caches.size() ? l.descent_caches[line] : 1;
} }
while (c[end] != 0 && !(end && c[end - 1] == ' ' && c[end] != ' ')) { while (c[end] != 0 && !(end && c[end - 1] == ' ' && c[end] != ' ')) {
@ -348,7 +366,7 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
int cw = 0; int cw = 0;
bool visible = visible_characters < 0 || p_char_count < visible_characters && YRANGE_VISIBLE(y + lh - (fh - 0 * ascent), fh); //getting rid of ascent seems to work?? bool visible = visible_characters < 0 || p_char_count < visible_characters && YRANGE_VISIBLE(y + lh - line_descent - line_ascent, line_ascent + line_descent);
if (visible) if (visible)
line_is_blank = false; line_is_blank = false;
@ -360,11 +378,11 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
cw = font->get_char_size(c[i], c[i + 1]).x; cw = font->get_char_size(c[i], c[i + 1]).x;
draw_rect(Rect2(p_ofs.x + pofs, p_ofs.y + y, cw, lh), selection_bg); draw_rect(Rect2(p_ofs.x + pofs, p_ofs.y + y, cw, lh), selection_bg);
if (visible) if (visible)
font->draw_char(ci, p_ofs + Point2(align_ofs + pofs, y + lh - (fh - ascent)), c[i], c[i + 1], override_selected_font_color ? selection_fg : color); font->draw_char(ci, p_ofs + Point2(align_ofs + pofs, y + lh - line_descent), c[i], c[i + 1], override_selected_font_color ? selection_fg : color);
} else { } else {
if (visible) if (visible)
cw = font->draw_char(ci, p_ofs + Point2(align_ofs + pofs, y + lh - (fh - ascent)), c[i], c[i + 1], color); cw = font->draw_char(ci, p_ofs + Point2(align_ofs + pofs, y + lh - line_descent), c[i], c[i + 1], color);
} }
p_char_count++; p_char_count++;
@ -379,7 +397,7 @@ int RichTextLabel::_process_line(ItemFrame *p_frame, const Vector2 &p_ofs, int &
if (underline) { if (underline) {
Color uc = color; Color uc = color;
uc.a *= 0.5; uc.a *= 0.5;
int uy = y + lh - fh + ascent + 2; int uy = y + lh - line_descent + 2;
float underline_width = 1.0; float underline_width = 1.0;
#ifdef TOOLS_ENABLED #ifdef TOOLS_ENABLED
underline_width *= EDSCALE; underline_width *= EDSCALE;

2
scene/gui/rich_text_label.h
View file

@ -80,6 +80,8 @@ private:
Item *from; Item *from;
Vector<int> offset_caches; Vector<int> offset_caches;
Vector<int> height_caches; Vector<int> height_caches;
Vector<int> ascent_caches;
Vector<int> descent_caches;
Vector<int> space_caches; Vector<int> space_caches;
int height_cache; int height_cache;
int height_accum_cache; int height_accum_cache;