gengrid: frame variable should be used for position changes.
authorAmitesh Singh <amitesh.sh@samsung.com>
Wed, 30 Apr 2014 08:59:40 +0000 (17:59 +0900)
committerDaniel Juyung Seo <seojuyung2@gmail.com>
Wed, 30 Apr 2014 08:59:40 +0000 (17:59 +0900)
Summary:
Earlier pos was getting used for the item coordinates. To support other Ecore_Pos_Map variations, frame variable should be used.
@fix

Test Plan:
elementary_test-> "Gengrid Focus". Enable reorder mode and use keys to move
the focused item.

Reviewers: raster, seoz

Reviewed By: seoz

CC: seoz
Differential Revision: https://phab.enlightenment.org/D811

src/lib/elm_gengrid.c

index 12d1a2e..1034f4d 100644 (file)
@@ -2103,11 +2103,11 @@ _item_move_cb(void *data, double pos)
 
    dx = sd->reorder.x2 - sd->reorder.x1;
    dy = sd->reorder.y2 - sd->reorder.y1;
-   xx1 = sd->reorder.x1 + dx * pos;
-   yy1 = sd->reorder.y1 + dy * pos;
+   xx1 = sd->reorder.x1 + dx * frame;
+   yy1 = sd->reorder.y1 + dy * frame;
 
-   xx2 = sd->reorder.x2 - dx * pos;
-   yy2 = sd->reorder.y2 - dy * pos;
+   xx2 = sd->reorder.x2 - dx * frame;
+   yy2 = sd->reorder.y2 - dy * frame;
 
    evas_object_move(VIEW(sd->reorder.it1), xx1, yy1);
    evas_object_move(VIEW(sd->reorder.it2), xx2, yy2);