X-Git-Url: https://git.street.me.uk/andy/viking.git/blobdiff_plain/0ae518e4ccbd2f3ac809748282b1e1c4c255da79..2ba4dc06631b9c0c953ceb3859e1c3ef54384899:/src/uibuilder.c diff --git a/src/uibuilder.c b/src/uibuilder.c index 5908f17a..f2731eb4 100644 --- a/src/uibuilder.c +++ b/src/uibuilder.c @@ -32,57 +32,96 @@ VikLayerParamData vik_lpd_true_default ( void ) { return VIK_LPD_BOOLEAN ( TRUE ); } VikLayerParamData vik_lpd_false_default ( void ) { return VIK_LPD_BOOLEAN ( FALSE ); } +/** i18n note + * Since UI builder often uses static structures, the text is marked with N_() + * however to actually get it to apply the widget (e.g. to a label) then + * an additional call to _() needs to occur on that string + **/ + GtkWidget *a_uibuilder_new_widget ( VikLayerParam *param, VikLayerParamData data ) { + // Perform pre conversion if necessary + VikLayerParamData vlpd = data; + if ( param->convert_to_display ) + vlpd = param->convert_to_display ( data ); + GtkWidget *rv = NULL; switch ( param->widget_type ) { case VIK_LAYER_WIDGET_COLOR: if ( param->type == VIK_LAYER_PARAM_COLOR ) - rv = gtk_color_button_new_with_color ( &(data.c) ); + rv = gtk_color_button_new_with_color ( &(vlpd.c) ); break; case VIK_LAYER_WIDGET_CHECKBUTTON: if ( param->type == VIK_LAYER_PARAM_BOOLEAN ) { //rv = gtk_check_button_new_with_label ( //param->title ); rv = gtk_check_button_new (); - if ( data.b ) + if ( vlpd.b ) gtk_toggle_button_set_active ( GTK_TOGGLE_BUTTON(rv), TRUE ); } break; case VIK_LAYER_WIDGET_COMBOBOX: if ( param->type == VIK_LAYER_PARAM_UINT && param->widget_data ) { + /* Build a simple combobox */ gchar **pstr = param->widget_data; rv = vik_combo_box_text_new (); while ( *pstr ) - vik_combo_box_text_append ( rv, *(pstr++) ); + vik_combo_box_text_append ( rv, _(*(pstr++)) ); if ( param->extra_widget_data ) /* map of alternate uint values for options */ { + /* Set the effective default value */ int i; for ( i = 0; ((const char **)param->widget_data)[i]; i++ ) - if ( ((guint *)param->extra_widget_data)[i] == data.u ) + if ( ((guint *)param->extra_widget_data)[i] == vlpd.u ) { + /* Match default value */ gtk_combo_box_set_active ( GTK_COMBO_BOX(rv), i ); break; } } else - gtk_combo_box_set_active ( GTK_COMBO_BOX ( rv ), data.u ); + gtk_combo_box_set_active ( GTK_COMBO_BOX ( rv ), vlpd.u ); } - else if ( param->type == VIK_LAYER_PARAM_STRING && param->widget_data ) + else if ( param->type == VIK_LAYER_PARAM_STRING && param->widget_data && !param->extra_widget_data ) { + /* Build a combobox with editable text */ gchar **pstr = param->widget_data; #if GTK_CHECK_VERSION (2, 24, 0) rv = gtk_combo_box_text_new_with_entry (); #else rv = gtk_combo_box_entry_new_text (); #endif - if ( data.s ) - vik_combo_box_text_append ( rv, data.s ); + if ( vlpd.s ) + vik_combo_box_text_append ( rv, _(vlpd.s) ); while ( *pstr ) - vik_combo_box_text_append ( rv, *(pstr++) ); - if ( data.s ) + vik_combo_box_text_append ( rv, _(*(pstr++)) ); + if ( vlpd.s ) + gtk_combo_box_set_active ( GTK_COMBO_BOX ( rv ), 0 ); + } + else if ( param->type == VIK_LAYER_PARAM_STRING && param->widget_data && param->extra_widget_data) + { + /* Build a combobox with fixed selections without editable text */ + gchar **pstr = param->widget_data; + rv = GTK_WIDGET ( vik_combo_box_text_new () ); + while ( *pstr ) + vik_combo_box_text_append ( rv, _(*(pstr++)) ); + if ( vlpd.s ) + { + /* Set the effective default value */ + /* In case of value does not exist, set the first value */ + gtk_combo_box_set_active ( GTK_COMBO_BOX ( rv ), 0 ); + int i; + for ( i = 0; ((const char **)param->widget_data)[i]; i++ ) + if ( strcmp(((const char **)param->extra_widget_data)[i], vlpd.s) == 0 ) + { + /* Match default value */ + gtk_combo_box_set_active ( GTK_COMBO_BOX ( rv ), i ); + break; + } + } + else gtk_combo_box_set_active ( GTK_COMBO_BOX ( rv ), 0 ); } break; @@ -96,14 +135,14 @@ GtkWidget *a_uibuilder_new_widget ( VikLayerParam *param, VikLayerParamData data int i; int nb_elem = g_list_length(param->widget_data); for ( i = 0; i < nb_elem; i++ ) - if ( GPOINTER_TO_UINT ( g_list_nth_data(param->extra_widget_data, i) ) == data.u ) + if ( GPOINTER_TO_UINT ( g_list_nth_data(param->extra_widget_data, i) ) == vlpd.u ) { vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), i ); break; } } - else if ( data.u ) /* zero is already default */ - vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), data.u ); + else if ( vlpd.u ) /* zero is already default */ + vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), vlpd.u ); } break; case VIK_LAYER_WIDGET_RADIOGROUP_STATIC: @@ -114,21 +153,21 @@ GtkWidget *a_uibuilder_new_widget ( VikLayerParam *param, VikLayerParamData data { int i; for ( i = 0; ((const char **)param->widget_data)[i]; i++ ) - if ( ((guint *)param->extra_widget_data)[i] == data.u ) + if ( ((guint *)param->extra_widget_data)[i] == vlpd.u ) { vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), i ); break; } } - else if ( data.u ) /* zero is already default */ - vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), data.u ); + else if ( vlpd.u ) /* zero is already default */ + vik_radio_group_set_selected ( VIK_RADIO_GROUP(rv), vlpd.u ); } break; case VIK_LAYER_WIDGET_SPINBUTTON: if ( (param->type == VIK_LAYER_PARAM_DOUBLE || param->type == VIK_LAYER_PARAM_UINT || param->type == VIK_LAYER_PARAM_INT) && param->widget_data ) { - gdouble init_val = (param->type == VIK_LAYER_PARAM_DOUBLE) ? data.d : (param->type == VIK_LAYER_PARAM_UINT ? data.u : data.i); + gdouble init_val = (param->type == VIK_LAYER_PARAM_DOUBLE) ? vlpd.d : (param->type == VIK_LAYER_PARAM_UINT ? vlpd.u : vlpd.i); VikLayerParamScale *scale = (VikLayerParamScale *) param->widget_data; rv = gtk_spin_button_new ( GTK_ADJUSTMENT(gtk_adjustment_new( init_val, scale->min, scale->max, scale->step, scale->step, 0 )), scale->step, scale->digits ); } @@ -137,8 +176,8 @@ GtkWidget *a_uibuilder_new_widget ( VikLayerParam *param, VikLayerParamData data if ( param->type == VIK_LAYER_PARAM_STRING ) { rv = gtk_entry_new (); - if (data.s) - gtk_entry_set_text ( GTK_ENTRY(rv), data.s ); + if ( vlpd.s ) + gtk_entry_set_text ( GTK_ENTRY(rv), vlpd.s ); } break; case VIK_LAYER_WIDGET_PASSWORD: @@ -146,8 +185,8 @@ GtkWidget *a_uibuilder_new_widget ( VikLayerParam *param, VikLayerParamData data { rv = gtk_entry_new (); gtk_entry_set_visibility ( GTK_ENTRY(rv), FALSE ); - if (data.s) - gtk_entry_set_text ( GTK_ENTRY(rv), data.s ); + if ( vlpd.s ) + gtk_entry_set_text ( GTK_ENTRY(rv), vlpd.s ); gtk_widget_set_tooltip_text ( GTK_WIDGET(rv), _("Take care that this password will be stored clearly in a plain file.") ); } @@ -155,35 +194,47 @@ GtkWidget *a_uibuilder_new_widget ( VikLayerParam *param, VikLayerParamData data case VIK_LAYER_WIDGET_FILEENTRY: if ( param->type == VIK_LAYER_PARAM_STRING ) { - rv = vik_file_entry_new (GTK_FILE_CHOOSER_ACTION_OPEN); - vik_file_entry_set_filename ( VIK_FILE_ENTRY(rv), data.s ); + rv = vik_file_entry_new (GTK_FILE_CHOOSER_ACTION_OPEN, GPOINTER_TO_INT(param->widget_data), NULL, NULL); + if ( vlpd.s ) + vik_file_entry_set_filename ( VIK_FILE_ENTRY(rv), vlpd.s ); } break; case VIK_LAYER_WIDGET_FOLDERENTRY: if ( param->type == VIK_LAYER_PARAM_STRING ) { - rv = vik_file_entry_new (GTK_FILE_CHOOSER_ACTION_SELECT_FOLDER); - vik_file_entry_set_filename ( VIK_FILE_ENTRY(rv), data.s ); + rv = vik_file_entry_new (GTK_FILE_CHOOSER_ACTION_SELECT_FOLDER, VF_FILTER_NONE, NULL, NULL); + if ( vlpd.s ) + vik_file_entry_set_filename ( VIK_FILE_ENTRY(rv), vlpd.s ); } break; case VIK_LAYER_WIDGET_FILELIST: if ( param->type == VIK_LAYER_PARAM_STRING_LIST ) { - rv = vik_file_list_new ( _(param->title) ); - vik_file_list_set_files ( VIK_FILE_LIST(rv), data.sl ); + rv = vik_file_list_new ( _(param->title), NULL ); + vik_file_list_set_files ( VIK_FILE_LIST(rv), vlpd.sl ); } break; case VIK_LAYER_WIDGET_HSCALE: if ( (param->type == VIK_LAYER_PARAM_DOUBLE || param->type == VIK_LAYER_PARAM_UINT || param->type == VIK_LAYER_PARAM_INT) && param->widget_data ) { - gdouble init_val = (param->type == VIK_LAYER_PARAM_DOUBLE) ? data.d : (param->type == VIK_LAYER_PARAM_UINT ? data.u : data.i); + gdouble init_val = (param->type == VIK_LAYER_PARAM_DOUBLE) ? vlpd.d : (param->type == VIK_LAYER_PARAM_UINT ? vlpd.u : vlpd.i); VikLayerParamScale *scale = (VikLayerParamScale *) param->widget_data; rv = gtk_hscale_new_with_range ( scale->min, scale->max, scale->step ); gtk_scale_set_digits ( GTK_SCALE(rv), scale->digits ); gtk_range_set_value ( GTK_RANGE(rv), init_val ); } + break; + + case VIK_LAYER_WIDGET_BUTTON: + if ( param->type == VIK_LAYER_PARAM_PTR && param->widget_data ) { + rv = gtk_button_new_with_label ( _(param->widget_data) ); + g_signal_connect ( G_OBJECT(rv), "clicked", G_CALLBACK (vlpd.ptr), param->extra_widget_data ); + } + break; + + default: break; } if ( rv && !gtk_widget_get_tooltip_text ( rv ) ) { if ( param->tooltip ) @@ -214,11 +265,21 @@ VikLayerParamData a_uibuilder_widget_get_value ( GtkWidget *widget, VikLayerPara } if ( param->type == VIK_LAYER_PARAM_STRING) { + if ( param->extra_widget_data ) + { + /* Combobox displays labels and we want values from extra */ + int pos = gtk_combo_box_get_active ( GTK_COMBO_BOX(widget) ); + rv.s = ((const char **)param->extra_widget_data)[pos]; + } + else + { + /* Return raw value */ #if GTK_CHECK_VERSION (2, 24, 0) - rv.s = gtk_entry_get_text (GTK_ENTRY (gtk_bin_get_child (GTK_BIN (widget)))); + rv.s = gtk_entry_get_text (GTK_ENTRY (gtk_bin_get_child (GTK_BIN (widget)))); #else - rv.s = gtk_combo_box_get_active_text ( GTK_COMBO_BOX(widget) ); + rv.s = gtk_combo_box_get_active_text ( GTK_COMBO_BOX(widget) ); #endif + } g_debug("%s: %s", __FUNCTION__, rv.s); } break; @@ -255,17 +316,30 @@ VikLayerParamData a_uibuilder_widget_get_value ( GtkWidget *widget, VikLayerPara else rv.d = gtk_range_get_value ( GTK_RANGE(widget) ); break; + default: break; } + + // Perform conversion if necessary + if ( param->convert_to_internal ) + rv = param->convert_to_internal ( rv ); + return rv; } -gint a_uibuilder_properties_factory ( const gchar *dialog_name, GtkWindow *parent, VikLayerParam *params, - guint16 params_count, gchar **groups, guint8 groups_count, - gboolean (*setparam) (gpointer,guint16,VikLayerParamData,gpointer,gboolean), - gpointer pass_along1, gpointer pass_along2, - VikLayerParamData (*getparam) (gpointer,guint16,gboolean), - gpointer pass_along_getparam ) - /* pass_along1 and pass_along2 are for set_param first and last params */ +//static void draw_to_image_file_total_area_cb (GtkSpinButton *spinbutton, gpointer *pass_along) +gint a_uibuilder_properties_factory ( const gchar *dialog_name, + GtkWindow *parent, + VikLayerParam *params, + guint16 params_count, + gchar **groups, + guint8 groups_count, + gboolean (*setparam) (gpointer,gpointer), + gboolean (*setparam4) (gpointer,guint16,VikLayerParamData,gpointer), + gpointer pass_along1, + gpointer pass_along2, + VikLayerParamData (*getparam) (gpointer,guint16,gboolean), + gpointer pass_along_getparam, + void (*changeparam) (GtkWidget*, ui_change_values) ) { guint16 i, j, widget_count = 0; gboolean must_redraw = FALSE; @@ -298,7 +372,9 @@ gint a_uibuilder_properties_factory ( const gchar *dialog_name, GtkWindow *paren GtkWidget **tables = NULL; /* for more than one group */ GtkWidget *notebook = NULL; + GtkWidget **labels = g_malloc ( sizeof(GtkWidget *) * widget_count ); GtkWidget **widgets = g_malloc ( sizeof(GtkWidget *) * widget_count ); + ui_change_values *change_values = g_malloc ( sizeof(ui_change_values) * widget_count ); if ( groups && groups_count > 1 ) { @@ -340,13 +416,48 @@ gint a_uibuilder_properties_factory ( const gchar *dialog_name, GtkWindow *paren widgets[j] = a_uibuilder_new_widget ( &(params[i]), getparam ( pass_along_getparam, i, FALSE ) ); if ( widgets[j] ) { - gtk_table_attach ( GTK_TABLE(table), gtk_label_new(_(params[i].title)), 0, 1, j, j+1, 0, 0, 0, 0 ); + labels[j] = gtk_label_new(_(params[i].title)); + gtk_table_attach ( GTK_TABLE(table), labels[j], 0, 1, j, j+1, 0, 0, 0, 0 ); gtk_table_attach ( GTK_TABLE(table), widgets[j], 1, 2, j, j+1, GTK_EXPAND | GTK_FILL, 0, 2, 2 ); + + if ( changeparam ) + { + change_values[j][UI_CHG_LAYER] = pass_along1; + change_values[j][UI_CHG_PARAM] = ¶ms[i]; + change_values[j][UI_CHG_PARAM_ID] = GINT_TO_POINTER((gint)i); + change_values[j][UI_CHG_WIDGETS] = widgets; + change_values[j][UI_CHG_LABELS] = labels; + + switch ( params[i].widget_type ) + { + // Change conditions for other widget types can be added when needed + case VIK_LAYER_WIDGET_COMBOBOX: + g_signal_connect ( G_OBJECT(widgets[j]), "changed", G_CALLBACK(changeparam), change_values[j] ); + break; + case VIK_LAYER_WIDGET_CHECKBUTTON: + g_signal_connect ( G_OBJECT(widgets[j]), "toggled", G_CALLBACK(changeparam), change_values[j] ); + break; + default: break; + } + } } j++; } } + // Repeat run through to force changeparam callbacks now that the widgets have been created + // This primarily so the widget sensitivities get set up + if ( changeparam ) { + for ( i = 0, j = 0; i < params_count; i++ ) { + if ( params[i].group != VIK_LAYER_NOT_IN_PROPERTIES ) { + if ( widgets[j] ) { + changeparam ( widgets[j], change_values[j] ); + } + j++; + } + } + } + if ( response_w ) gtk_widget_grab_focus ( response_w ); @@ -355,32 +466,43 @@ gint a_uibuilder_properties_factory ( const gchar *dialog_name, GtkWindow *paren resp = gtk_dialog_run (GTK_DIALOG (dialog)); if ( resp == GTK_RESPONSE_ACCEPT ) { + VikLayerSetParam vlsp; + vlsp.is_file_operation = FALSE; + vlsp.dirpath = NULL; for ( i = 0, j = 0; i < params_count; i++ ) { if ( params[i].group != VIK_LAYER_NOT_IN_PROPERTIES ) { - if ( setparam ( pass_along1, - i, - a_uibuilder_widget_get_value ( widgets[j], &(params[i]) ), - pass_along2, - FALSE ) ) + vlsp.id = i; + vlsp.vp = pass_along2; + vlsp.data = a_uibuilder_widget_get_value ( widgets[j], &(params[i]) ); + // Main callback into each layer's setparam + if ( setparam && setparam ( pass_along1, &vlsp ) ) + must_redraw = TRUE; + // Or a basic callback for each parameter + else if ( setparam4 && setparam4 ( pass_along1, i, vlsp.data, pass_along2 ) ) must_redraw = TRUE; j++; } } - gtk_widget_destroy ( dialog ); /* hide before redrawing. */ g_free ( widgets ); + g_free ( labels ); + g_free ( change_values ); if ( tables ) g_free ( tables ); + gtk_widget_destroy ( dialog ); /* hide before redrawing. */ return must_redraw ? 2 : 3; /* user clicked OK */ } + g_free ( widgets ); + g_free ( labels ); + g_free ( change_values ); if ( tables ) g_free ( tables ); gtk_widget_destroy ( dialog ); - g_free ( widgets ); + return 0; } } @@ -415,11 +537,13 @@ VikLayerParamData *a_uibuilder_run_dialog ( const gchar *dialog_name, GtkWindow params_count, groups, groups_count, + NULL, (gpointer) uibuilder_run_setparam, paramdatas, params, (gpointer) uibuilder_run_getparam, - params_defaults ) > 0 ) { + params_defaults, + NULL ) > 0 ) { return paramdatas; }