Re: [evolution-patches] Re: patch for calender accessibility (48078 and 50845)

[wu yang <Yang Wu sun com>]

JP Rosevear wrote:

> On Mon, 2003-12-01 at 01:47, wu yang wrote:
>  
>
> > Index: calendar//gui/e-week-view.c
> > ===================================================================
> > RCS file: /cvs/gnome/evolution/calendar/gui/e-week-view.c,v
> > retrieving revision 1.191
> > diff -u -r1.191 e-week-view.c
> > --- calendar//gui/e-week-view.c	7 Nov 2003 05:52:00 -0000	1.191
> > +++ calendar//gui/e-week-view.c	26 Nov 2003 03:27:51 -0000
> > @@ -3208,15 +3208,19 @@
> > 	switch (event->keyval) {
> > 	case GDK_Up:
> > 		e_week_view_on_key_up (week_view, event);
> > +		g_signal_emit_by_name (week_view, "selected_time_changed");
> > 		return TRUE;
> > 	case GDK_Down:
> > 		e_week_view_on_key_down (week_view, event);
> > +		g_signal_emit_by_name (week_view, "selected_time_changed");
> > 		return TRUE;	
> > 	case GDK_Left:
> > 		e_week_view_on_key_left (week_view, event);
> > +		g_signal_emit_by_name (week_view, "selected_time_changed");
> > 		return TRUE;
> > 	case GDK_Right:
> > 		e_week_view_on_key_right (week_view, event);
> > +		g_signal_emit_by_name (week_view, "selected_time_changed");
> > 		return TRUE;
> > 	default:
> > 		break;
> >    
>
> To be consistent with the e-day-view, I think the signal emissions
> should go into the e_week_view_on_key_* calls.
>
> -JP
>  
I will change these in my next patch for 45776.
This code in e-week-view just make it work.
I will repalce e_week_view_on_key_* function with others.
Thanks