# HG changeset patch
# User Richard Laager <rlaager@wiktel.com>
# Date 1136958825 0
# Node ID 7b8e885c1be30992f84fc3fcb96d1899d1aa2622
# Parent  292a846fdeea7fe86b8c92b7b3ce70b07cc5bae4
[gaim-migrate @ 15172]
Someone mentioned in #gaim the other day that "Close this tab" should be the last option in the menu. I agree. I keep looking for it there. Anyone disagree?

committer: Tailor Script <tailor@pidgin.im>

diff -r 292a846fdeea -r 7b8e885c1be3 src/gtkconv.c
--- a/src/gtkconv.c	Wed Jan 11 05:45:48 2006 +0000
+++ b/src/gtkconv.c	Wed Jan 11 05:53:45 2006 +0000
@@ -7169,6 +7169,20 @@
 	menu = notebook->menu;
 	gaim_separator(GTK_WIDGET(menu));
 
+	item = gtk_menu_item_new_with_label("Close other tabs");
+	gtk_widget_show(item);
+	gtk_menu_shell_append(GTK_MENU_SHELL(menu), item);
+	g_signal_connect(G_OBJECT(item), "activate",
+					G_CALLBACK(close_others_cb), menu);
+
+	item = gtk_menu_item_new_with_label("Close all tabs");
+	gtk_widget_show(item);
+	gtk_menu_shell_append(GTK_MENU_SHELL(menu), item);
+	g_signal_connect(G_OBJECT(item), "activate",
+					G_CALLBACK(close_window), win);
+
+	gaim_separator(menu);
+
 	item = gtk_menu_item_new_with_label("Detach this tab");
 	gtk_widget_show(item);
 	gtk_menu_shell_append(GTK_MENU_SHELL(menu), item);
@@ -7181,20 +7195,6 @@
 	g_signal_connect(G_OBJECT(item), "activate",
 					G_CALLBACK(close_tab_cb), menu);
 
-	gaim_separator(menu);
-
-	item = gtk_menu_item_new_with_label("Close other tabs");
-	gtk_widget_show(item);
-	gtk_menu_shell_append(GTK_MENU_SHELL(menu), item);
-	g_signal_connect(G_OBJECT(item), "activate",
-					G_CALLBACK(close_others_cb), menu);
-
-	item = gtk_menu_item_new_with_label("Close all tabs");
-	gtk_widget_show(item);
-	gtk_menu_shell_append(GTK_MENU_SHELL(menu), item);
-	g_signal_connect(G_OBJECT(item), "activate",
-					G_CALLBACK(close_window), win);
-
 	return FALSE;
 }