From f0989a7e39d3822ddaef38d197fd2a2747df9542 Mon Sep 17 00:00:00 2001 From: Daniel Juyung Seo Date: Mon, 3 Sep 2012 06:20:47 +0000 Subject: [PATCH] elm genlist: Fixed wrong reorder animation. Patch by Bluezery On Fri, Aug 31, 2012 at 8:50 PM, Bluezery wrote: > Hello, > > Sometimes reorder animation ended incorrectly > You can test & see the problem when background color is black and > genlist item color is white. > I made the added value for animation as difference between old item > position and new item position if difference is below predefined dy. > (y = diff, if diff < dy) So item position can be finally new item > position at last. > Please review this patch. > > Thanks SVN revision: 75950 --- src/lib/elm_genlist.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/lib/elm_genlist.c b/src/lib/elm_genlist.c index bdec6dc..5cf1039 100644 --- a/src/lib/elm_genlist.c +++ b/src/lib/elm_genlist.c @@ -1743,7 +1743,8 @@ _reorder_move_animator_cb(void *data) else y = dy; diff = abs(it->item->old_scrl_y - it->item->scrl_y); - if (diff > it->item->h) y = diff / 2; + if (diff < dy) y = diff; + else if (diff > it->item->h) y = diff / 2; if (it->item->old_scrl_y < it->item->scrl_y) { -- 2.7.4