Added click-to-move in addition to drag-to-move

[grrobot] / src / grr_board_view.c

diff --git a/src/grr_board_view.c b/src/grr_board_view.c
index d4d0d263fb7b77c9cbfcbd12e2ba40d512271f4a..c4cf316e49103c9c158bd5373c9fc237206f9f3b 100644 (file)
--- a/src/grr_board_view.c
+++ b/src/grr_board_view.c
@@ -543,26 +543,30 @@ grr_board_view_expose (GtkWidget      *widget,
 
     /* Draw grid. */
     XrSave (xrs);
-    XrSetRGBColor (xrs, .75, .75, 1);
-    XrSetLineWidth (xrs, 1);
-    for (j=0; j < view->board_height; j++) {
-       XrMoveTo (xrs, 0, j * view->cell_height + 0.5);
-       XrRelLineTo (xrs, view->board_width * view->cell_width, 0);
-    }
-    for (i=0; i < view->board_width; i++) {
-       XrMoveTo (xrs, i * view->cell_width + 0.5, 0);
-       XrRelLineTo (xrs, 0, view->board_height * view->cell_height);
+    {
+       XrSetRGBColor (xrs, .75, .75, 1);
+       XrSetLineWidth (xrs, 1);
+       for (j=0; j < view->board_height; j++) {
+           XrMoveTo (xrs, 0, j * view->cell_height + 0.5);
+           XrRelLineTo (xrs, view->board_width * view->cell_width, 0);
+       }
+       for (i=0; i < view->board_width; i++) {
+           XrMoveTo (xrs, i * view->cell_width + 0.5, 0);
+           XrRelLineTo (xrs, 0, view->board_height * view->cell_height);
+       }
+       XrStroke (xrs);
     }
-    XrStroke (xrs);
     XrRestore (xrs);
 
     /* Draw goal target in center of board */
     XrSave (xrs);
-    XrTranslate (xrs,
-                (view->board_width / 2 - 1) * view->cell_width,
-                (view->board_height / 2 - 1) * view->cell_height);
-    grr_cell_draw (xrs, goal_target, goal_target,
-                  view->cell_width * 2, view->cell_height * 2);
+    {
+       XrTranslate (xrs,
+                    (view->board_width / 2 - 1) * view->cell_width,
+                    (view->board_height / 2 - 1) * view->cell_height);
+       grr_cell_draw (xrs, goal_target, goal_target,
+                      view->cell_width * 2, view->cell_height * 2);
+    }
     XrRestore (xrs);
 
     /* Draw walls */
@@ -617,8 +621,6 @@ grr_board_view_button_press (GtkWidget      *widget,
        gtk_grab_add (widget);
 
        view->button = event->button;
-       view->drag_x = x;
-       view->drag_y = y;
        view->drag_robot = RR_CELL_GET_ROBOT (cell);
 
        grr_board_view_update_mouse (view, event->x, event->y);
@@ -629,12 +631,13 @@ grr_board_view_button_press (GtkWidget      *widget,
 
 static gint
 grr_board_view_button_release (GtkWidget      *widget,
-                         GdkEventButton *event)
+                              GdkEventButton *event)
 {
     grr_board_view_t *view;
     const char *dir;
     const char *robot;
     int x, y;
+    int robot_x, robot_y;
     int dx, dy;
 
     g_return_val_if_fail (widget != NULL, FALSE);
@@ -645,53 +648,53 @@ grr_board_view_button_release (GtkWidget      *widget,
 
     if (view->button == event->button) {
       gtk_grab_remove (widget);
-
       view->button = 0;
+    }
+
+    grr_board_view_pointer_coords_to_grid (view, event->x, event->y, &x, &y);
 
-      grr_board_view_pointer_coords_to_grid (view, event->x, event->y, &x, &y);
-
-      dx = x - view->drag_x;
-      dy = y - view->drag_y;
-      if (dx == 0 && dy == 0)
-         return FALSE;
-
-      if (abs(dx) > abs(dy))
-         if (x > view->drag_x)
-             dir = "east";
-         else
-             dir = "west";
-      else
-         if (y > view->drag_y)
-             dir = "south";
-         else
-             dir = "north";
-
-      switch (view->drag_robot) {
-      case RR_ROBOT_BLUE:
-         robot = "blue";
-         break;
-      case RR_ROBOT_GREEN:
-         robot = "green";
-         break;
-      case RR_ROBOT_RED:
-         robot = "RED";
-         break;
-      case RR_ROBOT_YELLOW:
-         robot = "YELLOW";
-         break;
-      default:
-         return FALSE;
-      }
-
-      if (view->client) {
-         char *move_str;
-
-         grr_sprintf_alloc (&move_str, "%s %s", robot, dir);
-         if (move_str == NULL)
-             return FALSE;
-         rr_client_move (view->client, move_str);
-         free (move_str);
-      }
+    rr_board_find_robot (view->board, view->drag_robot, &robot_x, &robot_y);
+    dx = x - robot_x;
+    dy = y - robot_y;
+    if (dx == 0 && dy == 0)
+       return FALSE;
+
+    if (abs(dx) > abs(dy))
+       if (x > robot_x)
+           dir = "east";
+       else
+           dir = "west";
+    else
+       if (y > robot_y)
+           dir = "south";
+       else
+           dir = "north";
+    
+    switch (view->drag_robot) {
+    case RR_ROBOT_BLUE:
+       robot = "blue";
+       break;
+    case RR_ROBOT_GREEN:
+       robot = "green";
+       break;
+    case RR_ROBOT_RED:
+       robot = "RED";
+       break;
+    case RR_ROBOT_YELLOW:
+       robot = "YELLOW";
+       break;
+    default:
+       return FALSE;
+    }
+    
+    if (view->client) {
+       char *move_str;
+       
+       grr_sprintf_alloc (&move_str, "%s %s", robot, dir);
+       if (move_str == NULL)
+           return FALSE;
+       rr_client_move (view->client, move_str);
+       free (move_str);
     }
 
   return FALSE;