#include "config.h"
#endif
+#include <string.h>
+
#include <gtk/gtk.h>
#include <epan/prefs.h>
static GtkWidget *remove_bt, *field_te, *field_lb, *fmt_cmb;
-static gint cur_fmt, cur_row;
+static gulong column_menu_changed_handler_id;
+static gulong column_field_changed_handler_id;
+static gulong column_row_deleted_handler_id;
-static void column_list_select_cb(GtkTreeSelection *, gpointer);
-static void column_field_changed_cb(GtkEditable *, gpointer);
static void column_list_new_cb(GtkWidget *, gpointer);
-static void column_menu_changed_cb(GtkWidget *, gpointer);
static void column_list_delete_cb(GtkWidget *, gpointer);
+static void column_list_select_cb(GtkTreeSelection *, gpointer);
+static void column_menu_changed_cb(GtkWidget *, gpointer);
+static void column_field_changed_cb(GtkEditable *, gpointer);
static void column_dnd_row_deleted_cb(GtkTreeModel *, GtkTreePath *, gpointer);
static gboolean column_title_changed_cb(GtkCellRendererText *, const gchar *, const gchar *, gpointer);
-#define E_COL_NAME_KEY "column_name"
-#define E_COL_LBL_KEY "column_label"
-#define E_COL_CM_KEY "in_col_cancel_mode"
-
-/* Create and display the column selection widgets. */
-/* Called when the 'Columns' preference notebook page is selected. */
+/*
+ * Create and display the column selection widgets.
+ * Called as part of the creation of the Preferences notebook ( Edit ! Preferences )
+ */
GtkWidget *
column_prefs_show(GtkWidget *prefs_window) {
- GtkWidget *main_vb, *bottom_hb, *column_l, *add_bt,
- *tb, *lb;
- GtkWidget *list_vb, *list_lb, *list_sc;
- GtkWidget *add_remove_vb;
- GtkWidget *props_fr, *props_hb;
- GList *clp = NULL;
- fmt_data *cfmt;
- gint i;
- gchar *fmt;
- const gchar *column_titles[] = {"Title", "Format"};
- GtkListStore *store;
- GtkCellRenderer *renderer;
- GtkTreeViewColumn *column;
- GtkTreeSelection *sel;
- GtkTreeIter iter;
- GtkTreeIter first_iter;
- gint first_row = TRUE;
-
- /* Container for each row of widgets */
- main_vb = gtk_vbox_new(FALSE, 5);
- gtk_container_set_border_width(GTK_CONTAINER(main_vb), 5);
- gtk_widget_show(main_vb);
- g_object_set_data(G_OBJECT(GTK_OBJECT(main_vb)), E_COL_CM_KEY, (gpointer)FALSE);
-
- /* Top row: Columns list frame */
- /*list_fr = gtk_frame_new("Columns");
- gtk_box_pack_start (GTK_BOX (main_vb), list_fr, TRUE, TRUE, 0);
- gtk_widget_show(list_fr);
- */
-
- list_vb = gtk_vbox_new (FALSE, 0);
- gtk_container_set_border_width (GTK_CONTAINER (list_vb), 5);
- gtk_widget_show (list_vb);
- gtk_box_pack_start (GTK_BOX (main_vb), list_vb, TRUE, TRUE, 0);
- /*gtk_container_add(GTK_CONTAINER(list_fr), list_vb);*/
-
- list_lb = gtk_label_new (("[First list entry will be displayed left]"));
- gtk_widget_show (list_lb);
- gtk_box_pack_start (GTK_BOX (list_vb), list_lb, FALSE, FALSE, 0);
-
- list_sc = scrolled_window_new(NULL, NULL);
- gtk_scrolled_window_set_shadow_type(GTK_SCROLLED_WINDOW(list_sc),
- GTK_SHADOW_IN);
- gtk_container_add(GTK_CONTAINER(list_vb), list_sc);
- gtk_widget_show(list_sc);
-
- store = gtk_list_store_new(3, G_TYPE_STRING, G_TYPE_STRING, G_TYPE_POINTER);
- g_signal_connect(GTK_TREE_MODEL(store), "row-deleted", G_CALLBACK(column_dnd_row_deleted_cb), NULL);
-
- column_l = tree_view_new(GTK_TREE_MODEL(store));
- gtk_tree_view_set_headers_visible(GTK_TREE_VIEW(column_l), TRUE);
- gtk_tree_view_set_headers_clickable(GTK_TREE_VIEW(column_l), FALSE);
- gtk_tree_view_set_reorderable(GTK_TREE_VIEW(column_l), TRUE);
- renderer = gtk_cell_renderer_text_new();
- g_object_set(G_OBJECT(renderer), "editable", TRUE, NULL);
- g_signal_connect (renderer, "edited", G_CALLBACK(column_title_changed_cb), GTK_TREE_MODEL(store));
- column = gtk_tree_view_column_new_with_attributes(column_titles[0], renderer,
- "text", 0, NULL);
- gtk_tree_view_column_set_sizing(column, GTK_TREE_VIEW_COLUMN_AUTOSIZE);
- gtk_tree_view_append_column(GTK_TREE_VIEW(column_l), column);
- renderer = gtk_cell_renderer_text_new();
- column = gtk_tree_view_column_new_with_attributes(column_titles[1], renderer,
- "text", 1, NULL);
- gtk_tree_view_column_set_sizing(column, GTK_TREE_VIEW_COLUMN_AUTOSIZE);
- gtk_tree_view_append_column(GTK_TREE_VIEW(column_l), column);
- /* XXX - make this match the packet list prefs? */
- sel = gtk_tree_view_get_selection(GTK_TREE_VIEW(column_l));
- gtk_tree_selection_set_mode(sel, GTK_SELECTION_SINGLE);
- g_signal_connect(sel, "changed", G_CALLBACK(column_list_select_cb), column_l);
-
- gtk_container_add(GTK_CONTAINER(list_sc), column_l);
- gtk_widget_show(column_l);
-
- clp = g_list_first(prefs.col_list);
- while (clp) {
- cfmt = (fmt_data *) clp->data;
- cur_fmt = get_column_format_from_str(cfmt->fmt);
- if (cur_fmt == COL_CUSTOM) {
- fmt = g_strdup_printf("%s (%s)", col_format_desc(cur_fmt), cfmt->custom_field);
- } else {
- fmt = g_strdup_printf("%s", col_format_desc(cur_fmt));
- }
- gtk_list_store_append(store, &iter);
- gtk_list_store_set(store, &iter, 0, cfmt->title, 1, fmt, 2, clp, -1);
- if (first_row) {
- first_iter = iter;
- first_row = FALSE;
+ GtkWidget *main_vb, *bottom_hb, *column_l, *add_bt, *tb, *lb;
+ GtkWidget *list_vb, *list_lb, *list_sc;
+ GtkWidget *add_remove_vb;
+ GtkWidget *props_fr, *props_hb;
+ GList *clp;
+ fmt_data *cfmt;
+ gint i;
+ gchar *fmt;
+ gint cur_fmt;
+ const gchar *column_titles[] = {"Title", "Format"};
+ GtkListStore *store;
+ GtkCellRenderer *renderer;
+ GtkTreeViewColumn *column;
+ GtkTreeSelection *sel;
+ GtkTreeIter iter;
+ GtkTreeIter first_iter;
+ gint first_row = TRUE;
+
+ GtkTooltips *tooltips = gtk_tooltips_new();
+
+ /* Container for each row of widgets */
+ main_vb = gtk_vbox_new(FALSE, 5);
+ gtk_container_set_border_width(GTK_CONTAINER(main_vb), 5);
+ gtk_widget_show(main_vb);
+
+ list_vb = gtk_vbox_new (FALSE, 0);
+ gtk_container_set_border_width (GTK_CONTAINER (list_vb), 5);
+ gtk_widget_show (list_vb);
+ gtk_box_pack_start (GTK_BOX (main_vb), list_vb, TRUE, TRUE, 0);
+
+ list_lb = gtk_label_new (("[The first list entry will be displayed as the leftmost column]"));
+ gtk_widget_show (list_lb);
+ gtk_box_pack_start (GTK_BOX (list_vb), list_lb, FALSE, FALSE, 0);
+
+ list_sc = scrolled_window_new(NULL, NULL);
+ gtk_scrolled_window_set_shadow_type(GTK_SCROLLED_WINDOW(list_sc), GTK_SHADOW_IN);
+ gtk_container_add(GTK_CONTAINER(list_vb), list_sc);
+ gtk_widget_show(list_sc);
+
+ store = gtk_list_store_new(3, G_TYPE_STRING, G_TYPE_STRING, G_TYPE_POINTER);
+ column_row_deleted_handler_id =
+ g_signal_connect(GTK_TREE_MODEL(store), "row-deleted", G_CALLBACK(column_dnd_row_deleted_cb), NULL);
+
+ column_l = tree_view_new(GTK_TREE_MODEL(store));
+ gtk_tree_view_set_headers_visible(GTK_TREE_VIEW(column_l), TRUE);
+ gtk_tree_view_set_headers_clickable(GTK_TREE_VIEW(column_l), FALSE);
+ gtk_tree_view_set_reorderable(GTK_TREE_VIEW(column_l), TRUE);
+ gtk_tooltips_set_tip (tooltips, column_l,
+ "Click on a title to change its name.\nDrag an item to change its order.", NULL);
+
+ renderer = gtk_cell_renderer_text_new();
+ g_object_set(G_OBJECT(renderer), "editable", TRUE, NULL);
+ g_signal_connect (renderer, "edited", G_CALLBACK(column_title_changed_cb), GTK_TREE_MODEL(store));
+ column = gtk_tree_view_column_new_with_attributes(column_titles[0], renderer, "text", 0, NULL);
+ gtk_tree_view_column_set_sizing(column, GTK_TREE_VIEW_COLUMN_AUTOSIZE);
+ gtk_tree_view_append_column(GTK_TREE_VIEW(column_l), column);
+
+ renderer = gtk_cell_renderer_text_new();
+ column = gtk_tree_view_column_new_with_attributes(column_titles[1], renderer, "text", 1, NULL);
+ gtk_tree_view_column_set_sizing(column, GTK_TREE_VIEW_COLUMN_AUTOSIZE);
+ gtk_tree_view_append_column(GTK_TREE_VIEW(column_l), column);
+
+ /* XXX - make this match the packet list prefs? */
+ sel = gtk_tree_view_get_selection(GTK_TREE_VIEW(column_l));
+ gtk_tree_selection_set_mode(sel, GTK_SELECTION_SINGLE);
+ g_signal_connect(sel, "changed", G_CALLBACK(column_list_select_cb), NULL);
+
+ gtk_container_add(GTK_CONTAINER(list_sc), column_l);
+ gtk_widget_show(column_l);
+
+ clp = g_list_first(prefs.col_list);
+ while (clp) {
+ cfmt = (fmt_data *) clp->data;
+ cur_fmt = get_column_format_from_str(cfmt->fmt);
+ if (cur_fmt == COL_CUSTOM) {
+ fmt = g_strdup_printf("%s (%s)", col_format_desc(cur_fmt), cfmt->custom_field);
+ } else {
+ fmt = g_strdup_printf("%s", col_format_desc(cur_fmt));
+ }
+ gtk_list_store_append(store, &iter);
+ gtk_list_store_set(store, &iter, 0, cfmt->title, 1, fmt, 2, clp, -1);
+ if (first_row) {
+ first_iter = iter;
+ first_row = FALSE;
+ }
+ clp = clp->next;
+ g_free (fmt);
}
- clp = clp->next;
- g_free (fmt);
- }
- g_object_unref(G_OBJECT(store));
-
+ g_object_unref(G_OBJECT(store));
+
+ /* Bottom row: Add/remove buttons and properties */
+ bottom_hb = gtk_hbox_new(FALSE, 5);
+ gtk_box_pack_start (GTK_BOX (main_vb), bottom_hb, FALSE, TRUE, 0);
+ gtk_widget_show(bottom_hb);
+
+ /* Add / remove buttons */
+ add_remove_vb = gtk_vbox_new (FALSE, 0);
+ gtk_container_set_border_width (GTK_CONTAINER (add_remove_vb), 5);
+ gtk_box_pack_start (GTK_BOX (bottom_hb), add_remove_vb, FALSE, TRUE, 0);
+ gtk_widget_show(add_remove_vb);
+
+ add_bt = gtk_button_new_from_stock(GTK_STOCK_ADD);
+ g_signal_connect(add_bt, "clicked", G_CALLBACK(column_list_new_cb), column_l);
+ gtk_box_pack_start (GTK_BOX (add_remove_vb), add_bt, FALSE, FALSE, 5);
+ gtk_widget_show(add_bt);
+
+ remove_bt = gtk_button_new_from_stock(GTK_STOCK_REMOVE);
+ gtk_widget_set_sensitive(remove_bt, FALSE);
+ g_signal_connect(remove_bt, "clicked", G_CALLBACK(column_list_delete_cb), column_l);
+ gtk_box_pack_start (GTK_BOX (add_remove_vb), remove_bt, FALSE, FALSE, 5);
+ gtk_widget_show(remove_bt);
- /* Bottom row: Add/remove buttons and properties */
- bottom_hb = gtk_hbox_new(FALSE, 5);
- gtk_box_pack_start (GTK_BOX (main_vb), bottom_hb, FALSE, TRUE, 0);
- /*gtk_container_add(GTK_CONTAINER(main_vb), bottom_hb);*/
- gtk_widget_show(bottom_hb);
-
-
- /* Add / remove buttons */
- add_remove_vb = gtk_vbox_new (FALSE, 0);
- gtk_container_set_border_width (GTK_CONTAINER (add_remove_vb), 5);
- gtk_box_pack_start (GTK_BOX (bottom_hb), add_remove_vb, FALSE, TRUE, 0);
- /*gtk_container_add(GTK_CONTAINER(bottom_hb), add_remove_vb);*/
- gtk_widget_show(add_remove_vb);
-
- add_bt = gtk_button_new_from_stock(GTK_STOCK_ADD);
- g_signal_connect(add_bt, "clicked", G_CALLBACK(column_list_new_cb), column_l);
- gtk_box_pack_start (GTK_BOX (add_remove_vb), add_bt, FALSE, FALSE, 5);
- gtk_widget_show(add_bt);
-
- remove_bt = gtk_button_new_from_stock(GTK_STOCK_REMOVE);
- gtk_widget_set_sensitive(remove_bt, FALSE);
- g_signal_connect(remove_bt, "clicked", G_CALLBACK(column_list_delete_cb), column_l);
- gtk_box_pack_start (GTK_BOX (add_remove_vb), remove_bt, FALSE, FALSE, 5);
- gtk_widget_show(remove_bt);
-
- /* properties frame */
- props_fr = gtk_frame_new("Properties");
- gtk_box_pack_start (GTK_BOX (bottom_hb), props_fr, TRUE, TRUE, 0);
- gtk_widget_show(props_fr);
-
- /* Column name entry and format selection */
- tb = gtk_table_new(1, 4, FALSE);
- gtk_container_set_border_width(GTK_CONTAINER(tb), 5);
- gtk_container_add(GTK_CONTAINER(props_fr), tb);
- gtk_table_set_row_spacings(GTK_TABLE(tb), 10);
- gtk_table_set_col_spacings(GTK_TABLE(tb), 15);
- gtk_widget_show(tb);
-
- lb = gtk_label_new("Format:");
- gtk_misc_set_alignment(GTK_MISC(lb), 1.0, 0.5);
- gtk_table_attach_defaults(GTK_TABLE(tb), lb, 0, 1, 0, 1);
- gtk_widget_show(lb);
-
- props_hb = gtk_hbox_new(FALSE, 5);
- gtk_table_attach(GTK_TABLE(tb), props_hb, 1, 2, 0, 1, GTK_FILL,
- GTK_SHRINK, 0, 0);
- gtk_widget_show(props_hb);
-
- field_lb = gtk_label_new("Field name:");
- gtk_misc_set_alignment(GTK_MISC(field_lb), 1.0, 0.5);
- gtk_table_attach_defaults(GTK_TABLE(tb), field_lb, 2, 3, 0, 1);
- gtk_widget_hide(field_lb);
-
- field_te = gtk_entry_new();
- g_object_set_data (G_OBJECT(field_te), E_FILT_FIELD_NAME_ONLY_KEY, "");
- g_signal_connect(field_te, "changed", G_CALLBACK(filter_te_syntax_check_cb), NULL);
- g_object_set_data(G_OBJECT(main_vb), E_FILT_AUTOCOMP_PTR_KEY, NULL);
- g_signal_connect(field_te, "key-press-event", G_CALLBACK (filter_string_te_key_pressed_cb), NULL);
- g_signal_connect(prefs_window, "key-press-event", G_CALLBACK (filter_parent_dlg_key_pressed_cb), NULL);
- colorize_filter_te_as_empty(field_te);
- gtk_table_attach_defaults(GTK_TABLE(tb), field_te, 3, 4, 0, 1);
- gtk_widget_set_sensitive(field_te, FALSE);
- gtk_widget_hide(field_te);
-
- fmt_cmb = gtk_combo_box_new_text();
-
- for (i = 0; i < NUM_COL_FMTS; i++)
- gtk_combo_box_append_text(GTK_COMBO_BOX(fmt_cmb), col_format_desc(i));
-
- g_signal_connect(fmt_cmb, "changed", G_CALLBACK(column_menu_changed_cb), column_l);
-
- cur_fmt = 0;
- gtk_combo_box_set_active(GTK_COMBO_BOX(fmt_cmb), cur_fmt);
- gtk_widget_set_sensitive(fmt_cmb, FALSE);
- gtk_box_pack_start(GTK_BOX(props_hb), fmt_cmb, FALSE, FALSE, 0);
- gtk_widget_show(fmt_cmb);
-
- /* select the first row */
- gtk_tree_selection_select_iter(sel, &first_iter);
-
- return(main_vb);
-}
+ /* properties frame */
+ props_fr = gtk_frame_new("Properties");
+ gtk_box_pack_start (GTK_BOX (bottom_hb), props_fr, TRUE, TRUE, 0);
+ gtk_widget_show(props_fr);
+
+ /* Column name entry and format selection */
+ tb = gtk_table_new(1, 4, FALSE);
+ gtk_container_set_border_width(GTK_CONTAINER(tb), 5);
+ gtk_container_add(GTK_CONTAINER(props_fr), tb);
+ gtk_table_set_row_spacings(GTK_TABLE(tb), 10);
+ gtk_table_set_col_spacings(GTK_TABLE(tb), 15);
+ gtk_widget_show(tb);
+
+ lb = gtk_label_new("Format:");
+ gtk_misc_set_alignment(GTK_MISC(lb), 1.0f, 0.5f);
+ gtk_table_attach_defaults(GTK_TABLE(tb), lb, 0, 1, 0, 1);
+ gtk_widget_show(lb);
+
+ props_hb = gtk_hbox_new(FALSE, 5);
+ gtk_table_attach(GTK_TABLE(tb), props_hb, 1, 2, 0, 1, GTK_FILL, GTK_SHRINK, 0, 0);
+ gtk_widget_show(props_hb);
+
+ field_lb = gtk_label_new("Field name:");
+ gtk_misc_set_alignment(GTK_MISC(field_lb), 1.0f, 0.5f);
+ gtk_table_attach_defaults(GTK_TABLE(tb), field_lb, 2, 3, 0, 1);
+ gtk_widget_hide(field_lb);
+
+ field_te = gtk_entry_new();
+ g_object_set_data (G_OBJECT(field_te), E_FILT_FIELD_NAME_ONLY_KEY, "");
+ g_signal_connect(field_te, "changed", G_CALLBACK(filter_te_syntax_check_cb), NULL);
+
+ /* XXX: column_field_changed_cb will be called for every character entered in the entry box. */
+ /* Consider Changing logic so that the field is "accepted" only when a return is entered ?? */
+ /* Also: entry shouldn't be accepted if it's not a valid filter ? */
+ column_field_changed_handler_id =
+ g_signal_connect(field_te, "changed", G_CALLBACK(column_field_changed_cb), column_l);
-/* For each selection, set the entry and option menu widgets to match
- the currently selected item. Set the up/down button sensitivity.
- Draw focus to the entry widget. */
-static void
-column_list_select_cb(GtkTreeSelection *sel, gpointer data)
-{
- GtkTreeView *column_l = GTK_TREE_VIEW(data);
- fmt_data *cfmt;
- GList *clp;
- GtkTreeModel *model;
- GtkTreeIter iter;
- GtkTreePath *path;
- gchar *str_path;
+ g_object_set_data(G_OBJECT(main_vb), E_FILT_AUTOCOMP_PTR_KEY, NULL);
+ g_signal_connect(field_te, "key-press-event", G_CALLBACK (filter_string_te_key_pressed_cb), NULL);
+ g_signal_connect(prefs_window, "key-press-event", G_CALLBACK (filter_parent_dlg_key_pressed_cb), NULL);
+ colorize_filter_te_as_empty(field_te);
+ gtk_table_attach_defaults(GTK_TABLE(tb), field_te, 3, 4, 0, 1);
+ gtk_widget_set_sensitive(field_te, FALSE);
+ gtk_widget_hide(field_te);
- /* if something was selected */
- if (gtk_tree_selection_get_selected(sel, &model, &iter))
- {
- gtk_tree_model_get(model, &iter, 2, &clp, -1);
- g_assert(clp != NULL);
- cfmt = (fmt_data *) clp->data;
- cur_fmt = get_column_format_from_str(cfmt->fmt);
- g_assert(cur_fmt != -1); /* It should always be valid */
+ fmt_cmb = gtk_combo_box_new_text();
- path = gtk_tree_model_get_path(model, &iter);
- str_path = gtk_tree_path_to_string(path);
- cur_row = atoi(str_path);
- g_free(str_path);
- gtk_tree_path_free(path);
+ for (i = 0; i < NUM_COL_FMTS; i++)
+ gtk_combo_box_append_text(GTK_COMBO_BOX(fmt_cmb), col_format_desc(i));
- if (cur_fmt == COL_CUSTOM) {
- gtk_entry_set_text(GTK_ENTRY(field_te), cfmt->custom_field);
- gtk_widget_show(field_lb);
- gtk_widget_show(field_te);
- } else {
- gtk_widget_hide(field_lb);
- gtk_widget_hide(field_te);
- }
- g_signal_connect(field_te, "changed", G_CALLBACK(column_field_changed_cb), column_l);
+ column_menu_changed_handler_id = g_signal_connect(fmt_cmb, "changed", G_CALLBACK(column_menu_changed_cb), column_l);
- gtk_combo_box_set_active(GTK_COMBO_BOX(fmt_cmb), cur_fmt);
+ gtk_widget_set_sensitive(fmt_cmb, FALSE);
+ gtk_box_pack_start(GTK_BOX(props_hb), fmt_cmb, FALSE, FALSE, 0);
+ gtk_widget_show(fmt_cmb);
- gtk_widget_set_sensitive(remove_bt, TRUE);
- gtk_widget_set_sensitive(field_te, TRUE);
- gtk_widget_set_sensitive(fmt_cmb, TRUE);
- }
- else
- {
- cur_row = -1;
- gtk_editable_delete_text(GTK_EDITABLE(field_te), 0, -1);
+ /* select the first menu list row. */
+ /* Triggers call to column_list_select_cb(). */
+ gtk_tree_selection_select_iter(sel, &first_iter);
- gtk_widget_set_sensitive(remove_bt, FALSE);
- gtk_widget_set_sensitive(field_te, FALSE);
- gtk_widget_set_sensitive(fmt_cmb, FALSE);
- }
+ return(main_vb);
}
+
/* To do: add input checking to each of these callbacks */
static void
column_list_new_cb(GtkWidget *w _U_, gpointer data) {
fmt_data *cfmt;
+ gint cur_fmt;
const gchar *title = "New Column";
GtkTreeView *column_l = GTK_TREE_VIEW(data);
GtkTreeModel *model;
GtkTreeIter iter;
GtkTreePath *path;
GtkTreeViewColumn *title_column;
- gchar *str_path;
cur_fmt = COL_NUMBER;
cfmt = (fmt_data *) g_malloc(sizeof(fmt_data));
model = gtk_tree_view_get_model(column_l);
gtk_list_store_append(GTK_LIST_STORE(model), &iter);
- gtk_list_store_set(GTK_LIST_STORE(model), &iter, 0, title, 1,
- col_format_desc(cur_fmt), 2, g_list_last(prefs.col_list),
+ gtk_list_store_set(GTK_LIST_STORE(model), &iter,
+ 0, title,
+ 1, col_format_desc(cur_fmt),
+ 2, g_list_last(prefs.col_list),
-1);
- path = gtk_tree_model_get_path(model, &iter);
- str_path = gtk_tree_path_to_string(path);
- cur_row = atoi(str_path);
+ /* Triggers call to column_list_select_cb() */
+ gtk_tree_selection_select_iter(gtk_tree_view_get_selection(column_l), &iter);
- gtk_tree_selection_select_iter(gtk_tree_view_get_selection(column_l),
- &iter);
+ /* Set the cursor to the 'Title' column of the newly added row and enable editing */
+ /* XXX: If displaying the new title ["New column"] widens the title column of the */
+ /* treeview, then the set_cursor below doesn't properly generate an entry */
+ /* box around the title text. The width of the box appears to be the column */
+ /* width before the treeview title column was widened. Seems like a bug... */
+ /* I haven't found a work-around. */
+ path = gtk_tree_model_get_path(model, &iter);
title_column = gtk_tree_view_get_column(column_l, 0);
gtk_tree_view_set_cursor(column_l, path, title_column, TRUE);
-
- g_free(str_path);
gtk_tree_path_free(path);
+
cfile.cinfo.columns_changed = TRUE;
}
+
static void
column_list_delete_cb(GtkWidget *w _U_, gpointer data) {
GtkTreeView *column_l = GTK_TREE_VIEW(data);
GtkTreeSelection *sel;
GtkTreeModel *model;
GtkTreeIter iter;
+ GtkTreeIter new_iter;
sel = gtk_tree_view_get_selection(column_l);
if (gtk_tree_selection_get_selected(sel, &model, &iter))
cfmt = (fmt_data *) clp->data;
g_free(cfmt->title);
g_free(cfmt->fmt);
- if (cfmt->custom_field) {
- g_free (cfmt->custom_field);
- }
+ g_free(cfmt->custom_field);
g_free(cfmt);
prefs.col_list = g_list_remove_link(prefs.col_list, clp);
+ /* Change the row selection to the next row (if available) or */
+ /* the previous row (if available). If there's only one row */
+ /* in the store (no previous and no next), then the selection */
+ /* will not be changed. */
+
+ /* Note that gtk_tree_selection_select_iter() will trigger a */
+ /* call to column_list_select_cb(). */
+
+ new_iter = iter;
+ if ( gtk_tree_model_iter_next(model, &new_iter)) {
+ gtk_tree_selection_select_iter(sel, &new_iter);
+ } else { /* "gtk_tree_model_iter_prev" */
+ GtkTreePath *path = gtk_tree_model_get_path(model, &iter);
+ if (gtk_tree_path_prev(path)) {
+ gtk_tree_model_get_iter(model, &new_iter, path);
+ gtk_tree_selection_select_iter(sel, &new_iter);
+ }
+ gtk_tree_path_free(path);
+ }
+
+ /* Remove the row from the list store. */
+ /* We prevent triggering call to column_row_deleted_cb() since */
+ /* the entry has already been removed from prefs.col_list and */
+ /* since rebuilding the list is not needed because the order */
+ /* of the list hasn't changed. */
+
+ g_signal_handler_block(model, column_row_deleted_handler_id);
+
+ /* list_store_remove below will trigger a call to */
+ /* column_list_select_cb() only when deleting the last entry in */
+ /* the column list. */
+ /* (This is because the selection in this case changes to */
+ /* "nothing selected" when the last row is removed. */
+
gtk_list_store_remove(GTK_LIST_STORE(model), &iter);
+
+ g_signal_handler_unblock (model, column_row_deleted_handler_id);
+
+ cfile.cinfo.columns_changed = TRUE;
}
- cfile.cinfo.columns_changed = TRUE;
}
+
static gboolean
column_title_changed_cb(GtkCellRendererText *cell _U_, const gchar *str_path, const gchar *new_title, gpointer data) {
- fmt_data *cfmt;
- GList *clp;
- GtkTreeModel *model = (GtkTreeModel *)data;
- GtkTreePath *path = gtk_tree_path_new_from_string (str_path);
- GtkTreeIter iter;
+ fmt_data *cfmt;
+ GList *clp;
+ GtkTreeModel *model = (GtkTreeModel *)data;
+ GtkTreePath *path = gtk_tree_path_new_from_string (str_path);
+ GtkTreeIter iter;
- gtk_tree_model_get_iter(model, &iter, path);
+ gtk_tree_model_get_iter(model, &iter, path);
- gtk_list_store_set(GTK_LIST_STORE(model), &iter, 0, new_title, -1);
+ gtk_list_store_set(GTK_LIST_STORE(model), &iter, 0, new_title, -1);
- gtk_tree_model_get(model, &iter, 2, &clp, -1);
- if (clp) {
- cfmt = (fmt_data *) clp->data;
- g_free(cfmt->title);
- cfmt->title = g_strdup(new_title);
- }
+ gtk_tree_model_get(model, &iter, 2, &clp, -1);
+ if (clp) {
+ cfmt = (fmt_data *) clp->data;
+ g_free(cfmt->title);
+ cfmt->title = g_strdup(new_title);
+ }
- gtk_tree_path_free (path);
- cfile.cinfo.columns_changed = TRUE;
- return TRUE;
+ gtk_tree_path_free (path);
+ cfile.cinfo.columns_changed = TRUE;
+ return TRUE;
}
-/* The user changed the custom field entry box. */
+/*
+ * column list row selection changed.
+ * Set the "Properties" widgets to match the currently selected column row item.
+ */
+
static void
-column_field_changed_cb(GtkEditable *te, gpointer data) {
- fmt_data *cfmt;
- GList *clp;
- gchar *field, *fmt;
- GtkTreeView *tree = (GtkTreeView *)data;
- GtkTreeSelection *sel;
- GtkTreeModel *model;
- GtkTreeIter iter;
+column_list_select_cb(GtkTreeSelection *sel, gpointer data _U_)
+{
+ fmt_data *cfmt;
+ gint cur_fmt;
+ GList *clp;
+ GtkTreeModel *model;
+ GtkTreeIter iter;
- sel = gtk_tree_view_get_selection(tree);
+ /* if something was selected */
if (gtk_tree_selection_get_selected(sel, &model, &iter))
{
- field = gtk_editable_get_chars(te, 0, -1);
gtk_tree_model_get(model, &iter, 2, &clp, -1);
- cfmt = (fmt_data *) clp->data;
- fmt = g_strdup_printf("%s (%s)", col_format_desc(cur_fmt), field);
- gtk_list_store_set(GTK_LIST_STORE(model), &iter, 1, fmt, -1);
- g_free(fmt);
- if (cfmt->custom_field) {
- g_free(cfmt->custom_field);
+ g_assert(clp != NULL);
+ cfmt = (fmt_data *) clp->data;
+ cur_fmt = get_column_format_from_str(cfmt->fmt);
+ g_assert(cur_fmt != -1); /* It should always be valid */
+
+ g_signal_handler_block (fmt_cmb, column_menu_changed_handler_id);
+ gtk_combo_box_set_active(GTK_COMBO_BOX(fmt_cmb), cur_fmt);
+ g_signal_handler_unblock(fmt_cmb, column_menu_changed_handler_id);
+
+ g_signal_handler_block (field_te, column_field_changed_handler_id);
+ if (cur_fmt == COL_CUSTOM) {
+ gtk_entry_set_text(GTK_ENTRY(field_te), cfmt->custom_field);
+ gtk_widget_show(field_lb);
+ gtk_widget_show(field_te);
+ } else {
+ gtk_editable_delete_text(GTK_EDITABLE(field_te), 0, -1);
+ gtk_widget_hide(field_lb);
+ gtk_widget_hide(field_te);
}
- cfmt->custom_field = field;
+ g_signal_handler_unblock(field_te, column_field_changed_handler_id);
+
+ gtk_widget_set_sensitive(remove_bt, TRUE);
+ gtk_widget_set_sensitive(field_te, TRUE);
+ gtk_widget_set_sensitive(fmt_cmb, TRUE);
+ }
+ else
+ {
+ gtk_editable_delete_text(GTK_EDITABLE(field_te), 0, -1);
+
+ gtk_widget_set_sensitive(remove_bt, FALSE);
+ gtk_widget_set_sensitive(field_te, FALSE);
+ gtk_widget_set_sensitive(fmt_cmb, FALSE);
}
- cfile.cinfo.columns_changed = TRUE;
}
-/* The user changed the format menu. */
+
+/*
+ * The user selected a new entry in the format combo-box;
+ * Note: column_menu_changed_cb is expected to be called only
+ * when the user changes the format combo-box.
+ * Action:
+ * If no selection active in the column list:
+ * Hide 'field"; desensitize buttons.
+ * XXX: Can this happen ?
+ * If a column list selection is active:
+ * 1. Update the display of "field" as req'd (depending upon
+ * whether the active combo-box entry is the "custom"
+ * format type).
+ * 2. Update the column_list and prefs.col_formats.
+ * Set columns_changed = TRUE.
+ */
+
static void
column_menu_changed_cb(GtkWidget *w, gpointer data) {
GtkTreeView *column_l = GTK_TREE_VIEW(data);
fmt_data *cfmt;
+ gint cur_fmt;
+ gint cur_cb_fmt;
GList *clp;
- const gchar *fmt;
+ gchar *fmt;
GtkTreeSelection *sel;
GtkTreeModel *model;
GtkTreeIter iter;
sel = gtk_tree_view_get_selection(column_l);
- if (gtk_tree_selection_get_selected(sel, &model, &iter))
- {
- cur_fmt = gtk_combo_box_get_active(GTK_COMBO_BOX(w));
- gtk_tree_model_get(model, &iter, 2, &clp, -1);
- cfmt = (fmt_data *) clp->data;
+ if (! (gtk_tree_selection_get_selected(sel, &model, &iter)))
+ return; /* no column list selection [Can this happen ?]: ignore callback */
- if (cur_fmt == COL_CUSTOM) {
- if (cfmt->custom_field == NULL) {
- cfmt->custom_field = g_strdup("");
- }
- gtk_entry_set_text(GTK_ENTRY(field_te), cfmt->custom_field);
- fmt = g_strdup_printf("%s (%s)", col_format_desc(cur_fmt), cfmt->custom_field);
- gtk_widget_show(field_lb);
- gtk_widget_show(field_te);
- } else {
- fmt = g_strdup_printf("%s", col_format_desc(cur_fmt));
- gtk_widget_hide(field_lb);
- gtk_widget_hide(field_te);
- }
+ cur_cb_fmt = gtk_combo_box_get_active(GTK_COMBO_BOX(w));
+ gtk_tree_model_get(model, &iter, 2, &clp, -1);
+ cfmt = (fmt_data *) clp->data;
+ cur_fmt = get_column_format_from_str(cfmt->fmt);
- gtk_list_store_set(GTK_LIST_STORE(model), &iter, 1, fmt, -1);
- g_free(cfmt->fmt);
- cfmt->fmt = g_strdup(col_format_to_string(cur_fmt));
+ g_assert(cur_cb_fmt != cur_fmt);
+
+ /* The User has selected a new format in the combo-box */
+ /* (IE: combo-box format != current selected row format) */
+ /* Update field widgets, list_store, column format array */
+ /* entry as appropriate. */
+ if (cur_fmt == COL_CUSTOM) {
+ /* Changing from custom to non-custom */
+ g_free(cfmt->custom_field);
+ cfmt->custom_field = NULL;
+ fmt = g_strdup_printf("%s", col_format_desc(cur_cb_fmt));
+ gtk_widget_hide(field_lb);
+ gtk_widget_hide(field_te);
+
+ } else if (cur_cb_fmt == COL_CUSTOM) {
+ /* Changing from non-custom to custom */
+ g_assert (cfmt->custom_field == NULL);
+ cfmt->custom_field = g_strdup("");
+ /* The following doesn't trigger a call to menu_field_changed_cb() */
+ gtk_editable_delete_text(GTK_EDITABLE(field_te), 0, -1);
+ fmt = g_strdup_printf("%s (%s)", col_format_desc(cur_cb_fmt), cfmt->custom_field);
+ gtk_widget_show(field_lb);
+ gtk_widget_show(field_te);
+
+ } else {
+ /* Changing from non-custom to non-custom */
+ fmt = g_strdup_printf("%s", col_format_desc(cur_cb_fmt));
}
+
+ gtk_list_store_set(GTK_LIST_STORE(model), &iter, 1, fmt, -1);
+ g_free(fmt);
+ g_free(cfmt->fmt);
+ cfmt->fmt = g_strdup(col_format_to_string(cur_cb_fmt));
cfile.cinfo.columns_changed = TRUE;
}
+
+/*
+ * The user changed the custom field entry box or
+ * the field entry box has been updated because a new
+ * column row with custom format has been selected.
+ * If the current field entry matches that of the current
+ * column row, this is just an update because a new
+ * column row has been selected. Do nothing.
+ * If the two are different, then update the column row & etc.
+ */
+static void
+column_field_changed_cb(GtkEditable *te, gpointer data) {
+ fmt_data *cfmt;
+ gint cur_fmt;
+ GList *clp;
+ gchar *field, *fmt;
+ GtkTreeView *tree = (GtkTreeView *)data;
+ GtkTreeSelection *sel;
+ GtkTreeModel *model;
+ GtkTreeIter iter;
+
+ sel = gtk_tree_view_get_selection(tree);
+ if ( ! (gtk_tree_selection_get_selected(sel, &model, &iter))) {
+ return;
+ }
+
+ field = gtk_editable_get_chars(te, 0, -1);
+ gtk_tree_model_get(model, &iter, 2, &clp, -1);
+ cfmt = (fmt_data *) clp->data;
+ if (strcmp(cfmt->custom_field, field) == 0) {
+ return; /* no action req'd */
+ }
+
+ /* The user has entered a new value in the field entry box: make the req'd changes */
+ cur_fmt = get_column_format_from_str(cfmt->fmt);
+ fmt = g_strdup_printf("%s (%s)", col_format_desc(cur_fmt), field);
+
+ gtk_list_store_set(GTK_LIST_STORE(model), &iter, 1, fmt, -1);
+ g_free(fmt);
+ g_free(cfmt->custom_field);
+ cfmt->custom_field = field;
+ cfile.cinfo.columns_changed = TRUE;
+}
+
+
/*
* Callback for the "row-deleted" signal emitted when a list item is dragged.
* http://library.gnome.org/devel/gtk/stable/GtkTreeModel.html#GtkTreeModel-rows-reordered
- * says that DND deletes, THEN inserts the row. If this isn't the case, we'll
- * have to find another way to do this (e.g. by rebuilding prefs.col_list by
- * iterating over the tree model.
+ * says that DND deletes, THEN inserts the row.
+ *
+ * XXX: For the record: For Gtk+ 2.16.0 testing shows the actual sequence for drag-and-drop to be as follows:
+ * 1. Insert a new, empty row at the destination;
+ * 2. Emit a "row-inserted" signal on the model; invoke any row-inserted callbacks & etc;
+ * 3. Copy the source row data to the new (empty) destination row;
+ * 4. Delete the source row;
+ * 5. Emit a "row-deleted" signal; invoke any row-deleted callbacks & etc.
+ *
+ * The code below (invoked as a consequence of a "row-deleted" signal) rebuilds
+ * prefs.col_list by iterating over the (re-ordered) tree model.
*/
static void
column_dnd_row_deleted_cb(GtkTreeModel *model, GtkTreePath *path _U_, gpointer data _U_) {
- GtkTreeIter iter;
- /* gpointer cfmt; */
- GList *clp, *new_col_list = NULL;
- gchar *title, *format;
- gboolean items_left;
-
- /*
- * XXX - This rebuilds prefs.col_list based on the current model. We
- * might just want to do this when the prefs are applied
- */
- for (items_left = gtk_tree_model_get_iter_first (model, &iter);
- items_left;
- items_left = gtk_tree_model_iter_next (model, &iter)) {
-
- gtk_tree_model_get(model, &iter, 0, &title, 1, &format, 2, &clp, -1);
- if (clp) {
- prefs.col_list = g_list_remove_link(prefs.col_list, clp);
- new_col_list = g_list_concat(new_col_list, clp);
+ GtkTreeIter iter;
+ GList *clp, *new_col_list = NULL;
+ gboolean items_left;
+
+ /*
+ * XXX - This rebuilds prefs.col_list based on the current model. We
+ * might just want to do this when the prefs are applied
+ */
+ for (items_left = gtk_tree_model_get_iter_first (model, &iter);
+ items_left;
+ items_left = gtk_tree_model_iter_next (model, &iter)) {
+
+ gtk_tree_model_get(model, &iter, 2, &clp, -1);
+ if (clp) {
+ prefs.col_list = g_list_remove_link(prefs.col_list, clp);
+ new_col_list = g_list_concat(new_col_list, clp);
+ }
+ }
+ if (prefs.col_list) {
+ g_warning("column_dnd_row_deleted_cb: prefs.col_list has %d leftover data",
+ g_list_length(prefs.col_list));
+ g_list_free(prefs.col_list);
}
- }
- if (prefs.col_list) {
- g_warning("column_dnd_row_deleted_cb: prefs.col_list has %d leftover data",
- g_list_length(prefs.col_list));
- g_list_free(prefs.col_list);
- }
- prefs.col_list = new_col_list;
- cfile.cinfo.columns_changed = TRUE;
+
+ prefs.col_list = new_col_list;
+ cfile.cinfo.columns_changed = TRUE;
}
+
void
column_prefs_fetch(GtkWidget *w _U_) {
}
+
void
column_prefs_apply(GtkWidget *w _U_)
{
/* Redraw the packet list if the columns were changed */
if(cfile.cinfo.columns_changed) {
+#ifndef NEW_PACKET_LIST
packet_list_recreate();
+#endif
cfile.cinfo.columns_changed = FALSE; /* Reset value */
}
}
+
void
-column_prefs_destroy(GtkWidget *w) {
- /* Let the list cb know we're about to destroy the widget tree, so it */
- /* doesn't operate on widgets that don't exist. */
- g_object_set_data(G_OBJECT(w), E_COL_CM_KEY, (gpointer)TRUE);
+column_prefs_destroy(GtkWidget *w _U_) {
}