Re: gEDA-user: Confusing new 1.6 behavior

[Peter Clifton <pcjc2@xxxxxxxxx>]

On Sun, 2009-11-15 at 15:19 +0000, Peter Clifton wrote:
> On Sun, 2009-11-15 at 06:39 -0700, John Doty wrote:

> Does it feel any better with the attached patch applied?

(Make it this one..)

From 7ae8cdaeedc94216b2941b666d76716e7729acf3 Mon Sep 17 00:00:00 2001
From: Peter Clifton <pcjc2@xxxxxxxxx>
Date: Sun, 15 Nov 2009 15:16:15 +0000
Subject: [PATCH] Don't immediately redraw rubberband objects after placing

This allows the user to see the placed (and selected) object
without it being overwriten by any object still attached to the
mouse cursor. (This made it appear that the object had not been
placed).

The rubberband objects attached to the cursor are re-drawn as soon
as the mouse moves.
---
 gschem/src/x_event.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/gschem/src/x_event.c b/gschem/src/x_event.c
index d13255b..1b1f4b2 100644
--- a/gschem/src/x_event.c
+++ b/gschem/src/x_event.c
@@ -296,9 +296,6 @@ gint x_event_button_pressed(GtkWidget *widget, GdkEventButton *event,
           w_current->inside_action = 0;
           i_set_state(w_current, SELECT);
           i_update_toolbar(w_current);
-        } else {
-          o_place_invalidate_rubber (w_current, TRUE);
-          w_current->rubber_visible = 1;
         }
         break;
 
@@ -588,8 +585,6 @@ gint x_event_button_released(GtkWidget *widget, GdkEventButton *event,
         /* having this stay in copy was driving me nuts*/
         w_current->inside_action = 1;
         /* Keep the state and the inside_action, as the copy has not finished. */
-        o_place_invalidate_rubber (w_current, TRUE);
-        w_current->rubber_visible = 1;
         i_set_state(w_current, ENDMCOPY);
         i_update_toolbar(w_current);
         o_undo_savestate(w_current, UNDO_ALL);
-- 
1.6.3.3


_______________________________________________
geda-user mailing list
geda-user@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user