bu5hm4n pushed a commit to branch master. http://git.enlightenment.org/core/efl.git/commit/?id=210db8fc5d26e1d8fc772f533fedadbf37fa255b
commit 210db8fc5d26e1d8fc772f533fedadbf37fa255b Author: Yeongjong Lee <clean...@naver.com> Date: Tue Jan 16 21:17:11 2018 +0100 efl_ui_focus: merge efl_ui_focus_user and efl_ui_focus_object into one mixin Summary: focus_user and focus_object are similar classes. by merging them into one mixin, we can maintain consistency. Test Plan: make check Reviewers: bu5hm4n Subscribers: cedric, Jaehyun_Cho, woohyun, jpeg Differential Revision: https://phab.enlightenment.org/D5734 --- src/Makefile_Elementary.am | 1 - src/lib/elementary/Efl_Ui.h | 6 ++++- src/lib/elementary/Elementary.h | 6 ++++- src/lib/elementary/efl_ui_focus_layer.c | 2 +- src/lib/elementary/efl_ui_focus_layer.eo | 2 +- src/lib/elementary/efl_ui_focus_manager_calc.c | 6 ++--- src/lib/elementary/efl_ui_focus_manager_sub.c | 18 +++++++-------- src/lib/elementary/efl_ui_focus_manager_sub.eo | 2 +- src/lib/elementary/efl_ui_focus_object.c | 1 - src/lib/elementary/efl_ui_focus_object.eo | 25 ++++++++++++++++++++ src/lib/elementary/efl_ui_focus_user.eo | 32 -------------------------- src/lib/elementary/efl_ui_focus_util.c | 8 +++---- src/lib/elementary/efl_ui_multibuttonentry.c | 4 ++-- src/lib/elementary/efl_ui_widget.c | 16 ++++++------- src/lib/elementary/efl_ui_widget.eo | 12 +++++----- src/lib/elementary/elc_fileselector.c | 2 +- src/lib/elementary/elc_naviframe.c | 2 +- src/lib/elementary/elm_access.c | 2 +- src/lib/elementary/elm_hover.c | 2 +- src/lib/elementary/elm_main.c | 10 ++++---- src/lib/elementary/elm_spinner.c | 2 +- src/lib/elementary/elm_widget.h | 2 +- src/tests/elementary/elm_test_focus_common.c | 4 ++-- src/tests/elementary/elm_test_focus_sub.c | 6 ++--- src/tests/elementary/focus_test.eo | 6 ++--- src/tests/elementary/focus_test_sub_main.eo | 6 ++--- 26 files changed, 92 insertions(+), 93 deletions(-) diff --git a/src/Makefile_Elementary.am b/src/Makefile_Elementary.am index d0af9ad567..b080c61f87 100644 --- a/src/Makefile_Elementary.am +++ b/src/Makefile_Elementary.am @@ -55,7 +55,6 @@ elm_public_eolian_files = \ lib/elementary/efl_ui_focus_manager_sub.eo \ lib/elementary/efl_ui_focus_manager_root_focus.eo \ lib/elementary/efl_ui_focus_object.eo \ - lib/elementary/efl_ui_focus_user.eo \ lib/elementary/efl_ui_focus_layer.eo \ lib/elementary/efl_ui_focus_composition.eo \ lib/elementary/efl_ui_focus_util.eo \ diff --git a/src/lib/elementary/Efl_Ui.h b/src/lib/elementary/Efl_Ui.h index f605fc3f0c..cb548f70a7 100644 --- a/src/lib/elementary/Efl_Ui.h +++ b/src/lib/elementary/Efl_Ui.h @@ -136,12 +136,16 @@ extern EAPI double _efl_startup_time; // EO types. Defined for legacy-only builds as legacy uses typedef of EO types. #include "efl_ui.eot.h" +//define focus manager earlier since focus object and manager is circular +typedef Eo Efl_Ui_Focus_Manager; +#define _EFL_UI_FOCUS_MANAGER_EO_CLASS_TYPE + # include <efl_ui_focus_object.eo.h> # include <efl_ui_focus_manager.eo.h> # include <efl_ui_focus_manager_calc.eo.h> # include <efl_ui_focus_manager_sub.eo.h> # include <efl_ui_focus_manager_root_focus.eo.h> -# include <efl_ui_focus_user.eo.h> +# include <efl_ui_focus_util.eo.h> # include <efl_ui_textpath.eo.h> # include <efl_ui_translatable.eo.h> # include <efl_ui_focus_composition.eo.h> diff --git a/src/lib/elementary/Elementary.h b/src/lib/elementary/Elementary.h index 36057bdcea..e5597b8e1a 100644 --- a/src/lib/elementary/Elementary.h +++ b/src/lib/elementary/Elementary.h @@ -148,12 +148,16 @@ EAPI extern Elm_Version *elm_version; #include <elm_focus.h> #if defined (EFL_EO_API_SUPPORT) && defined (EFL_BETA_API_SUPPORT) + +//define focus manager earlier since focus object and manager is circular +typedef Eo Efl_Ui_Focus_Manager; +#define _EFL_UI_FOCUS_MANAGER_EO_CLASS_TYPE + # include <efl_ui_focus_object.eo.h> # include <efl_ui_focus_manager.eo.h> # include <efl_ui_focus_manager_calc.eo.h> # include <efl_ui_focus_manager_sub.eo.h> # include <efl_ui_focus_manager_root_focus.eo.h> -# include <efl_ui_focus_user.eo.h> # include <efl_ui_focus_util.eo.h> # include <efl_ui_textpath.eo.h> # include <efl_ui_translatable.eo.h> diff --git a/src/lib/elementary/efl_ui_focus_layer.c b/src/lib/elementary/efl_ui_focus_layer.c index e19b0f802e..363b1742f5 100644 --- a/src/lib/elementary/efl_ui_focus_layer.c +++ b/src/lib/elementary/efl_ui_focus_layer.c @@ -59,7 +59,7 @@ _efl_ui_focus_layer_efl_object_destructor(Eo *obj, Efl_Ui_Focus_Layer_Data *pd E } EOLIAN static Efl_Ui_Focus_Manager* -_efl_ui_focus_layer_efl_ui_focus_user_focus_manager_get(Eo *obj, Efl_Ui_Focus_Layer_Data *pd EINA_UNUSED) +_efl_ui_focus_layer_efl_ui_focus_object_focus_manager_get(Eo *obj, Efl_Ui_Focus_Layer_Data *pd EINA_UNUSED) { return elm_widget_top_get(obj); } diff --git a/src/lib/elementary/efl_ui_focus_layer.eo b/src/lib/elementary/efl_ui_focus_layer.eo index 5d4b2d8c35..cb955fc19e 100644 --- a/src/lib/elementary/efl_ui_focus_layer.eo +++ b/src/lib/elementary/efl_ui_focus_layer.eo @@ -25,7 +25,7 @@ mixin Efl.Ui.Focus.Layer (Efl.Interface, Efl.Ui.Widget, Efl.Gfx, Efl.Ui.Focus.Ma Efl.Ui.Widget.focus_state_apply; Efl.Gfx.visible {set;} Efl.Ui.Focus.Manager.move; - Efl.Ui.Focus.User.focus_manager { get; } + Efl.Ui.Focus.Object.focus_manager { get; } Efl.Object.constructor; Efl.Object.destructor; } diff --git a/src/lib/elementary/efl_ui_focus_manager_calc.c b/src/lib/elementary/efl_ui_focus_manager_calc.c index 2e1b10a1d3..7fa873ab70 100644 --- a/src/lib/elementary/efl_ui_focus_manager_calc.c +++ b/src/lib/elementary/efl_ui_focus_manager_calc.c @@ -86,12 +86,12 @@ _manager_in_chain_set(Eo *obj, Efl_Ui_Focus_Manager_Calc_Data *pd) EINA_SAFETY_ON_NULL_RETURN(pd->root); if (!efl_isa(pd->root->focusable, EFL_UI_WIN_CLASS)) - EINA_SAFETY_ON_NULL_RETURN(efl_ui_focus_user_focus_manager_get(pd->root->focusable)); + EINA_SAFETY_ON_NULL_RETURN(efl_ui_focus_object_focus_manager_get(pd->root->focusable)); //so we dont run infinitly this does not fix it, but at least we only have a error - EINA_SAFETY_ON_TRUE_RETURN(efl_ui_focus_user_focus_manager_get(pd->root->focusable) == obj); + EINA_SAFETY_ON_TRUE_RETURN(efl_ui_focus_object_focus_manager_get(pd->root->focusable) == obj); - manager = efl_ui_focus_user_focus_manager_get(pd->root->focusable); + manager = efl_ui_focus_object_focus_manager_get(pd->root->focusable); if (manager) efl_ui_focus_manager_focus_set(manager, pd->root->focusable); else diff --git a/src/lib/elementary/efl_ui_focus_manager_sub.c b/src/lib/elementary/efl_ui_focus_manager_sub.c index 4e1ce22052..d6f45ad5c5 100644 --- a/src/lib/elementary/efl_ui_focus_manager_sub.c +++ b/src/lib/elementary/efl_ui_focus_manager_sub.c @@ -49,7 +49,7 @@ _border_flush(Eo *obj, Efl_Ui_Focus_Manager_Sub_Data *pd) Efl_Ui_Focus_Object *node, *logical; Efl_Ui_Focus_Manager *manager; - manager = efl_ui_focus_user_focus_manager_get(obj); + manager = efl_ui_focus_object_focus_manager_get(obj); logical = obj; borders = efl_ui_focus_manager_border_elements_get(obj); @@ -90,7 +90,7 @@ _border_unregister(Eo *obj, Efl_Ui_Focus_Manager_Sub_Data *pd) Efl_Ui_Focus_Object *node; Efl_Ui_Focus_Manager *manager; - manager = efl_ui_focus_user_focus_manager_get(obj); + manager = efl_ui_focus_object_focus_manager_get(obj); EINA_LIST_FREE(pd->current_border, node) { if (node == obj) continue; @@ -135,12 +135,12 @@ _logical_manager_change(void *data EINA_UNUSED, const Efl_Event *ev) { MY_DATA(data, pd); Eina_List *n; - Efl_Ui_Focus_User *b; + Efl_Ui_Focus_Object *b; Efl_Ui_Focus_Manager *manager; if (!ev->info) return; - manager = efl_ui_focus_user_focus_manager_get(ev->object); + manager = efl_ui_focus_object_focus_manager_get(ev->object); EINA_LIST_FOREACH(pd->current_border, n, b) { if (b == ev->object) continue; @@ -153,11 +153,11 @@ _flush_manager(Eo *obj, Efl_Ui_Focus_Manager_Sub_Data *pd) { Efl_Ui_Focus_Manager *manager; Efl_Ui_Focus_Object *logical; - Efl_Ui_Focus_User *b; + Efl_Ui_Focus_Object *b; Eina_List *n; - logical = efl_ui_focus_user_focus_parent_get(obj); - manager = efl_ui_focus_user_focus_manager_get(obj); + logical = efl_ui_focus_object_focus_parent_get(obj); + manager = efl_ui_focus_object_focus_manager_get(obj); //unregister from the old if (pd->manager) efl_event_callback_array_del(pd->manager, parent_manager(), obj); @@ -184,8 +184,8 @@ _manager_change(void *data, const Efl_Event *ev EINA_UNUSED) EFL_CALLBACKS_ARRAY_DEFINE(self_manager, {EFL_UI_FOCUS_MANAGER_EVENT_COORDS_DIRTY, _self_manager_dirty}, - {EFL_UI_FOCUS_USER_EVENT_LOGICAL_CHANGED, _logical_manager_change}, - {EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, _manager_change} + {EFL_UI_FOCUS_OBJECT_EVENT_LOGICAL_CHANGED, _logical_manager_change}, + {EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, _manager_change} ); EOLIAN static Efl_Object* diff --git a/src/lib/elementary/efl_ui_focus_manager_sub.eo b/src/lib/elementary/efl_ui_focus_manager_sub.eo index 899e96b4b5..edf2ee0e76 100644 --- a/src/lib/elementary/efl_ui_focus_manager_sub.eo +++ b/src/lib/elementary/efl_ui_focus_manager_sub.eo @@ -3,7 +3,7 @@ mixin Efl.Ui.Focus.Manager.Sub (Efl.Interface, Efl.Ui.Focus.Manager, Efl.Object) [[A class that automatically registeres its border elements in the parent manager This sub manager will register its border elements on the parent manager. - The parent manager is found with the @Efl.Ui.Focus.User interface of the + The parent manager is found with the @Efl.Ui.Focus.Object interface of the @Efl.Object.parent. Each sub manager also has to be a focus object, the object itself will be diff --git a/src/lib/elementary/efl_ui_focus_object.c b/src/lib/elementary/efl_ui_focus_object.c index e1361dfad1..df565b4300 100644 --- a/src/lib/elementary/efl_ui_focus_object.c +++ b/src/lib/elementary/efl_ui_focus_object.c @@ -28,4 +28,3 @@ _efl_ui_focus_object_focus_get(Eo *obj EINA_UNUSED, Efl_Ui_Focus_Object_Data *pd #include "efl_ui_focus_object.eo.c" -#include "efl_ui_focus_user.eo.c" diff --git a/src/lib/elementary/efl_ui_focus_object.eo b/src/lib/elementary/efl_ui_focus_object.eo index 96976919ee..6842dfabc7 100644 --- a/src/lib/elementary/efl_ui_focus_object.eo +++ b/src/lib/elementary/efl_ui_focus_object.eo @@ -36,6 +36,25 @@ mixin Efl.Ui.Focus.Object focus : bool; [[The state in which the object should be put]] } } + @property focus_manager { + [[Describes which manager is used to register. + + If an instance of this interface is the root of a manager, this instance should not have a manager where as root of this property. The other manager in this instance will be set as focused in the corresponding manager. + This instance should be registered with its own manager as redirect. + ]] + get {} + values { + manager : Efl.Ui.Focus.Manager; [[The manager object]] + } + } + @property focus_parent { + [[Describes which logical parent is used by this object.]] + get {} + values { + logical_parent : Efl.Ui.Focus.Object; [[The focus parent.]] + } + } + prepare_logical { [[Tells the object that its children will be queried soon by the given manager. @@ -46,8 +65,14 @@ mixin Efl.Ui.Focus.Object implements { @empty .focus_geometry; @empty .prepare_logical; + @empty .focus_manager; + @empty .focus_parent; } events { focus,changed : bool; [[Emitted if the focus state has changed]] + manager,changed: Efl.Ui.Focus.Manager; [[Emitted when a new manager is + the parent for this object.]] + logical,changed: Efl.Ui.Focus.Object; [[Emitted when a new logical + parent should be used.]] } } diff --git a/src/lib/elementary/efl_ui_focus_user.eo b/src/lib/elementary/efl_ui_focus_user.eo deleted file mode 100644 index 11de8e9547..0000000000 --- a/src/lib/elementary/efl_ui_focus_user.eo +++ /dev/null @@ -1,32 +0,0 @@ -interface Efl.Ui.Focus.User { - [[Interface which is used to handle the manager of a user. - - @since 1.20 - ]] - methods { - @property focus_manager { - [[Describes which manager is used to register. - - If an instance of this interface is the root of a manager, this instance should not have a manager where as root of this property. The other manager in this instance will be set as focused in the corresponding manager. - This instance should be registered with its own manager as redirect. - ]] - get {} - values { - manager : Efl.Ui.Focus.Manager; [[The manager object]] - } - } - @property focus_parent { - [[Describes which logical parent is used by this object.]] - get {} - values { - logical_parent : Efl.Ui.Focus.Object; [[The focus parent.]] - } - } - } - events { - manager,changed: Efl.Ui.Focus.Manager; [[Emitted when a new manager is - the parent for this object.]] - logical,changed: Efl.Ui.Focus.Object; [[Emitted when a new logical - parent should be used.]] - } -} diff --git a/src/lib/elementary/efl_ui_focus_util.c b/src/lib/elementary/efl_ui_focus_util.c index aa0fbbdb42..8758047eda 100644 --- a/src/lib/elementary/efl_ui_focus_util.c +++ b/src/lib/elementary/efl_ui_focus_util.c @@ -16,23 +16,23 @@ _manager_changed(void *data, const Efl_Event *event EINA_UNUSED) } EOLIAN static void -_efl_ui_focus_util_focus(Eo *obj EINA_UNUSED, void *pd EINA_UNUSED, Efl_Ui_Focus_User *user) +_efl_ui_focus_util_focus(Eo *obj EINA_UNUSED, void *pd EINA_UNUSED, Efl_Ui_Focus_Object *user) { Efl_Ui_Widget *top, *o; Efl_Ui_Focus_Manager *m; - m = efl_ui_focus_user_focus_manager_get(user); + m = efl_ui_focus_object_focus_manager_get(user); top = elm_widget_top_get(user); o = efl_key_data_get(top, "__delayed_focus_set"); - if (o) efl_event_callback_del(o, EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, _manager_changed, o); + if (o) efl_event_callback_del(o, EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, _manager_changed, o); efl_key_data_set(top, "__delayed_focus_set", NULL); if (!m) { //delayed focusung efl_key_data_set(top, "__delayed_focus_set", user); - efl_event_callback_add(user, EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, _manager_changed, user); + efl_event_callback_add(user, EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, _manager_changed, user); return; } diff --git a/src/lib/elementary/efl_ui_multibuttonentry.c b/src/lib/elementary/efl_ui_multibuttonentry.c index ca24ed4ff6..630ac60ce3 100644 --- a/src/lib/elementary/efl_ui_multibuttonentry.c +++ b/src/lib/elementary/efl_ui_multibuttonentry.c @@ -1677,7 +1677,7 @@ static void _legacy_manager_changed_cb(void *data EINA_UNUSED, const Efl_Event *ev) { efl_event_callback_del(ev->info, EFL_UI_FOCUS_MANAGER_EVENT_FOCUSED, _legacy_focused, ev->object); - efl_event_callback_add(efl_ui_focus_user_focus_manager_get(ev->object), EFL_UI_FOCUS_MANAGER_EVENT_FOCUSED, _legacy_focused, ev->object); + efl_event_callback_add(efl_ui_focus_object_focus_manager_get(ev->object), EFL_UI_FOCUS_MANAGER_EVENT_FOCUSED, _legacy_focused, ev->object); } EOLIAN static Eo * @@ -1689,7 +1689,7 @@ _efl_ui_multibuttonentry_efl_object_constructor(Eo *obj, Efl_Ui_Multibuttonentry efl_access_role_set(obj, EFL_ACCESS_ROLE_PANEL); //listen to manager changes here - efl_event_callback_add(obj, EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, _legacy_manager_changed_cb, NULL); + efl_event_callback_add(obj, EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, _legacy_manager_changed_cb, NULL); return obj; } diff --git a/src/lib/elementary/efl_ui_widget.c b/src/lib/elementary/efl_ui_widget.c index 4b7b2b2d1a..dfe47d7661 100644 --- a/src/lib/elementary/efl_ui_widget.c +++ b/src/lib/elementary/efl_ui_widget.c @@ -313,7 +313,7 @@ _focus_manager_eval(Eo *obj, Elm_Widget_Smart_Data *pd) } else if (parent) { - new = efl_ui_focus_user_focus_manager_get(parent); + new = efl_ui_focus_object_focus_manager_get(parent); provider = parent; } @@ -322,13 +322,13 @@ _focus_manager_eval(Eo *obj, Elm_Widget_Smart_Data *pd) old = pd->manager.manager; if (pd->manager.provider) - efl_event_callback_del(pd->manager.provider, EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, _manager_changed_cb, obj); + efl_event_callback_del(pd->manager.provider, EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, _manager_changed_cb, obj); pd->manager.manager = new; pd->manager.provider = provider; if (pd->manager.provider) - efl_event_callback_add(pd->manager.provider, EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, _manager_changed_cb, obj); + efl_event_callback_add(pd->manager.provider, EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, _manager_changed_cb, obj); } return old; @@ -541,13 +541,13 @@ _full_eval(Eo *obj, Elm_Widget_Smart_Data *pd) if (old_registered_parent != pd->focus.parent) { efl_event_callback_call(obj, - EFL_UI_FOCUS_USER_EVENT_LOGICAL_CHANGED, old_registered_parent); + EFL_UI_FOCUS_OBJECT_EVENT_LOGICAL_CHANGED, old_registered_parent); } if (old_registered_manager != pd->focus.manager) { efl_event_callback_call(obj, - EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, old_registered_manager); + EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, old_registered_manager); } } @@ -5200,7 +5200,7 @@ _efl_ui_widget_efl_object_destructor(Eo *obj, Elm_Widget_Smart_Data *sd) { if (sd->manager.provider) { - efl_event_callback_del(sd->manager.provider, EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, _manager_changed_cb, obj); + efl_event_callback_del(sd->manager.provider, EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, _manager_changed_cb, obj); sd->manager.provider = NULL; } efl_access_attributes_clear(obj); @@ -5494,13 +5494,13 @@ _efl_ui_widget_efl_object_provider_find(const Eo *obj, Elm_Widget_Smart_Data *pd } EOLIAN static Efl_Ui_Focus_Manager* -_efl_ui_widget_efl_ui_focus_user_focus_parent_get(Eo *obj EINA_UNUSED, Elm_Widget_Smart_Data *pd EINA_UNUSED) +_efl_ui_widget_efl_ui_focus_object_focus_parent_get(Eo *obj EINA_UNUSED, Elm_Widget_Smart_Data *pd EINA_UNUSED) { return pd->focus.parent; } EOLIAN static Efl_Ui_Focus_Manager* -_efl_ui_widget_efl_ui_focus_user_focus_manager_get(Eo *obj EINA_UNUSED, Elm_Widget_Smart_Data *pd EINA_UNUSED) +_efl_ui_widget_efl_ui_focus_object_focus_manager_get(Eo *obj EINA_UNUSED, Elm_Widget_Smart_Data *pd EINA_UNUSED) { return pd->focus.manager; } diff --git a/src/lib/elementary/efl_ui_widget.eo b/src/lib/elementary/efl_ui_widget.eo index 1560b2a93c..b50f2d7da0 100644 --- a/src/lib/elementary/efl_ui_widget.eo +++ b/src/lib/elementary/efl_ui_widget.eo @@ -9,15 +9,15 @@ function Efl.Ui.Scrollable_On_Show_Region { struct Efl.Ui.Widget.Focus_State { [[All relevant fields needed for the current state of focus registeration]] manager : Efl.Ui.Focus.Manager; [[The manager where the widget is registered in]] - parent : Efl.Ui.Focus.User; [[The parent the widget is using as logical parent]] + parent : Efl.Ui.Focus.Object; [[The parent the widget is using as logical parent]] logical : bool; [[$true if this is registered as logical currently]] } /* ATTENTION: Rename to Efl.Ui.Widget is imminent. */ abstract Efl.Ui.Widget (Efl.Canvas.Group, Efl.Access, - Efl.Access.Component, Efl.Ui.Focus.User, Efl.Part, - Efl.Ui.Focus.Object, Efl.Ui.Base, Efl.Ui.Cursor, - Efl.Ui.Translatable, Efl.Selection, Efl.Ui.Dnd) + Efl.Access.Component, Efl.Part, Efl.Ui.Focus.Object, + Efl.Ui.Base, Efl.Ui.Cursor, Efl.Ui.Translatable, + Efl.Selection, Efl.Ui.Dnd) { [[Efl UI widget abstract class]] legacy_prefix: elm_widget; @@ -598,8 +598,8 @@ abstract Efl.Ui.Widget (Efl.Canvas.Group, Efl.Access, Efl.Access.parent { get; } Efl.Access.attributes { get; } Efl.Access.Component.focus_grab; - Efl.Ui.Focus.User.focus_manager { get; } - Efl.Ui.Focus.User.focus_parent { get; } + Efl.Ui.Focus.Object.focus_manager { get; } + Efl.Ui.Focus.Object.focus_parent { get; } Efl.Ui.Focus.Object.focus_geometry { get; } Efl.Ui.Focus.Object.focus { set; } Efl.Ui.Base.mirrored { get; set; } diff --git a/src/lib/elementary/elc_fileselector.c b/src/lib/elementary/elc_fileselector.c index dddd47ccf5..d5d3d732cc 100644 --- a/src/lib/elementary/elc_fileselector.c +++ b/src/lib/elementary/elc_fileselector.c @@ -117,7 +117,7 @@ _focus_chain_update(Eo *obj, Elm_Fileselector_Data *pd) #undef A - efl_ui_focus_manager_calc_update_order(efl_ui_focus_user_focus_manager_get(obj), obj, tmp); + efl_ui_focus_manager_calc_update_order(efl_ui_focus_object_focus_manager_get(obj), obj, tmp); } static void diff --git a/src/lib/elementary/elc_naviframe.c b/src/lib/elementary/elc_naviframe.c index 0e1e85e190..ef8cb9fa81 100644 --- a/src/lib/elementary/elc_naviframe.c +++ b/src/lib/elementary/elc_naviframe.c @@ -92,7 +92,7 @@ _prev_page_focus_recover(Elm_Naviframe_Item_Data *it) { Elm_Naviframe *n = it->base->widget; - efl_ui_focus_manager_focus_set(efl_ui_focus_user_focus_manager_get(n), n); + efl_ui_focus_manager_focus_set(efl_ui_focus_object_focus_manager_get(n), n); } EOLIAN static void diff --git a/src/lib/elementary/elm_access.c b/src/lib/elementary/elm_access.c index 0c69b0b6bd..dc3efd783a 100644 --- a/src/lib/elementary/elm_access.c +++ b/src/lib/elementary/elm_access.c @@ -537,7 +537,7 @@ _access_highlight_next_get(Evas_Object *obj, Elm_Focus_Direction dir) { Efl_Ui_Focus_Relations *rel; - rel = efl_ui_focus_manager_fetch(efl_ui_focus_user_focus_manager_get(obj), obj); + rel = efl_ui_focus_manager_fetch(efl_ui_focus_object_focus_manager_get(obj), obj); if (rel) { diff --git a/src/lib/elementary/elm_hover.c b/src/lib/elementary/elm_hover.c index 1204277b7b..f23fa1c64d 100644 --- a/src/lib/elementary/elm_hover.c +++ b/src/lib/elementary/elm_hover.c @@ -664,7 +664,7 @@ _elm_hover_efl_gfx_visible_set(Eo *obj, Elm_Hover_Data *pd, Eina_Bool vis) { _hov_show_do(obj); //we just set ourself as redirect in the next upper manager - pd->redirected = efl_ui_focus_user_focus_manager_get(pd->target); + pd->redirected = efl_ui_focus_object_focus_manager_get(pd->target); efl_ui_focus_manager_redirect_set(pd->redirected, obj); } else diff --git a/src/lib/elementary/elm_main.c b/src/lib/elementary/elm_main.c index ebaef80d68..9b0f4ddb0a 100644 --- a/src/lib/elementary/elm_main.c +++ b/src/lib/elementary/elm_main.c @@ -1586,13 +1586,13 @@ elm_object_focus_get(const Evas_Object *obj) if (!elm_widget_is(obj)) return evas_object_focus_get(obj); - m = efl_ui_focus_user_focus_manager_get(obj); + m = efl_ui_focus_object_focus_manager_get(obj); //no manager means not registered if (!m) return EINA_FALSE; //assertion: our redirect manager m is in the redirect chain - m = efl_ui_focus_user_focus_manager_get(obj); + m = efl_ui_focus_object_focus_manager_get(obj); //if obj is the redriect manager its kind of focused if (efl_ui_focus_manager_redirect_get(m) == obj) return EINA_TRUE; @@ -1607,7 +1607,7 @@ elm_object_focus_get(const Evas_Object *obj) { if (focused_child == obj) return EINA_TRUE; - focused_child = efl_ui_focus_user_focus_parent_get(focused_child); + focused_child = efl_ui_focus_object_focus_parent_get(focused_child); } return efl_ui_focus_object_focus_get(obj); @@ -1635,8 +1635,8 @@ elm_object_focus_set(Evas_Object *obj, efl_ui_focus_util_focus(EFL_UI_FOCUS_UTIL_CLASS, obj); else { - if (efl_ui_focus_manager_focus_get(efl_ui_focus_user_focus_manager_get(obj)) == obj) - efl_ui_focus_manager_pop_history_stack(efl_ui_focus_user_focus_manager_get(obj)); + if (efl_ui_focus_manager_focus_get(efl_ui_focus_object_focus_manager_get(obj)) == obj) + efl_ui_focus_manager_pop_history_stack(efl_ui_focus_object_focus_manager_get(obj)); } } else diff --git a/src/lib/elementary/elm_spinner.c b/src/lib/elementary/elm_spinner.c index c150d27eb5..5f48b51c3e 100644 --- a/src/lib/elementary/elm_spinner.c +++ b/src/lib/elementary/elm_spinner.c @@ -660,7 +660,7 @@ _toggle_entry(Evas_Object *obj) efl_ui_focus_composition_elements_set(obj, items); } - efl_ui_focus_manager_focus_set(efl_ui_focus_user_focus_manager_get(obj), sd->ent); + efl_ui_focus_manager_focus_set(efl_ui_focus_object_focus_manager_get(obj), sd->ent); } } diff --git a/src/lib/elementary/elm_widget.h b/src/lib/elementary/elm_widget.h index 4bdff24148..6ebea70f18 100644 --- a/src/lib/elementary/elm_widget.h +++ b/src/lib/elementary/elm_widget.h @@ -439,7 +439,7 @@ typedef struct _Elm_Widget_Smart_Data } logical; struct { Efl_Ui_Focus_Manager *manager; - Efl_Ui_Focus_User *provider; + Efl_Ui_Focus_Object *provider; } manager; Eina_Bool scroll_x_locked : 1; diff --git a/src/tests/elementary/elm_test_focus_common.c b/src/tests/elementary/elm_test_focus_common.c index a84dc062f3..9cd355722f 100644 --- a/src/tests/elementary/elm_test_focus_common.c +++ b/src/tests/elementary/elm_test_focus_common.c @@ -101,13 +101,13 @@ _focus_test_efl_gfx_geometry_get(Eo *obj EINA_UNUSED, Focus_Test_Data *pd) } EOLIAN static Efl_Ui_Focus_Manager* -_focus_test_efl_ui_focus_user_focus_manager_get(Eo *obj EINA_UNUSED, Focus_Test_Data *pd) +_focus_test_efl_ui_focus_object_focus_manager_get(Eo *obj EINA_UNUSED, Focus_Test_Data *pd) { return pd->manager; } EOLIAN static Efl_Ui_Focus_Object* -_focus_test_efl_ui_focus_user_focus_parent_get(Eo *obj, Focus_Test_Data *pd EINA_UNUSED) +_focus_test_efl_ui_focus_object_focus_parent_get(Eo *obj, Focus_Test_Data *pd EINA_UNUSED) { return efl_parent_get(obj); } diff --git a/src/tests/elementary/elm_test_focus_sub.c b/src/tests/elementary/elm_test_focus_sub.c index 733abb5d96..8ba3a7e165 100644 --- a/src/tests/elementary/elm_test_focus_sub.c +++ b/src/tests/elementary/elm_test_focus_sub.c @@ -13,13 +13,13 @@ _focus_test_sub_main_efl_ui_focus_object_focus_geometry_get(Eo *obj EINA_UNUSED, } EOLIAN static Efl_Ui_Focus_Manager* -_focus_test_sub_main_efl_ui_focus_user_focus_manager_get(Eo *obj, Focus_Test_Sub_Main_Data *pd EINA_UNUSED) +_focus_test_sub_main_efl_ui_focus_object_focus_manager_get(Eo *obj, Focus_Test_Sub_Main_Data *pd EINA_UNUSED) { return efl_key_data_get(obj, "__user_manager"); } EOLIAN static Efl_Ui_Focus_Object* -_focus_test_sub_main_efl_ui_focus_user_focus_parent_get(Eo *obj, Focus_Test_Sub_Main_Data *pd EINA_UNUSED) +_focus_test_sub_main_efl_ui_focus_object_focus_parent_get(Eo *obj, Focus_Test_Sub_Main_Data *pd EINA_UNUSED) { return efl_key_data_get(obj, "__user_parent"); } @@ -96,7 +96,7 @@ _setup(Efl_Ui_Focus_Manager **m, Efl_Ui_Focus_Manager_Sub **sub, Efl_Ui_Focus_Ob efl_composite_attach(focus_main, subm); - efl_event_callback_call(focus_main, EFL_UI_FOCUS_USER_EVENT_MANAGER_CHANGED, NULL); + efl_event_callback_call(focus_main, EFL_UI_FOCUS_OBJECT_EVENT_MANAGER_CHANGED, NULL); efl_ui_focus_manager_calc_register_logical(manager, focus_main, root_manager, subm); diff --git a/src/tests/elementary/focus_test.eo b/src/tests/elementary/focus_test.eo index 2e8534ab78..63075eb9e2 100644 --- a/src/tests/elementary/focus_test.eo +++ b/src/tests/elementary/focus_test.eo @@ -1,4 +1,4 @@ -class Focus.Test(Efl.Object, Efl.Ui.Focus.Object, Efl.Ui.Focus.User, Efl.Gfx) { +class Focus.Test(Efl.Object, Efl.Ui.Focus.Object, Efl.Gfx) { methods { size { params { @@ -18,8 +18,8 @@ class Focus.Test(Efl.Object, Efl.Ui.Focus.Object, Efl.Ui.Focus.User, Efl.Gfx) { Efl.Object.constructor; Efl.Ui.Focus.Object.focus_geometry { get; } Efl.Ui.Focus.Object.focus { set; } - Efl.Ui.Focus.User.focus_manager { get; } - Efl.Ui.Focus.User.focus_parent {get; } + Efl.Ui.Focus.Object.focus_manager { get; } + Efl.Ui.Focus.Object.focus_parent {get; } Efl.Gfx.geometry { get; } } } diff --git a/src/tests/elementary/focus_test_sub_main.eo b/src/tests/elementary/focus_test_sub_main.eo index af3f330b12..b60c431c87 100644 --- a/src/tests/elementary/focus_test_sub_main.eo +++ b/src/tests/elementary/focus_test_sub_main.eo @@ -1,7 +1,7 @@ -class Focus.Test.Sub.Main(Efl.Object, Efl.Ui.Focus.User, Efl.Ui.Focus.Object, Efl.Ui.Focus.Manager.Sub) { +class Focus.Test.Sub.Main(Efl.Object, Efl.Ui.Focus.Object, Efl.Ui.Focus.Manager.Sub) { implements { - Efl.Ui.Focus.User.focus_manager { get; } - Efl.Ui.Focus.User.focus_parent { get; } + Efl.Ui.Focus.Object.focus_manager { get; } + Efl.Ui.Focus.Object.focus_parent { get; } Efl.Ui.Focus.Object.focus_geometry { get; } } } --