Remove accelerator actions for extra keybindings.

2005-11-12  Philip Langdale  <philipl@mail.utexas.edu>

	* data/ui/epiphany-ui.xml: Remove accelerator actions
	for extra keybindings.

	* src/ephy-lockdown.c: (update_window):
	Don't manipulate removed "FileSave" action.

	* src/ephy-toolbar.c
	* src/ephy-toolbar.h: (ephy_toolbar_get_action_group):

	Add getter for the toolbar's action group.

	* src/ephy-window.c: (ephy_window_key_press_event):

	Transition over to the Galeon mechanism for handling
	extra keybindings. This maps the keybindings to
	actions so that no extra actions are required. The
	only subtlety is that some of the actions come from
	the toolbar, so access to its action group is required.

	* src/window-commands.c
	* src/window-commands.h: (window_cmd_go_back/forward/home):
	Remove the now unsued back/forward/home callbacks.

6 files changed, 39 insertions, 132 deletions

diff --git a/src/ephy-lockdown.c b/src/ephy-lockdown.c
index 5af6216b4..48f6f5e1a 100644
--- a/src/ephy-lockdown.c
+++ b/src/ephy-lockdown.c
@@ -182,8 +182,6 @@ update_window (EphyWindow *window,
 	disabled = eel_gconf_get_boolean (CONF_LOCKDOWN_DISABLE_SAVE_TO_DISK);
 	action = gtk_action_group_get_action (action_group, "FileSaveAs");
 	ephy_action_change_sensitivity_flags (action, LOCKDOWN_FLAG, disabled);
-	action = gtk_action_group_get_action (action_group, "FileSave");
-	ephy_action_change_sensitivity_flags (action, LOCKDOWN_FLAG, disabled);
 	action = gtk_action_group_get_action (popups_action_group, "DownloadLink");
 	ephy_action_change_sensitivity_flags (action, LOCKDOWN_FLAG, disabled);
 	action = gtk_action_group_get_action (popups_action_group, "DownloadLinkAs");
diff --git a/src/ephy-toolbar.c b/src/ephy-toolbar.c
index 80c032868..5cf152168 100755
--- a/src/ephy-toolbar.c
+++ b/src/ephy-toolbar.c
@@ -363,6 +363,12 @@ ephy_toolbar_set_window (EphyToolbar *toolbar,
 
 /* public functions */
 
+GtkActionGroup *
+ephy_toolbar_get_action_group (EphyToolbar *toolbar)
+{
+   return toolbar->priv->action_group;   
+}
+
 void
 ephy_toolbar_set_favicon (EphyToolbar *toolbar,
 			  GdkPixbuf *icon)
diff --git a/src/ephy-toolbar.h b/src/ephy-toolbar.h
index 8cb954edd..f9077fa08 100644
--- a/src/ephy-toolbar.h
+++ b/src/ephy-toolbar.h
@@ -65,6 +65,8 @@ GType		ephy_toolbar_get_type			(void);
 
 EphyToolbar    *ephy_toolbar_new			(EphyWindow *window);
 
+GtkActionGroup *ephy_toolbar_get_action_group		(EphyToolbar *toolbar);
+
 void		ephy_toolbar_set_favicon		(EphyToolbar *toolbar,
 							 GdkPixbuf *icon);
 
diff --git a/src/ephy-window.c b/src/ephy-window.c
index 77b543630..6317028e3 100644
--- a/src/ephy-window.c
+++ b/src/ephy-window.c
@@ -137,9 +137,6 @@ static const GtkActionEntry ephy_menu_entries [] = {
 	{ "FileSaveAs", GTK_STOCK_SAVE_AS, N_("Save _As..."), "<shift><control>S",
 	  N_("Save the current page"),
 	  G_CALLBACK (window_cmd_file_save_as) },
-	{ "FileSave", GTK_STOCK_SAVE_AS, N_("Save _As..."), "<control>S",
-	  N_("Save the current page"),
-	  G_CALLBACK (window_cmd_file_save_as) },
 	{ "FilePrintSetup", STOCK_PRINT_SETUP, N_("Print Set_up..."), NULL,
 	  N_("Setup the page settings for printing"),
 	  G_CALLBACK (window_cmd_file_print_setup) },
@@ -205,9 +202,6 @@ static const GtkActionEntry ephy_menu_entries [] = {
 	{ "ViewReload", GTK_STOCK_REFRESH, N_("_Reload"), "<control>R",
 	  N_("Display the latest content of the current page"),
 	  G_CALLBACK (window_cmd_view_reload) },
-	{ "ForceReload", NULL, "", "<shift><control>R",
-	  NULL,
-	  G_CALLBACK (window_cmd_view_reload) },
 	{ "ViewZoomIn", GTK_STOCK_ZOOM_IN, N_("Zoom _In"), "<control>plus",
 	  N_("Increase the text size"),
 	  G_CALLBACK (window_cmd_view_zoom_in) },
@@ -351,30 +345,34 @@ static const GtkActionEntry ephy_popups_entries [] = {
 	  NULL, G_CALLBACK (popup_cmd_copy_image_location) },
 };
 
-#ifdef HAVE_X11_XF86KEYSYM_H
 static const struct
 {
-	const char *name;
 	guint keyval;
-	GCallback callback;
-} xf_key_actions [] = {
-	{ "XFKeyHomePage", 	XF86XK_HomePage,	G_CALLBACK (window_cmd_go_home ) },
-	{ "XFKeyBack",		XF86XK_Back,		G_CALLBACK (window_cmd_go_back ) },
-	{ "XFKeyForward",	XF86XK_Forward,		G_CALLBACK (window_cmd_go_forward ) },
-	{ "XFKeyStop", 		XF86XK_Stop,		G_CALLBACK (window_cmd_view_stop ) },
-	{ "XFKeyRefresh", 	XF86XK_Refresh, 	G_CALLBACK (window_cmd_view_reload ) },
-	{ "XFKeyFavorites", 	XF86XK_Favorites,	G_CALLBACK (window_cmd_go_bookmarks ) },
-	{ "XFKeyHistory", 	XF86XK_History, 	G_CALLBACK (window_cmd_go_history ) },
-	{ "XFKeyOpenURL", 	XF86XK_OpenURL, 	G_CALLBACK (window_cmd_go_location ) },
-	{ "XFKeyAddFavorite", 	XF86XK_AddFavorite, 	G_CALLBACK (window_cmd_file_bookmark_page ) },
-	{ "XFKeyGo", 		XF86XK_Go,	 	G_CALLBACK (window_cmd_go_location ) },
-	{ "XFKeyReload", 	XF86XK_Reload,	 	G_CALLBACK (window_cmd_view_reload ) },
-	{ "XFKeySendTo", 	XF86XK_Send,	 	G_CALLBACK (window_cmd_file_send_to) },
-	{ "XFKeyZoomIn", 	XF86XK_ZoomIn,	 	G_CALLBACK (window_cmd_view_zoom_in ) },
-	{ "XFKeyZoomOut", 	XF86XK_ZoomOut,	 	G_CALLBACK (window_cmd_view_zoom_out) }
+	GdkModifierType modifier;
+	const gchar *action;
+	gboolean fromToolbar;
+} extra_keybindings [] = {
+	{ GDK_s,		GDK_CONTROL_MASK,	"FileSaveAs",		FALSE },
+	{ GDK_R,		GDK_CONTROL_MASK |
+				GDK_SHIFT_MASK,		"ViewReload",		FALSE },
+#ifdef HAVE_X11_XF86KEYSYM_H
+	{ XF86XK_HomePage,	0,			"GoHome",		TRUE  },
+	{ XF86XK_Back,		0,			"NavigationBack",	TRUE  },
+	{ XF86XK_Forward,	0,			"NavigationForward",	TRUE  },
+	{ XF86XK_Stop,		0,			"ViewStop",		FALSE },
+	{ XF86XK_Refresh, 	0,			"ViewReload",		FALSE },
+	{ XF86XK_Reload,	0, 			"ViewReload",		FALSE },
+	{ XF86XK_Favorites,	0,			"GoBookmarks",		FALSE },
+	{ XF86XK_History, 	0,			"GoHistory",		FALSE },
+	{ XF86XK_OpenURL, 	0,			"GoLocation",		FALSE },
+	{ XF86XK_Go,	 	0,			"GoLocation",		FALSE },
+	{ XF86XK_AddFavorite, 	0,			"FileBookmarkPage",	FALSE },
+	{ XF86XK_Send,	 	0,			"FileSendTo",		FALSE },
+	{ XF86XK_ZoomIn,	0, 			"ViewZoomIn",		FALSE },
+	{ XF86XK_ZoomOut,	0, 			"ViewZoomOut",		FALSE },
 	/* FIXME: what about ScrollUp, ScrollDown, Menu*, Option, LogOff, Save,.. any others? */
-};
 #endif /* HAVE_X11_XF86KEYSYM_H */
+};
 
 #define CONF_LOCKDOWN_HIDE_MENUBAR "/apps/epiphany/lockdown/hide_menubar"
 #define CONF_DESKTOP_BG_PICTURE "/desktop/gnome/background/picture_filename"
@@ -746,6 +744,7 @@ ephy_window_key_press_event (GtkWidget *widget,
 	GtkWidget *menubar, *focus_widget;
 	gboolean shortcircuit = FALSE, force_chain = FALSE, handled = FALSE;
 	guint modifier = event->state & gtk_accelerator_get_default_mod_mask ();
+	guint i;
 
 	/* In an attempt to get the mozembed playing nice with things like emacs keybindings
 	 * we are passing important events to the focused child widget before letting the window's
@@ -796,27 +795,22 @@ ephy_window_key_press_event (GtkWidget *widget,
 		}
 	}
 
-	/**
-	 * FIXME: The following construct allows us to use extra keybindings
-	 * mapped directly to actions. This is better than the current mechanism
-	 * of mapping them to the same callback as the desired actions.
-	 */
-#if 0
 	/* Handle accelerators that we want bound, but aren't associated with
 	 * an action */
 	for (i = 0; i < G_N_ELEMENTS (extra_keybindings); i++)
 	{
-		if (modifier == extra_keybindings[i].modifier &&
-		    event->keyval == extra_keybindings[i].keyval)
+		if (event->keyval == extra_keybindings[i].keyval &&
+		    modifier == extra_keybindings[i].modifier)
 		{
-			GtkAction * action = gtk_action_group_get_action 
-				(priv->action_group, 
-				 extra_keybindings[i].action);
+			GtkAction * action = gtk_action_group_get_action
+				(extra_keybindings[i].fromToolbar ? 
+					ephy_toolbar_get_action_group (priv->toolbar) :
+					priv->action_group,
+				extra_keybindings[i].action);
 			gtk_action_activate (action);
 			return TRUE;
 		}
 	}
-#endif
 
 	/* Don't activate menubar in ppv mode, or in lockdown mode */
 	if (priv->ppv_mode || eel_gconf_get_boolean (CONF_LOCKDOWN_HIDE_MENUBAR))
@@ -1232,35 +1226,6 @@ update_chromes_actions (EphyWindow *window)
 		 			   window);
 }
 
-#ifdef HAVE_X11_XF86KEYSYM_H
-static void
-setup_multimedia_key_actions (EphyWindow *window)
-{
-	GtkActionGroup *action_group = window->priv->action_group;
-	GtkAction *action;
-	const char *agname, *name;
-	char *accel_path;
-	guint i;
-
-	agname = gtk_action_group_get_name (action_group);
-
-	for (i = 0; i < G_N_ELEMENTS (xf_key_actions); i++)
-	{
-		name = xf_key_actions[i].name;
-
-		action = g_object_new (GTK_TYPE_ACTION, "name", name, NULL);
-		g_signal_connect (action, "activate",
-				  xf_key_actions[i].callback, window);
-
-		accel_path = g_strconcat ("<Actions>/", agname, "/", name, NULL);
-		gtk_action_set_accel_path (action, accel_path);
-		gtk_accel_map_add_entry (accel_path, xf_key_actions[i].keyval, 0);
-		gtk_action_group_add_action (action_group, action);
-		g_free (accel_path);
-	}
-}
-#endif /* HAVE_X11_XF86KEYSYM_H */
-
 static void
 setup_ui_manager (EphyWindow *window)
 {
@@ -1331,10 +1296,6 @@ setup_ui_manager (EphyWindow *window)
 	g_signal_connect (manager, "add_widget", G_CALLBACK (add_widget), window);
 	gtk_window_add_accel_group (GTK_WINDOW (window),
 				    gtk_ui_manager_get_accel_group (manager));
-
-#ifdef HAVE_X11_XF86KEYSYM_H
-	setup_multimedia_key_actions (window);
-#endif				    
 }
 
 static void
diff --git a/src/window-commands.c b/src/window-commands.c
index 0b5a5fc8d..f6a698870 100644
--- a/src/window-commands.c
+++ b/src/window-commands.c
@@ -96,20 +96,6 @@ window_cmd_file_print (GtkAction *action,
 }
 
 void
-window_cmd_go_back (GtkAction *action,
-		    EphyWindow *window)
-{
-	EphyEmbed *embed;
-
-	embed = ephy_window_get_active_embed (window);
-	g_return_if_fail (embed != NULL);
-
-	gtk_widget_grab_focus (GTK_WIDGET (embed));
-
-	ephy_embed_go_back (embed);
-}
-
-void
 window_cmd_file_send_to	(GtkAction *action,
 			 EphyWindow *window)
 {
@@ -137,20 +123,6 @@ window_cmd_file_send_to	(GtkAction *action,
 	g_free (url);
 }
 
-void
-window_cmd_go_forward (GtkAction *action,
-		       EphyWindow *window)
-{
-	EphyEmbed *embed;
-
-	embed = ephy_window_get_active_embed (window);
-	g_return_if_fail (embed != NULL);
-
-	gtk_widget_grab_focus (GTK_WIDGET (embed));
-
-	ephy_embed_go_forward (embed);
-}
-
 static gboolean
 event_with_shift (void)
 {
@@ -887,25 +859,3 @@ window_cmd_browse_with_caret (GtkAction *action,
 	active = gtk_toggle_action_get_active (GTK_TOGGLE_ACTION (action));
 	eel_gconf_set_boolean (CONF_BROWSE_WITH_CARET, active);
 }
-
-#ifdef HAVE_X11_XF86KEYSYM_H
-void
-window_cmd_go_home (GtkAction *action,
-		    EphyWindow *window)
-{
-	char *location;
-
-	location = eel_gconf_get_string (CONF_GENERAL_HOMEPAGE);
-
-	if (location == NULL || location[0] == '\0')
-	{
-		g_free (location);
-
-		location = g_strdup ("about:blank");
-	}
-
-	ephy_window_load_url (window, location);
-
-	g_free (location);
-}
-#endif /* HAVE_X11_XF86KEYSYM_H */
diff --git a/src/window-commands.h b/src/window-commands.h
index d74a54fab..8043bda59 100644
--- a/src/window-commands.h
+++ b/src/window-commands.h
@@ -28,12 +28,6 @@ void window_cmd_edit_find	(GtkAction *action,
 void window_cmd_view_stop	(GtkAction *action,
 				 EphyWindow *window);
 
-void window_cmd_go_back		(GtkAction *action,
-				 EphyWindow *window);
-
-void window_cmd_go_forward	(GtkAction *action,
-				 EphyWindow *window);
-
 void window_cmd_go_location	(GtkAction *action,
 				 EphyWindow *window);
 
@@ -163,7 +157,3 @@ void window_cmd_load_location   (GtkAction *action,
 void window_cmd_browse_with_caret (GtkAction *action,
 				   EphyWindow *window);
 
-#ifdef HAVE_X11_XF86KEYSYM_H
-void window_cmd_go_home		(GtkAction *action,
-				 EphyWindow *window);
-#endif