Re: [evolution-patches] patch for #48999

[Charles Zhang <Charles Zhang Sun Com>]

Sorry, my words wasn't clear enough.
In fact, it had been verified and tested ok before I posted it.

Please review.

Best Regards
Charles Zhang

Mike Kestner wrote:

On Thu, 2003-10-09 at 13:42, Charles Zhang wrote:
  

In this patch, I try to fix the bug 48999.

Here, I disable the apply button while initializing the dialog and 
pressing apply button.

Please see if it works.
    

Are you saying you haven't tested that it fixes the bug?  If so, please
test the patch and resubmit.

  

Index: gal/e-table/e-table-config.c
===================================================================
RCS file: /cvs/gnome/gal/gal/e-table/e-table-config.c,v
retrieving revision 1.58.2.1
diff -u -p -r1.58.2.1 e-table-config.c
--- gal/e-table/e-table-config.c	11 Sep 2003 21:08:43 -0000	1.58.2.1
+++ gal/e-table/e-table-config.c	9 Oct 2003 18:32:35 -0000
@@ -1181,7 +1181,10 @@ e_table_config_new (const char          
 		return NULL;
 	}
 
+	gtk_dialog_set_response_sensitive (GTK_DIALOG (config->dialog_toplevel),
+					   GTK_RESPONSE_APPLY, FALSE);
 	gtk_widget_show (config->dialog_toplevel);
+
 	return E_TABLE_CONFIG (config);
 }
 
Index: gal/e-table/e-table-header-item.c
===================================================================
RCS file: /cvs/gnome/gal/gal/e-table/e-table-header-item.c,v
retrieving revision 1.124
diff -u -p -r1.124 e-table-header-item.c
--- gal/e-table/e-table-header-item.c	18 Mar 2003 17:33:02 -0000	1.124
+++ gal/e-table/e-table-header-item.c	9 Oct 2003 18:32:45 -0000
@@ -1426,6 +1426,9 @@ apply_changes (ETableConfig *config, ETa
 	if (ethi->tree)
 		e_tree_set_state (ethi->tree, state);
 	g_free (state);
+
+	gtk_dialog_set_response_sensitive (GTK_DIALOG (config->dialog_toplevel),
+					   GTK_RESPONSE_APPLY, FALSE);
 }
 
 static void