summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authora.srour <a.srour@samsung.com>2020-01-16 17:53:36 +0900
committerWooHyun Jung <wh0705.jung@samsung.com>2020-01-16 17:53:36 +0900
commit05bb02cbb021e97bbb4caf21af756ee15f144e9d (patch)
treebada50df5f799b95d9017d42f5937b9be1f6b4ab
parent3a4c5cf6a5f36dcdf27b2f2426d8e955a68e6d9e (diff)
downloadefl-05bb02cbb021e97bbb4caf21af756ee15f144e9d.tar.gz
Efl.Ui.Textbox: Fix Ctrl+Backspace & Ctrl+Delete keys
Summary: This should fix the behaviour of Ctrl+Backspace & Ctrl+Delete key down on `Efl.Ui.Textbox`. Also added two tests for it. Reviewers: woohyun, ali.alzyod, cedric, bu5hm4n Reviewed By: woohyun Subscribers: ProhtMeyhet, stefan_schmidt, segfaultxavi, bu5hm4n, #committers, #reviewers Tags: #efl Differential Revision: https://phab.enlightenment.org/D11040
-rw-r--r--src/lib/elementary/efl_ui_internal_text_interactive.c4
-rw-r--r--src/tests/elementary/efl_ui_test_text.c38
2 files changed, 40 insertions, 2 deletions
diff --git a/src/lib/elementary/efl_ui_internal_text_interactive.c b/src/lib/elementary/efl_ui_internal_text_interactive.c
index 3851aa82a1..8e1ad98c6a 100644
--- a/src/lib/elementary/efl_ui_internal_text_interactive.c
+++ b/src/lib/elementary/efl_ui_internal_text_interactive.c
@@ -1215,7 +1215,7 @@ _key_down_cb(void *data EINA_UNUSED, Evas *e EINA_UNUSED, Evas_Object *obj, void
// del to start of previous word
Efl_Text_Cursor *tc = efl_canvas_textblock_cursor_create(obj);
- _cur_pos_copy(tc, cur);
+ _cur_pos_copy(cur, tc);
efl_text_cursor_move(cur, EFL_TEXT_CURSOR_MOVE_TYPE_CHAR_PREV);
efl_text_cursor_move(cur, EFL_TEXT_CURSOR_MOVE_TYPE_WORD_START);
@@ -1250,7 +1250,7 @@ _key_down_cb(void *data EINA_UNUSED, Evas *e EINA_UNUSED, Evas_Object *obj, void
// del to end of next word
Efl_Text_Cursor *tc = efl_canvas_textblock_cursor_create(obj);
- _cur_pos_copy(tc, cur);
+ _cur_pos_copy(cur, tc);
efl_text_cursor_move(cur, EFL_TEXT_CURSOR_MOVE_TYPE_WORD_END);
efl_text_cursor_move(cur, EFL_TEXT_CURSOR_MOVE_TYPE_CHAR_NEXT);
diff --git a/src/tests/elementary/efl_ui_test_text.c b/src/tests/elementary/efl_ui_test_text.c
index 49ba50d91d..a49c7e7cc9 100644
--- a/src/tests/elementary/efl_ui_test_text.c
+++ b/src/tests/elementary/efl_ui_test_text.c
@@ -178,6 +178,43 @@ EFL_START_TEST(text_change_event)
}
EFL_END_TEST
+EFL_START_TEST(text_keys_handler)
+{
+ Eo *txt;
+ Eo *win = win_add();
+
+ txt = efl_add(EFL_UI_TEXTBOX_CLASS, win);
+ efl_gfx_entity_size_set(txt, EINA_SIZE2D(300, 300));
+ efl_text_set(txt, "Hello");
+
+ efl_gfx_entity_visible_set(txt, EINA_TRUE);
+ Evas *e = evas_object_evas_get(txt);
+ efl_ui_focus_util_focus(txt);
+
+ efl_text_cursor_move(efl_text_interactive_main_cursor_get(txt), EFL_TEXT_CURSOR_MOVE_TYPE_LAST);
+ evas_key_modifier_on(e, "Control");
+ evas_event_feed_key_down(e, "BackSpace", "BackSpace", "\b", "\b", time(NULL), NULL);
+ ecore_main_loop_iterate();
+ ck_assert_str_eq(efl_text_get(txt),"");
+ ck_assert_int_eq(efl_text_cursor_position_get(efl_text_interactive_main_cursor_get(txt)), 0);
+ evas_event_feed_key_up(e, "BackSpace", "BackSpace", "\b", "\b", time(NULL), NULL);
+ ecore_main_loop_iterate();
+
+ efl_text_set(txt, "Hello");
+ efl_text_cursor_position_set(efl_text_interactive_main_cursor_get(txt), 0);
+ evas_key_modifier_on(e, "Control");
+ evas_event_feed_key_down(e, "Delete", "Delete", "\177", "\177", time(NULL), NULL);
+ ecore_main_loop_iterate();
+ ck_assert_str_eq(efl_text_get(txt),"");
+ ck_assert_int_eq(efl_text_cursor_position_get(efl_text_interactive_main_cursor_get(txt)), 0);
+ evas_event_feed_key_up(e, "Delete", "Delete", "\177", "\177", time(NULL), NULL);
+ ecore_main_loop_iterate();
+
+ efl_del(txt);
+ efl_del(win);
+}
+EFL_END_TEST
+
void efl_ui_test_text(TCase *tc)
{
tcase_add_test(tc, text_cnp);
@@ -186,4 +223,5 @@ void efl_ui_test_text(TCase *tc)
tcase_add_test(tc, text_user_change);
tcase_add_test(tc, text_scroll_mode);
tcase_add_test(tc, text_change_event);
+ tcase_add_test(tc, text_keys_handler);
}