From 49d64e1f1afd32c68aea3730e9db3a985680ab8c Mon Sep 17 00:00:00 2001 From: Bowon Ryu Date: Wed, 8 Aug 2018 15:20:42 +0900 Subject: [PATCH] efl_ui_tab_pager: clean up and remove unnecessary signals. Summary: TAB of efl_ui_tab_bar is no longer widget_item. TAB is now layout, so, do not need below signals. "efl,text,visible" "efl,text,hidden" "efl,icon,visible" "efl,icon,hidden" Test Plan: elementary_test -to efl.ui.tab_pager Reviewers: Jaehyun_Cho, Hermet Reviewed By: Jaehyun_Cho Subscribers: #reviewers, cedric, #committers, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6770 --- data/elementary/themes/edc/efl/tab_bar.edc | 18 +++------- src/lib/elementary/efl_ui_tab_bar.c | 39 ++++------------------ 2 files changed, 12 insertions(+), 45 deletions(-) diff --git a/data/elementary/themes/edc/efl/tab_bar.edc b/data/elementary/themes/edc/efl/tab_bar.edc index f7f039cb79..e9f0b913db 100644 --- a/data/elementary/themes/edc/efl/tab_bar.edc +++ b/data/elementary/themes/edc/efl/tab_bar.edc @@ -484,7 +484,7 @@ group { "efl/tab_bar/tab"; action: SIGNAL_EMIT "efl,mouse,out" "efl"; } program { name: "st1"; - signal: "efl,text,hidden"; source: "efl"; + signal: "efl,state,text,unset"; source: "efl"; script { new m = get_int(btmode); m &= ~LABEL; set_int(btmode, m); @@ -492,7 +492,7 @@ group { "efl/tab_bar/tab"; } } program { name: "st2"; - signal: "efl,text,visible"; source: "efl"; + signal: "efl,state,text,set"; source: "efl"; script { new m = get_int(btmode); m |= LABEL; set_int(btmode, m); @@ -500,7 +500,7 @@ group { "efl/tab_bar/tab"; } } program { name: "st3"; - signal: "efl,icon,hidden"; source: "efl"; + signal: "efl,state,icon,unset"; source: "efl"; script { new m = get_int(btmode); m &= ~ICON; set_int(btmode, m); @@ -508,7 +508,7 @@ group { "efl/tab_bar/tab"; } } program { name: "st4"; - signal: "efl,icon,visible"; source: "efl"; + signal: "efl,state,icon,set"; source: "efl"; script { new m = get_int(btmode); m |= ICON; set_int(btmode, m); @@ -537,16 +537,8 @@ group { "efl/tab_bar/tab"; target: "shadow1"; target: "shadow2"; } - program { - signal: "efl,state,icon_set,forward"; source: "efl"; - after: "icon_set,animation"; - } - program { - signal: "efl,state,icon_set,backward"; source: "efl"; - after: "icon_set,animation"; - } program { name: "icon_set,animation"; - signal: "efl,state,icon_set"; source: "efl"; + signal: "efl,state,icon_new,set"; source: "efl"; action: STATE_SET "animation" 0.0; target: "icon_clip1"; target: "icon_clip2"; diff --git a/src/lib/elementary/efl_ui_tab_bar.c b/src/lib/elementary/efl_ui_tab_bar.c index ffbc1fdbce..f377ff2747 100644 --- a/src/lib/elementary/efl_ui_tab_bar.c +++ b/src/lib/elementary/efl_ui_tab_bar.c @@ -46,11 +46,6 @@ _tab_icon_update(Tab_Info *ti) efl_content_unset(efl_part(ti->tab, "efl.icon")); efl_content_set(efl_part(ti->tab, "efl.icon"), ti->icon); - if (ti->icon) - efl_layout_signal_emit(ti->tab, "efl,state,icon,visible", "efl"); - else - efl_layout_signal_emit(ti->tab, "efl,state,icon,hidden", "efl"); - efl_del(old_icon); } @@ -63,7 +58,7 @@ _tab_icon_set_cb(void *data, Tab_Info *ti = data; _tab_icon_update(ti); - efl_layout_signal_callback_del(obj, emission, source, _tab_icon_set_cb, NULL); + efl_layout_signal_callback_del(obj, emission, source, _tab_icon_set_cb, ti); efl_layout_signal_emit(ti->tab, "efl,state,icon,reset", "efl"); } @@ -71,8 +66,7 @@ static void _tab_icon_obj_set(Eo *obj, Tab_Info *ti, Eo *icon_obj, - const char *icon_str, - const char *sig) + const char *icon_str) { Eo *old_icon; const char *s; @@ -104,7 +98,7 @@ _tab_icon_obj_set(Eo *obj, } efl_content_set (efl_part(ti->tab, "efl.icon_new"), ti->icon); - efl_layout_signal_emit(ti->tab, sig, "efl"); + efl_layout_signal_emit(ti->tab, "efl,state,icon_new,set", "efl"); efl_layout_signal_callback_add (ti->tab, "efl,state,icon_set,done", "efl", _tab_icon_set_cb, ti); } @@ -223,29 +217,10 @@ _tab_add(Eo *obj, const char *label, const char *icon) (tab, "efl,action,click", "efl", _action_click_cb, ti); if (ti->icon) - { - efl_content_set(efl_part(tab, "efl.icon"), ti->icon); - efl_layout_signal_emit(tab, "efl,state,icon,visible", "efl"); - efl_layout_signal_emit(tab, "efl,icon,visible", "efl"); - efl_gfx_entity_visible_set(ti->icon, EINA_TRUE); - } - else - { - efl_layout_signal_emit(tab, "efl,state,icon,hidden", "efl"); - efl_layout_signal_emit(tab, "efl,icon,hidden", "efl"); - } + efl_content_set(efl_part(tab, "efl.icon"), ti->icon); if (ti->label) - { - efl_text_set(efl_part(tab, "efl.text"), ti->label); - efl_layout_signal_emit(tab, "efl,state,text,visible", "efl"); - efl_layout_signal_emit(tab, "efl,text,visible", "efl"); - } - else - { - efl_layout_signal_emit(tab, "efl,state,text,hidden", "efl"); - efl_layout_signal_emit(tab, "efl,text,hidden", "efl"); - } + efl_text_set(efl_part(tab, "efl.text"), ti->label); efl_ui_widget_focus_allow_set(tab, EINA_TRUE); @@ -323,10 +298,10 @@ _efl_ui_tab_bar_tab_icon_set(Eo *obj, Efl_Ui_Tab_Bar_Data *sd, int index, const icon_obj = elm_icon_add(obj); if (!icon_obj) return; if (_tab_icon_set(icon_obj, "toolbar/", icon)) - _tab_icon_obj_set(obj, ti, icon_obj, icon, "efl,state,icon_set"); + _tab_icon_obj_set(obj, ti, icon_obj, icon); else { - _tab_icon_obj_set(obj, ti, NULL, NULL, "efl,state,icon_set"); + _tab_icon_obj_set(obj, ti, NULL, NULL); efl_del(icon_obj); } }