Re: Request for comments on UI change (resend)

[rms39 columbia edu (Russell Steinthal)]

On Thu, 04 Nov 1999 10:21:14 MST, Darren Blaser wrote:

>If you'd like to forward me a copy of your patch. I'd be happy to 
code up
>the state machine part then we could decide whether or not to stick 
it in.
>
>One caveat: I've got a big project to finish up at work by the 12th 
so I may
>not get to it until next weekend. Of course, by being so slow 
someone else
>might beat me to the punch :-)

Ok, here's what I have at this point.  Although I'll note again that if people don't object to removing the double-click behavior, there's no need for you to code the state-machine either:

The patch is against HEAD, remove the #if 0 block to activate.

Index: gncal-day-view.c
===================================================================
RCS file: /cvs/gnome/gnome-pim/gncal/gncal-day-view.c,v
retrieving revision 1.17
diff -u -r1.17 gncal-day-view.c
--- gncal-day-view.c	1999/05/25 19:28:25	1.17
+++ gncal-day-view.c	1999/11/04 17:24:48
@@ -13,8 +13,8 @@
 #include "main.h"
 #include "eventedit.h"
 #include "popup-menu.h"
+#include "quick-view.h"
 
-
 #define TEXT_BORDER 2
 #define MIN_INFO_WIDTH 50
 
@@ -151,7 +151,21 @@
 
 	if (event->button == 1 && event->type == GDK_2BUTTON_PRESS)
 		gnome_calendar_dayjump (dayview->calendar, dayview->lower);
-
+#if 0
+	else if (event->button == 1 && event->type == GDK_BUTTON_PRESS) {
+		time_t day_begin_time, day_end_time;
+		GList *list;
+		GtkWidget *qv;
+		char date_str [256];
+		
+		strftime (date_str, sizeof (date_str), _("%a %b %d %Y"), 
+			  localtime (&dayview->lower));
+		qv = quick_view_new (dayview->calendar, date_str, dayview->events);
+		quick_view_do_popup (QUICK_VIEW (qv), event);
+		gtk_widget_destroy (qv);
+	}
+		
+#endif
 	return TRUE;
 }

-- 
Russell Steinthal		Columbia Law School, Class of 2002
<rms39@columbia.edu>		Columbia College, Class of 1999
<steintr@nj.org>		UNIX System Administrator, nj.org