summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorzmike <michael.blumenkrantz@gmail.com>2014-05-07 00:50:12 -0400
committerzmike <michael.blumenkrantz@gmail.com>2014-05-07 01:06:30 -0400
commit377179bdaf84aa1a86621cdfa64ed43613ab9d45 (patch)
tree47dd563701059045be7b59803526e6751ca3b0b6
parent5832b692aa5f68a2924bd7886390d489eb13aefb (diff)
downloadelementary-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.c6
-rw-r--r--src/lib/elm_genlist.c6
-rw-r--r--src/lib/elm_list.c10
-rw-r--r--src/lib/elm_toolbar.c6
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