aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPhilip Langdale <philipl@mail.utexas.edu>2005-11-13 04:48:16 +0800
committerPhilip Langdale <philipl@src.gnome.org>2005-11-13 04:48:16 +0800
commitda8c338ff084b90e43c2333b91bdcde8d765ff00 (patch)
treefe7a7429cff1800e2839ac7aa8a5c30919e6c744
parentd211a9678ff2a47bfcf6ce2ede0979f01f536fe9 (diff)
downloadgsoc2013-epiphany-da8c338ff084b90e43c2333b91bdcde8d765ff00.tar
gsoc2013-epiphany-da8c338ff084b90e43c2333b91bdcde8d765ff00.tar.gz
gsoc2013-epiphany-da8c338ff084b90e43c2333b91bdcde8d765ff00.tar.bz2
gsoc2013-epiphany-da8c338ff084b90e43c2333b91bdcde8d765ff00.tar.lz
gsoc2013-epiphany-da8c338ff084b90e43c2333b91bdcde8d765ff00.tar.xz
gsoc2013-epiphany-da8c338ff084b90e43c2333b91bdcde8d765ff00.tar.zst
gsoc2013-epiphany-da8c338ff084b90e43c2333b91bdcde8d765ff00.zip
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.
-rw-r--r--ChangeLog25
-rw-r--r--data/ui/epiphany-ui.xml16
-rw-r--r--src/ephy-lockdown.c2
-rwxr-xr-xsrc/ephy-toolbar.c6
-rw-r--r--src/ephy-toolbar.h2
-rw-r--r--src/ephy-window.c101
-rw-r--r--src/window-commands.c50
-rw-r--r--src/window-commands.h10
8 files changed, 64 insertions, 148 deletions
diff --git a/ChangeLog b/ChangeLog
index 97e6b9e7c..f9fcbe1cd 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,28 @@
+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.
+
2005-11-12 Christian Persch <chpe@cvs.gnome.org>
* data/Makefile.am:
diff --git a/data/ui/epiphany-ui.xml b/data/ui/epiphany-ui.xml
index 60339dfc7..087118f11 100644
--- a/data/ui/epiphany-ui.xml
+++ b/data/ui/epiphany-ui.xml
@@ -186,21 +186,5 @@
<accelerator name="AlwaysStopAccel" action="ViewAlwaysStop"/>
<accelerator name="BrowseWithCaretAccel" action="BrowseWithCaret"/>
-<accelerator name="FileSaveAccel" action="FileSave"/>
-<accelerator name="ForceReloadAccel" action="ForceReload"/>
-<accelerator name="XFKeyHomePageAccel" action="XFKeyHomePage"/>
-<accelerator name="XFKeyBackAccel" action="XFKeyBack"/>
-<accelerator name="XFKeyForwardAccel" action="XFKeyForward"/>
-<accelerator name="XFKeyStopAccel" action="XFKeyStop"/>
-<accelerator name="XFKeyRefreshAccel" action="XFKeyRefresh"/>
-<accelerator name="XFKeyFavoritesAccel" action="XFKeyFavorites"/>
-<accelerator name="XFKeyHistoryAccel" action="XFKeyHistory"/>
-<accelerator name="XFKeyOpenURLAccel" action="XFKeyOpenURL"/>
-<accelerator name="XFKeyAddFavoriteAccel" action="XFKeyAddFavorite"/>
-<accelerator name="XFKeyGoAccel" action="XFKeyGo"/>
-<accelerator name="XFKeyReloadAccel" action="XFKeyReload"/>
-<accelerator name="XFKeySendToAccel" action="XFKeySendTo"/>
-<accelerator name="XFKeyZoomInAccel" action="XFKeyZoomIn"/>
-<accelerator name="XFKeyZoomOutAccel" action="XFKeyZoomOut"/>
</ui>
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