diff options
author | zmike <michael.blumenkrantz@gmail.com> | 2014-05-07 00:50:12 -0400 |
---|---|---|
committer | zmike <michael.blumenkrantz@gmail.com> | 2014-05-07 01:06:30 -0400 |
commit | 377179bdaf84aa1a86621cdfa64ed43613ab9d45 (patch) | |
tree | 47dd563701059045be7b59803526e6751ca3b0b6 | |
parent | 5832b692aa5f68a2924bd7886390d489eb13aefb (diff) | |
download | elementary-377179bdaf84aa1a86621cdfa64ed43613ab9d45.tar.gz |
remove all scrolling changes for focused items.
this.is.BROOOOOOOOOOOOOOOKEEEEEEEEEEEEEEEEEEENNNNNNNNNNNNNNNNNNNNNNNNNNNNN.
SERIOUSLY.
you cannot go scrolling all over the place in every widget that has a scroller just because the widget gets focus. what user wants that? no user anywhere, under any circumstances, ever, in all of history.
if you dare to put this back in, I will continue to remove it for the rest of eternity until it never, ever scrolls in any unwanted case. the focused item doesn't have to always be in the viewport, and should never be moved into the viewport [[[[[[[ except to maintain an already-existing position inside the viewport ]]]]]]]
too much of my time wasted on this stupid "feature" which should have been MUCH more thoroughly tested and reviewed before it was merged.
T1205 STILL NOT FIXED
-rw-r--r-- | src/lib/elm_gengrid.c | 6 | ||||
-rw-r--r-- | src/lib/elm_genlist.c | 6 | ||||
-rw-r--r-- | src/lib/elm_list.c | 10 | ||||
-rw-r--r-- | src/lib/elm_toolbar.c | 6 |
4 files changed, 0 insertions, 28 deletions
diff --git a/src/lib/elm_gengrid.c b/src/lib/elm_gengrid.c index fde0afb03..38063bdfb 100644 --- a/src/lib/elm_gengrid.c +++ b/src/lib/elm_gengrid.c @@ -1487,12 +1487,6 @@ _elm_gengrid_item_focused(Elm_Gen_Item *it) (elm_widget_item_disabled_get(it))) return; - if (!_elm_config->focus_auto_scroll_bring_in_enable) - elm_gengrid_item_show - ((Elm_Object_Item *)it, ELM_GENGRID_ITEM_SCROLLTO_IN); - else - elm_gengrid_item_bring_in - ((Elm_Object_Item *)it, ELM_GENGRID_ITEM_SCROLLTO_IN); sd->focused_item = (Elm_Object_Item *)it; if (elm_widget_focus_highlight_enabled_get(obj)) diff --git a/src/lib/elm_genlist.c b/src/lib/elm_genlist.c index d6cc9d592..656744820 100644 --- a/src/lib/elm_genlist.c +++ b/src/lib/elm_genlist.c @@ -2432,12 +2432,6 @@ _elm_genlist_item_focused(Elm_Gen_Item *it) (elm_widget_item_disabled_get(it))) return; - if (!_elm_config->focus_auto_scroll_bring_in_enable) - elm_genlist_item_show((Elm_Object_Item *)it, - ELM_GENLIST_ITEM_SCROLLTO_IN); - else - elm_genlist_item_bring_in((Elm_Object_Item *)it, - ELM_GENLIST_ITEM_SCROLLTO_IN); sd->focused_item = (Elm_Object_Item *)it; if (elm_widget_focus_highlight_enabled_get(obj)) diff --git a/src/lib/elm_list.c b/src/lib/elm_list.c index 3dc553287..050897871 100644 --- a/src/lib/elm_list.c +++ b/src/lib/elm_list.c @@ -1072,21 +1072,11 @@ static void _elm_list_item_focused(Elm_List_Item *it) { ELM_LIST_DATA_GET(WIDGET(it), sd); - Evas_Coord x, y, w, h, sx, sy, sw, sh; const char *focus_raise; if ((!sd) || (sd->select_mode == ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY) || (it == (Elm_List_Item *)sd->focused_item)) return; - evas_object_geometry_get(VIEW(it), &x, &y, &w, &h); - evas_object_geometry_get(sd->hit_rect, &sx, &sy, &sw, &sh); - if ((x < sx) || (y < sy)|| ((x + w) > (sx + sw)) || ((y + h) > (sy + sh))) - { - if (!_elm_config->focus_auto_scroll_bring_in_enable) - elm_list_item_show((Elm_Object_Item *)it); - else - elm_list_item_bring_in((Elm_Object_Item *)it); - } sd->focused_item = (Elm_Object_Item *)it; if (elm_widget_focus_highlight_enabled_get(WIDGET(it))) { diff --git a/src/lib/elm_toolbar.c b/src/lib/elm_toolbar.c index 5c85bcb41..0a62fe1f6 100644 --- a/src/lib/elm_toolbar.c +++ b/src/lib/elm_toolbar.c @@ -601,12 +601,6 @@ _elm_toolbar_item_focused(Elm_Toolbar_Item *it) return; sd->focused_item = it; - if (!_elm_config->focus_auto_scroll_bring_in_enable) - elm_toolbar_item_show((Elm_Object_Item *)it, - ELM_TOOLBAR_ITEM_SCROLLTO_IN); - else - elm_toolbar_item_bring_in((Elm_Object_Item *)it, - ELM_TOOLBAR_ITEM_SCROLLTO_IN); if (elm_widget_focus_highlight_enabled_get(obj)) { edje_object_signal_emit |