summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--desktop/tree.h6
-rw-r--r--gtk/menu.c6
-rw-r--r--gtk/menu.h6
-rw-r--r--gtk/scaffolding.c77
4 files changed, 23 insertions, 72 deletions
diff --git a/desktop/tree.h b/desktop/tree.h
index c70f888dc..b4e0f66b7 100644
--- a/desktop/tree.h
+++ b/desktop/tree.h
@@ -33,9 +33,11 @@ struct sslcert_session_data;
struct tree;
struct redraw_context;
-/** Current ssl session data for treeview
+/**
+ * Current ssl session data for treeview
*
- * @fixme global certificate treeview state must go away, this is just wrong.
+ * @todo FIXME global certificate treeview state must go away, this is
+ * just wrong.
*/
extern struct sslcert_session_data *ssl_current_session;
extern const char *tree_hotlist_path;
diff --git a/gtk/menu.c b/gtk/menu.c
index 14eb1f286..04acb575b 100644
--- a/gtk/menu.c
+++ b/gtk/menu.c
@@ -510,12 +510,6 @@ struct nsgtk_popup_menu *nsgtk_popup_menu_create(GtkAccelGroup *group)
ADD_NAMED_SEP(popup, first, nmenu);
- IMAGE_ITEM(popup, opentab, gtkOpentab, nmenu, group);
- IMAGE_ITEM(popup, openwin, gtkOpenwin, nmenu, group);
- IMAGE_ITEM(popup, savelink, gtkSavelink, nmenu, group);
-
- ADD_NAMED_SEP(popup, second, nmenu);
-
IMAGE_ITEM(popup, back, gtkBack, nmenu, group);
IMAGE_ITEM(popup, forward, gtkForward, nmenu, group);
diff --git a/gtk/menu.h b/gtk/menu.h
index 48b3f0f5c..59d10b1c2 100644
--- a/gtk/menu.h
+++ b/gtk/menu.h
@@ -166,12 +166,6 @@ struct nsgtk_popup_menu {
GtkWidget *first_separator;
- GtkImageMenuItem *opentab_menuitem;
- GtkImageMenuItem *openwin_menuitem;
- GtkImageMenuItem *savelink_menuitem;
-
- GtkWidget *second_separator;
-
/* navigation entries */
GtkImageMenuItem *back_menuitem;
GtkImageMenuItem *forward_menuitem;
diff --git a/gtk/scaffolding.c b/gtk/scaffolding.c
index 5b05fccdf..05d3f3377 100644
--- a/gtk/scaffolding.c
+++ b/gtk/scaffolding.c
@@ -151,7 +151,7 @@ static struct contextual_content current_menu_ctx;
* Helper to hide popup menu entries by grouping
*/
static void popup_menu_hide(struct nsgtk_popup_menu *menu, bool submenu,
- bool link, bool nav, bool cnp, bool custom)
+ bool nav, bool cnp, bool custom)
{
if (submenu){
gtk_widget_hide(GTK_WIDGET(menu->file_menuitem));
@@ -163,14 +163,6 @@ static void popup_menu_hide(struct nsgtk_popup_menu *menu, bool submenu,
gtk_widget_hide(menu->first_separator);
}
- if (link) {
- gtk_widget_hide(GTK_WIDGET(menu->opentab_menuitem));
- gtk_widget_hide(GTK_WIDGET(menu->openwin_menuitem));
- gtk_widget_hide(GTK_WIDGET(menu->savelink_menuitem));
-
- gtk_widget_hide(menu->second_separator);
- }
-
if (nav) {
gtk_widget_hide(GTK_WIDGET(menu->back_menuitem));
gtk_widget_hide(GTK_WIDGET(menu->forward_menuitem));
@@ -194,7 +186,7 @@ static void popup_menu_hide(struct nsgtk_popup_menu *menu, bool submenu,
* Helper to show popup menu entries by grouping
*/
static void popup_menu_show(struct nsgtk_popup_menu *menu, bool submenu,
- bool link, bool nav, bool cnp, bool custom)
+ bool nav, bool cnp, bool custom)
{
if (submenu){
gtk_widget_show(GTK_WIDGET(menu->file_menuitem));
@@ -206,14 +198,6 @@ static void popup_menu_show(struct nsgtk_popup_menu *menu, bool submenu,
gtk_widget_show(menu->first_separator);
}
- if (link) {
- gtk_widget_show(GTK_WIDGET(menu->opentab_menuitem));
- gtk_widget_show(GTK_WIDGET(menu->openwin_menuitem));
- gtk_widget_show(GTK_WIDGET(menu->savelink_menuitem));
-
- gtk_widget_show(menu->second_separator);
- }
-
if (nav) {
gtk_widget_show(GTK_WIDGET(menu->back_menuitem));
gtk_widget_show(GTK_WIDGET(menu->forward_menuitem));
@@ -368,17 +352,6 @@ static guint nsgtk_scaffolding_update_edit_actions_sensitivity(
(g->buttons[PASTE_BUTTON]->sensitivity));
}
-static void nsgtk_scaffolding_enable_link_operations_sensitivity(
- struct gtk_scaffolding *g)
-{
-
- gtk_widget_set_sensitive(GTK_WIDGET(g->menu_popup->savelink_menuitem), TRUE);
- gtk_widget_set_sensitive(GTK_WIDGET(g->menu_popup->opentab_menuitem), TRUE);
- gtk_widget_set_sensitive(GTK_WIDGET(g->menu_popup->openwin_menuitem), TRUE);
-
- popup_menu_show(g->menu_popup, false, true, false, false, false);
-
-}
static void nsgtk_scaffolding_enable_edit_actions_sensitivity(
struct gtk_scaffolding *g)
@@ -389,7 +362,7 @@ static void nsgtk_scaffolding_enable_edit_actions_sensitivity(
g->buttons[CUT_BUTTON]->sensitivity = true;
nsgtk_scaffolding_set_sensitivity(g);
- popup_menu_show(g->menu_popup, false, false, false, true, false);
+ popup_menu_show(g->menu_popup, false, false, true, false);
}
/* signal handling functions for the toolbar, URL bar, and menu bar */
@@ -412,7 +385,6 @@ static gboolean nsgtk_window_edit_menu_hidden(GtkWidget *widget,
static gboolean nsgtk_window_popup_menu_hidden(GtkWidget *widget,
struct gtk_scaffolding *g)
{
- nsgtk_scaffolding_enable_link_operations_sensitivity(g);
nsgtk_scaffolding_enable_edit_actions_sensitivity(g);
return TRUE;
}
@@ -460,8 +432,8 @@ static gboolean nsgtk_window_tool_bar_clicked(GtkToolbar *toolbar,
struct gtk_scaffolding *g = (struct gtk_scaffolding *)data;
/* set visibility for right-click popup menu */
- popup_menu_hide(g->menu_popup, true, true, false, true, false);
- popup_menu_show(g->menu_popup, false, false, false, false, true);
+ popup_menu_hide(g->menu_popup, true, false, true, false);
+ popup_menu_show(g->menu_popup, false, false, false, true);
gtk_menu_popup(g->menu_popup->popup_menu, NULL, NULL, NULL, NULL, 0,
gtk_get_current_event_time());
@@ -1227,8 +1199,8 @@ MENUHANDLER(menubar)
gtk_widget_show(GTK_WIDGET(g->menu_bar->bar_menu));
- popup_menu_show(g->menu_popup, false, true, true, true, true);
- popup_menu_hide(g->menu_popup, true, false, false, false, false);
+ popup_menu_show(g->menu_popup, false, true, true, true);
+ popup_menu_hide(g->menu_popup, true, false, false, false);
} else {
w = GTK_WIDGET(g->menu_popup->view_submenu->toolbars_submenu->menubar_menuitem);
if (gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM(w)))
@@ -1242,7 +1214,7 @@ MENUHANDLER(menubar)
gtk_widget_hide(GTK_WIDGET(g->menu_bar->bar_menu));
- popup_menu_show(g->menu_popup, true, true, true, true, true);
+ popup_menu_show(g->menu_popup, true, true, true, true);
}
return TRUE;
@@ -1812,15 +1784,6 @@ nsgtk_new_scaffolding_popup(struct gtk_scaffolding *g, GtkAccelGroup *group)
g_signal_connect(nmenu->popup_menu, "hide",
G_CALLBACK(nsgtk_window_popup_menu_hidden), g);
- g_signal_connect(nmenu->savelink_menuitem, "activate",
- G_CALLBACK(nsgtk_on_savelink_activate_menu), g);
-
- g_signal_connect(nmenu->opentab_menuitem, "activate",
- G_CALLBACK(nsgtk_on_link_opentab_activate_menu), g);
-
- g_signal_connect(nmenu->openwin_menuitem, "activate",
- G_CALLBACK(nsgtk_on_link_openwin_activate_menu), g);
-
g_signal_connect(nmenu->cut_menuitem, "activate",
G_CALLBACK(nsgtk_on_cut_activate_menu), g);
@@ -1834,7 +1797,7 @@ nsgtk_new_scaffolding_popup(struct gtk_scaffolding *g, GtkAccelGroup *group)
G_CALLBACK(nsgtk_on_customize_activate_menu), g);
/* set initial popup menu visibility */
- popup_menu_hide(nmenu, true, false, false, false, true);
+ popup_menu_hide(nmenu, true, false, false, true);
return nmenu;
}
@@ -2605,33 +2568,31 @@ void nsgtk_scaffolding_context_menu(struct gtk_scaffolding *g,
if (current_menu_ctx.link_url != NULL) {
/* menu is opening over a link */
gtkmenu = g->link_menu->link_menu;
-
- popup_menu_show(g->menu_popup, false, true, false, false, false);
-
} else {
gtkmenu = g->menu_popup->popup_menu;
- popup_menu_hide(g->menu_popup, false, true, false, false, false);
-
nsgtk_scaffolding_update_edit_actions_sensitivity(g);
- if (!(g->buttons[COPY_BUTTON]->sensitivity))
+ if (!(g->buttons[COPY_BUTTON]->sensitivity)) {
gtk_widget_hide(GTK_WIDGET(g->menu_popup->copy_menuitem));
- else
+ } else {
gtk_widget_show(GTK_WIDGET(g->menu_popup->copy_menuitem));
+ }
- if (!(g->buttons[CUT_BUTTON]->sensitivity))
+ if (!(g->buttons[CUT_BUTTON]->sensitivity)) {
gtk_widget_hide(GTK_WIDGET(g->menu_popup->cut_menuitem));
- else
+ } else {
gtk_widget_show(GTK_WIDGET(g->menu_popup->cut_menuitem));
+ }
- if (!(g->buttons[PASTE_BUTTON]->sensitivity))
+ if (!(g->buttons[PASTE_BUTTON]->sensitivity)) {
gtk_widget_hide(GTK_WIDGET(g->menu_popup->paste_menuitem));
- else
+ } else {
gtk_widget_show(GTK_WIDGET(g->menu_popup->paste_menuitem));
+ }
/* hide customize */
- popup_menu_hide(g->menu_popup, false, false, false, false, true);
+ popup_menu_hide(g->menu_popup, false, false, false, true);
}
gtk_menu_popup(gtkmenu, NULL, NULL, NULL, NULL, 0,