kimcinoo
/
efl
forked from enlightenment/efl
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Canvas text: fix ellipsis RTL handling 

First, fixing ellipsis text positions: ellipsis items should be assigned the
text positions of the omitted text (while maintaining the formatting of the
last visual item). In the case where an entire item was rejected, it
will be assigned that item's text position. If an item was split, it will be
assigned the text position of the split portion.
The BiDi reorder code relies on properly-assigned text positions.

Second, fixing ellipsis handling: the width calc was only considering the
ellipsis item's width. However, if the ellipsis is placed as e.g. the first
visual item (such as in RTL cases), its advance value should've be considered,
instead.

Thanks Youngbok Shin for the test case and information.

@fix
This commit is contained in:
Daniel Hirt 2016-08-01 15:52:18 +03:00
parent d88f08f7e9
commit 2d44738fae
2 changed files with 45 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/lib/evas/canvas/evas_object_textblock.c
View File

@ -4869,14 +4869,25 @@ static inline void
_layout_handle_ellipsis(Ctxt *c, Evas_Object_Textblock_Item *it, Eina_List *i)
{
Evas_Object_Textblock_Text_Item *ti, *ellip_ti;
Evas_Object_Textblock_Item *last_it;
Evas_Coord save_cx;
Evas_Object_Textblock_Item *last_it, *prev_it;
Evas_Coord save_cx, save_cw, ellip_w;
int wrap;
ellip_ti = _layout_ellipsis_item_new(c, it);
last_it = it;
prev_it = last_it = it;
save_cx = c->x;
c->w -= ellip_ti->parent.w;
save_cw = c->w;
ellip_w = ellip_ti->parent.w;
#ifdef BIDI_SUPPORT
// XXX: with RTL considerations in mind, we need to take max(adv, w) as the
// line may be reordered in a way that the item placement will cause the
// formatted width to exceed the width constraints.
if (ellip_ti->parent.adv > ellip_ti->parent.w)
{
ellip_w = ellip_ti->parent.adv;
}
#endif
c->w -= ellip_w;
/* If there is no enough space for ellipsis item, remove all of items */
if (c->w <= 0)
@ -4901,6 +4912,7 @@ _layout_handle_ellipsis(Ctxt *c, Evas_Object_Textblock_Item *it, Eina_List *i)
if ((wrap > 0) && !IS_AT_END(ti, (size_t) wrap))
{
Evas_Object_Textblock_Text_Item *new_ti;
Eina_List *l = i;
while (l)
@ -4909,12 +4921,14 @@ _layout_handle_ellipsis(Ctxt *c, Evas_Object_Textblock_Item *it, Eina_List *i)
if (iit == _ITEM(ti)) break;
l = eina_list_prev(l);
}
_layout_item_text_split_strip_white(c, ti, l, wrap);
new_ti = _layout_item_text_split_strip_white(c, ti, l, wrap);
ellip_ti->parent.text_pos = new_ti->parent.text_pos;
break;
}
else if (wrap < 0)
{
// Removal of the previous item left enough space.
ellip_ti->parent.text_pos = prev_it->text_pos;
break;
}
}
@ -4936,6 +4950,7 @@ _layout_handle_ellipsis(Ctxt *c, Evas_Object_Textblock_Item *it, Eina_List *i)
EINA_INLIST_GET(last_it)));
}
prev_it = last_it;
last_it = (c->ln->items) ? _ITEM(EINA_INLIST_GET(c->ln->items)->last) : NULL;
if (last_it)
@ -4954,7 +4969,7 @@ _layout_handle_ellipsis(Ctxt *c, Evas_Object_Textblock_Item *it, Eina_List *i)
}
c->x = save_cx;
c->w += ellip_ti->parent.w;
c->w = save_cw;
/* If we should add this item, do it */
if (last_it == it)
{

23
src/tests/evas/evas_test_textblock.c
View File

@ -2259,6 +2259,29 @@ START_TEST(evas_textblock_wrapping)
evas_object_textblock_size_formatted_get(tb, &w, NULL);
ck_assert_int_eq(bw, w);
#ifdef HAVE_FRIBIDI
/* Check the ellipsis is placed at proper place
* in RTL text with formats */
evas_object_textblock_text_markup_set(tb, ")");
evas_object_textblock_size_native_get(tb, &bw, NULL);
bw++;
/* Expect to see: "...)ي" */
evas_object_textblock_text_markup_set(tb, "ي(ي)");
evas_textblock_cursor_format_prepend(cur, "+ ellipsis=1.0");
evas_object_textblock_size_native_get(tb, &nw, &nh);
evas_object_resize(tb, nw - bw, nh);
evas_object_textblock_size_formatted_get(tb, &bw, NULL);
/* Expect to see: "...)ي"
* But, Evas Textblock could put ellipsis item at wrong place: ")...ي"
* Then, formatted size could be different from the case without format. */
evas_object_textblock_text_markup_set(tb, "ي<color=#f00>(</color>ي)");
evas_textblock_cursor_format_prepend(cur, "+ ellipsis=1.0");
evas_object_textblock_size_formatted_get(tb, &w, NULL);
ck_assert_int_eq(bw, w);
#endif
END_TB_TEST();
}
END_TEST