elm genlist: Fixed wrong reorder animation. Patch by Bluezery <ohpowel@gmail.com>

On Fri, Aug 31, 2012 at 8:50 PM, Bluezery <ohpowel@gmail.com> 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

diff --git a/src/lib/elm_genlist.c b/src/lib/elm_genlist.c
index bdec6dc..5cf1039 100644 (file)
--- 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)
      {