Remove some api that was just silly wrappers. Return GtkTreeViewColumn on

2003-04-15  Marco Pesenti Gritti  <marco@it.gnome.org>

	* TODO:
	* src/bookmarks/ephy-bookmarks-editor.c: (cmd_select_all),
	(ephy_bookmarks_editor_construct):
	* src/bookmarks/ephy-node-view.c: (ephy_node_view_add_column),
	(ephy_node_view_remove):
	* src/bookmarks/ephy-node-view.h:

	Remove some api that was just silly wrappers.
	Return GtkTreeViewColumn on add_column.

	* src/ephy-nautilus-view.c: (gnv_embed_new_window_cb):
	* src/ephy-shell.c: (ephy_shell_new_window_cb),
	(ephy_shell_new_tab):
	* src/ephy-tab.c: (ephy_tab_new_window_cb):
	* src/ephy-window.c: (ephy_window_add_tab):
	* src/ephy-window.h:
	* src/session.c: (parse_embed):

	Ever open tab as last on File->New tab and similars.

3 files changed, 25 insertions, 37 deletions

diff --git a/src/bookmarks/ephy-bookmarks-editor.c b/src/bookmarks/ephy-bookmarks-editor.c
index 1dbcac36a..7239f2d20 100644
--- a/src/bookmarks/ephy-bookmarks-editor.c
+++ b/src/bookmarks/ephy-bookmarks-editor.c
@@ -350,14 +350,18 @@ cmd_select_all (EggAction *action,
 		EphyBookmarksEditor *editor)
 {
 	GtkWidget *widget = gtk_window_get_focus (GTK_WINDOW (editor));
+	GtkWidget *bm_view = editor->priv->bm_view;
 
 	if (GTK_IS_EDITABLE (widget))
 	{
 		gtk_editable_select_region (GTK_EDITABLE (widget), 0, -1);
 	}
-	else if (gtk_widget_is_focus (editor->priv->bm_view))
+	else if (gtk_widget_is_focus (bm_view))
 	{
-		ephy_node_view_select_all (EPHY_NODE_VIEW (editor->priv->bm_view));
+		GtkTreeSelection *sel;
+
+		sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (bm_view));
+		gtk_tree_selection_select_all (sel);
 	}
 }
 
@@ -831,6 +835,7 @@ node_dropped_cb (EphyNodeView *view, EphyNode *node,
 static void
 ephy_bookmarks_editor_construct (EphyBookmarksEditor *editor)
 {
+	GtkTreeSelection *selection;
 	GtkWidget *hbox, *vbox;
 	GtkWidget *bm_view, *key_view;
 	GtkWidget *scrolled_window;
@@ -908,7 +913,8 @@ ephy_bookmarks_editor_construct (EphyBookmarksEditor *editor)
 	ephy_node_view_enable_drag_dest (EPHY_NODE_VIEW (key_view),
 					 topic_drag_dest_types,
 				         n_topic_drag_dest_types);
-	ephy_node_view_set_browse_mode (EPHY_NODE_VIEW (key_view));
+	selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (key_view));
+	gtk_tree_selection_set_mode (selection, GTK_SELECTION_BROWSE);
 	ephy_node_view_add_column (EPHY_NODE_VIEW (key_view), _("Topics"),
 			           EPHY_TREE_MODEL_NODE_COL_KEYWORD, TRUE, TRUE);
 	gtk_container_add (GTK_CONTAINER (scrolled_window), key_view);
diff --git a/src/bookmarks/ephy-node-view.c b/src/bookmarks/ephy-node-view.c
index 8c75a40cf..1a2a0cf3d 100644
--- a/src/bookmarks/ephy-node-view.c
+++ b/src/bookmarks/ephy-node-view.c
@@ -556,7 +556,7 @@ cell_renderer_edited (GtkCellRendererText *cell,
 	g_value_unset (&value);
 }
 
-void
+GtkTreeViewColumn *
 ephy_node_view_add_column (EphyNodeView *view,
 			   const char  *title,
 			   EphyTreeModelNodeColumn column,
@@ -566,7 +566,7 @@ ephy_node_view_add_column (EphyNodeView *view,
 	GtkTreeViewColumn *gcolumn;
 	GtkCellRenderer *renderer;
 
-	g_return_if_fail (!editable || view->priv->editable_renderer == NULL);
+	g_return_val_if_fail (!editable || view->priv->editable_renderer == NULL, NULL);
 
 	gcolumn = (GtkTreeViewColumn *) gtk_tree_view_column_new ();
 	renderer = gtk_cell_renderer_text_new ();
@@ -594,6 +594,8 @@ ephy_node_view_add_column (EphyNodeView *view,
 		view->priv->default_sort_column_id = column;
 		g_idle_add ((GSourceFunc) set_sort_column_id, view);
 	}
+
+	return gcolumn;
 }
 
 void
@@ -665,13 +667,6 @@ ephy_node_view_get_selection (EphyNodeView *view)
 }
 
 void
-ephy_node_view_select_all (EphyNodeView *view)
-{
-	GtkTreeSelection *sel = gtk_tree_view_get_selection (GTK_TREE_VIEW (view));
-	gtk_tree_selection_select_all (sel);
-}
-
-void
 ephy_node_view_remove (EphyNodeView *view)
 {
 	GList *list;
@@ -687,15 +682,6 @@ ephy_node_view_remove (EphyNodeView *view)
 }
 
 void
-ephy_node_view_set_browse_mode (EphyNodeView *view)
-{
-	GtkTreeSelection *selection;
-
-	selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (view));
-	gtk_tree_selection_set_mode (selection, GTK_SELECTION_BROWSE);
-}
-
-void
 ephy_node_view_select_node (EphyNodeView *view,
 			    EphyNode *node)
 {
diff --git a/src/bookmarks/ephy-node-view.h b/src/bookmarks/ephy-node-view.h
index 1943695a6..9aa91a876 100644
--- a/src/bookmarks/ephy-node-view.h
+++ b/src/bookmarks/ephy-node-view.h
@@ -55,46 +55,42 @@ typedef struct
 	void (*show_popup)          (EphyNodeView *view);
 } EphyNodeViewClass;
 
-GType         ephy_node_view_get_type                 (void);
+GType              ephy_node_view_get_type	      (void);
 
-GtkWidget    *ephy_node_view_new                      (EphyNode *root,
+GtkWidget         *ephy_node_view_new                 (EphyNode *root,
 					               EphyNodeFilter *filter);
 
-void	      ephy_node_view_enable_dnd		      (EphyNodeView *view);
+void	           ephy_node_view_enable_dnd	      (EphyNodeView *view);
 
-void	      ephy_node_view_add_column		      (EphyNodeView *view,
+GtkTreeViewColumn *ephy_node_view_add_column	      (EphyNodeView *view,
 						       const char  *title,
 						       EphyTreeModelNodeColumn column,
 						       gboolean sortable,
 						       gboolean editable);
 
-void	      ephy_node_view_add_icon_column	      (EphyNodeView *view,
+void	           ephy_node_view_add_icon_column     (EphyNodeView *view,
 						       EphyTreeModelNodeColumn column);
 
-void	      ephy_node_view_remove		      (EphyNodeView *view);
+void		   ephy_node_view_remove              (EphyNodeView *view);
 
-gboolean      ephy_node_view_has_selection            (EphyNodeView *view,
+gboolean           ephy_node_view_has_selection       (EphyNodeView *view,
 						       gboolean *multiple);
 
-GList        *ephy_node_view_get_selection            (EphyNodeView *view);
+GList             *ephy_node_view_get_selection       (EphyNodeView *view);
 
-void	      ephy_node_view_select_all		      (EphyNodeView *view);
-
-void	      ephy_node_view_set_browse_mode	      (EphyNodeView *view);
-
-void	      ephy_node_view_select_node              (EphyNodeView *view,
+void	           ephy_node_view_select_node         (EphyNodeView *view,
 			                               EphyNode *node);
 
-void	      ephy_node_view_enable_drag_source       (EphyNodeView *view,
+void	           ephy_node_view_enable_drag_source  (EphyNodeView *view,
 						       GtkTargetEntry *types,
 						       int n_types,
 						       guint prop_id);
 
-void	      ephy_node_view_enable_drag_dest	      (EphyNodeView *view,
+void	           ephy_node_view_enable_drag_dest    (EphyNodeView *view,
 						       GtkTargetEntry *types,
 						       int n_types);
 
-void	      ephy_node_view_edit		      (EphyNodeView *view);
+void		   ephy_node_view_edit		      (EphyNodeView *view);
 
 G_END_DECLS