1998-11-02 Federico Mena Quintero <federico@nuclecu.unam.mx>

* gdesktop.c (select_range): Doh.  Fix dii -> ldii typo.  Also,
	use Windows semantics -- ranges are considered rectangular instead
	of linear.

gnome/ChangeLog

@@ -1,5 +1,9 @@
 1998-11-02  Federico Mena Quintero  <federico@nuclecu.unam.mx>
 
+	* gdesktop.c (select_range): Doh.  Fix dii -> ldii typo.  Also,
+	use Windows semantics -- ranges are considered rectangular instead
+	of linear.
+
 	* gdesktop.c: Now desktop icons are stored in the layout_slots
 	array.  We need to know which icons are in which slots.
 	(unselect_all): Use the layout_slots array.

gnome/gdesktop.c

@@ -61,7 +61,7 @@ static int layout_cols;
 static int layout_rows;
 static struct layout_slot *layout_slots;
 
-#define l_slots(x, y) (layout_slots[(x) * layout_rows + (y)])
+#define l_slots(u, v) (layout_slots[(u) * layout_rows + (v)])
 
 /* The last icon to be selected */
 static struct desktop_icon_info *last_selected_icon;
@@ -130,7 +130,7 @@ get_icon_snap_pos (int *x, int *y)
 			dy = sy - v;
 			dist = dx * dx + dy * dy;
 
-			if (((val == min) && (dist < min_dist)) || (val < min)) {
+			if ((val == min && dist < min_dist) || (val < min)) {
 				min_dist = dist;
 				min_x = u;
 				min_y = v;
@@ -212,40 +212,64 @@ unselect_all (void)
 static void
 select_range (struct desktop_icon_info *dii, int sel)
 {
-	int min, max;
+	int du, dv, lu, lv;
-	int i;
+	int min_u, min_v;
+	int max_u, max_v;
+	int u, v;
 	GList *l;
-	struct desktop_icon_info *ldii, *min_dii, *max_dii;
+	struct desktop_icon_info *ldii;
+	struct desktop_icon_info *min_udii, *min_vdii;
+	struct desktop_icon_info *max_udii, *max_vdii;
 
 	/* Find out the selection range */
 
 	if (!last_selected_icon)
 		last_selected_icon = dii;
 
-	if (last_selected_icon->slot < dii->slot) {
+	du = dii->slot / layout_rows;
-		min = last_selected_icon->slot;
+	dv = dii->slot % layout_rows;
-		max = dii->slot;
+	lu = last_selected_icon->slot / layout_rows;
-		min_dii = last_selected_icon;
+	lv = last_selected_icon->slot % layout_rows;
-		max_dii = dii;
+
+	if (du < lu) {
+		min_u = du;
+		max_u = lu;
+		min_udii = dii;
+		max_udii = last_selected_icon;
 	} else {
-		min = dii->slot;
+		min_u = lu;
-		max = last_selected_icon->slot;
+		max_u = du;
-		min_dii = dii;
+		min_udii = last_selected_icon;
-		max_dii = last_selected_icon;
+		max_udii = dii;
+	}
+
+	if (dv < lv) {
+		min_v = dv;
+		max_v = lv;
+		min_vdii = dii;
+		max_vdii = last_selected_icon;
+	} else {
+		min_v = lv;
+		max_v = dv;
+		min_vdii = last_selected_icon;
+		max_vdii = dii;
 	}
 
 	/* Select! */
 
-	for (i = min; i <= max; i++)
+	for (u = min_u; u <= max_u; u++)
-		for (l = layout_slots[i].icons; l; l = l->next) {
+		for (v = min_v; v <= max_v; v++)
+			for (l = l_slots (u, v).icons; l; l = l->next) {
 				ldii = l->data;
 
-			if (((i == min) && ((ldii->x < min_dii->x) || (ldii->y < min_dii->y)))
+				if ((u == min_u && ldii->x < min_udii->x)
-			    || ((i == max) && ((ldii->x > max_dii->x) || (ldii->y > max_dii->y))))
+				    || (v == min_v && ldii->y < min_vdii->y)
+				    || (u == max_u && ldii->x > max_udii->x)
+				    || (v == max_v && ldii->y > max_vdii->y))
 					continue;
 
-			desktop_icon_select (DESKTOP_ICON (dii->dicon), sel);
+				desktop_icon_select (DESKTOP_ICON (ldii->dicon), sel);
-			dii->selected = sel;
+				ldii->selected = sel;
 			}
 }