path: root/src/lib/elementary/elm_fileselector.eo
diff options
authorMarcel Hollerbach <>2017-09-02 19:08:18 +0200
committerMarcel Hollerbach <>2017-09-02 21:10:56 +0200
commitc51f35d42a148b34a9cf74714ecc16609984eecb (patch)
tree9e7fa063e796c20c93ca695fa4b729a9e1843044 /src/lib/elementary/elm_fileselector.eo
parent3af81932b04c60f89903834057dfb652cdcd9d5c (diff)
elm_widget: move the complete regsiter/unregister logic
We had here a little problem, state focus_state_eval function handled the unregisteration and consideration of the focus flags and then only called a helper function (which was a widget function), that then did the registeration in logical or regular mode. Elm scroller for example took that function overwrote it and did onyl permit logical registrations. Then again a evaluation of the focus state and flags took place, and the function considered elm_scroller should be registered as regular object, but found it to be logical. This lead to the problem that we permantently unregistered Elm.Scroller and registered it again as logical just to unregister it again. This was on the one side a performance downside. But also a bug since all items from within the Elm_Scrollers sub manager are getting reparent onto the parent, which means not the root of the scroller (the scroller itself) is the logical entrypoint to the widget but rather this reparented widget, which led to unexpected focus warps like described in T5923. tldr: this fixes T5923
Diffstat (limited to '')
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/elementary/elm_fileselector.eo b/src/lib/elementary/elm_fileselector.eo
index 3ac04a40db..ee33eb5ee5 100644
--- a/src/lib/elementary/elm_fileselector.eo
+++ b/src/lib/elementary/elm_fileselector.eo
@@ -41,7 +41,7 @@ class Elm.Fileselector (Efl.Ui.Layout, Elm.Interface.Fileselector,
41 Elm.Widget.widget_event; 41 Elm.Widget.widget_event;
42 Elm.Widget.theme_apply; 42 Elm.Widget.theme_apply;
43 Elm.Widget.focus_next_manager_is; 43 Elm.Widget.focus_next_manager_is;
44 Elm.Widget.focus_register; 44 Elm.Widget.focus_state_apply;
45 Elm.Interface.Fileselector.selected_models { get; } 45 Elm.Interface.Fileselector.selected_models { get; }
46 Elm.Interface.Fileselector.selected_model_get; 46 Elm.Interface.Fileselector.selected_model_get;
47 Elm.Interface.Fileselector.selected_model_set; 47 Elm.Interface.Fileselector.selected_model_set;