X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~mdw/git/disorder/blobdiff_plain/d4ef413201b6aa149d11992dc2527e07cd9085d4..7392b131424322234159283e35fcccefa31f5ff2:/disobedience/properties.c diff --git a/disobedience/properties.c b/disobedience/properties.c index 82af9ad..57ed65d 100644 --- a/disobedience/properties.c +++ b/disobedience/properties.c @@ -29,7 +29,7 @@ static void completed_namepart(struct prefdata *f); static const char *get_edited_namepart(struct prefdata *f); static void set_edited_namepart(struct prefdata *f, const char *value); static void set_namepart(struct prefdata *f, const char *value); -static void set_namepart_completed(void *v); +static void set_namepart_completed(void *v, const char *error); static void kickoff_string(struct prefdata *f); static void completed_string(struct prefdata *f); @@ -43,7 +43,7 @@ static const char *get_edited_boolean(struct prefdata *f); static void set_edited_boolean(struct prefdata *f, const char *value); static void set_boolean(struct prefdata *f, const char *value); -static void prefdata_completed(void *v, const char *value); +static void prefdata_completed(void *v, const char *error, const char *value); static void prefdata_onerror(struct callbackdata *cbd, int code, const char *msg); @@ -55,6 +55,10 @@ static void properties_ok(GtkButton *button, gpointer userdata); static void properties_apply(GtkButton *button, gpointer userdata); static void properties_cancel(GtkButton *button, gpointer userdata); +static void properties_logged_in(const char *event, + void *eventdata, + void *callbackdata); + /** @brief Data for a single preference */ struct prefdata { const char *track; @@ -121,27 +125,31 @@ static const struct pref { { "Album", "album", 0, &preftype_namepart }, { "Title", "title", 0, &preftype_namepart }, { "Tags", "tags", "", &preftype_string }, + { "Weight", "weight", "90000", &preftype_string }, { "Random", "pick_at_random", "1", &preftype_boolean }, }; #define NPREFS (int)(sizeof prefs / sizeof *prefs) /* Buttons that appear at the bottom of the window */ -static const struct button buttons[] = { +static struct button buttons[] = { { GTK_STOCK_OK, properties_ok, - "Apply all changes and close window" + "Apply all changes and close window", + 0 }, { GTK_STOCK_APPLY, properties_apply, - "Apply all changes and keep window open" + "Apply all changes and keep window open", + 0 }, { GTK_STOCK_CANCEL, properties_cancel, - "Discard all changes and close window" + "Discard all changes and close window", + 0 }, }; @@ -153,6 +161,7 @@ static struct prefdata *prefdatas; /* Current prefdatas */ static GtkWidget *properties_window; static GtkWidget *properties_table; static struct progress_window *pw; +static event_handle properties_event; static void propagate_clicked(GtkButton attribute((unused)) *button, gpointer userdata) { @@ -170,8 +179,7 @@ static void propagate_clicked(GtkButton attribute((unused)) *button, void properties(int ntracks, const char **tracks) { int n, m; struct prefdata *f; - GtkWidget *buttonbox, *vbox, *label, *entry, *propagate, *content; - GdkPixbuf *pb; + GtkWidget *buttonbox, *vbox, *label, *entry, *propagate; /* If no tracks, do nothign */ if(!ntracks) @@ -188,13 +196,16 @@ void properties(int ntracks, const char **tracks) { popup_msg(GTK_MESSAGE_ERROR, "Too many tracks selected"); return; } + properties_event = event_register("logged-in", properties_logged_in, 0); /* Create a new properties window */ properties_window = gtk_window_new(GTK_WINDOW_TOPLEVEL); + gtk_widget_set_style(properties_window, tool_style); g_signal_connect(properties_window, "destroy", G_CALLBACK(gtk_widget_destroyed), &properties_window); /* Most of the action is the table of preferences */ properties_table = gtk_table_new((NPREFS + 1) * ntracks, 2 + ntracks > 1, FALSE); + gtk_widget_set_style(properties_table, tool_style); g_signal_connect(properties_table, "destroy", G_CALLBACK(gtk_widget_destroyed), &properties_table); gtk_window_set_title(GTK_WINDOW(properties_window), "Track Properties"); @@ -206,6 +217,7 @@ void properties(int ntracks, const char **tracks) { /* The track itself */ /* Caption */ label = gtk_label_new("Track"); + gtk_widget_set_style(label, tool_style); gtk_misc_set_alignment(GTK_MISC(label), 1, 0); gtk_table_attach(GTK_TABLE(properties_table), label, @@ -215,6 +227,7 @@ void properties(int ntracks, const char **tracks) { 1, 1); /* The track name */ entry = gtk_entry_new(); + gtk_widget_set_style(entry, tool_style); gtk_entry_set_text(GTK_ENTRY(entry), tracks[n]); gtk_editable_set_editable(GTK_EDITABLE(entry), FALSE); gtk_table_attach(GTK_TABLE(properties_table), @@ -227,6 +240,7 @@ void properties(int ntracks, const char **tracks) { for(m = 0; m < NPREFS; ++m) { /* Caption */ label = gtk_label_new(prefs[m].label); + gtk_widget_set_style(label, tool_style); gtk_misc_set_alignment(GTK_MISC(label), 1, 0); gtk_table_attach(GTK_TABLE(properties_table), label, @@ -242,13 +256,7 @@ void properties(int ntracks, const char **tracks) { prefs[m].type->kickoff(f); if(ntracks > 1) { /* Propagation button */ - propagate = gtk_button_new(); - if((pb = find_image("propagate.png"))) - content = gtk_image_new_from_pixbuf(pb); - else - content = gtk_label_new("propagate.png"); - gtk_container_add(GTK_CONTAINER(propagate), content); - gtk_tooltips_set_tip(tips, propagate, "Copy to other tracks", ""); + propagate = iconbutton("propagate.png", "Copy to other tracks"); g_signal_connect(G_OBJECT(propagate), "clicked", G_CALLBACK(propagate_clicked), f); gtk_table_attach(GTK_TABLE(properties_table), @@ -269,7 +277,7 @@ void properties(int ntracks, const char **tracks) { scroll_widget(properties_table), TRUE, TRUE, 1); gtk_box_pack_start(GTK_BOX(vbox), buttonbox, FALSE, FALSE, 1); - gtk_container_add(GTK_CONTAINER(properties_window), vbox); + gtk_container_add(GTK_CONTAINER(properties_window), frame_widget(vbox, NULL)); /* The table only really wants to be vertically scrollable */ gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(GTK_WIDGET(properties_table)->parent->parent), GTK_POLICY_NEVER, @@ -322,24 +330,25 @@ static void set_edited_namepart(struct prefdata *f, const char *value) { static void set_namepart(struct prefdata *f, const char *value) { char *s; - struct callbackdata *cbd = xmalloc(sizeof *cbd); - cbd->u.f = f; byte_xasprintf(&s, "trackname_display_%s", f->p->part); /* We don't know what the default is so can never unset. This is a bug * relative to the original design, which is supposed to only ever allow for * non-trivial namepart preferences. I suppose the server could spot a * default being set and translate it into an unset. */ disorder_eclient_set(client, set_namepart_completed, f->track, s, value, - cbd); + f); } /* Called when we've set a namepart */ -static void set_namepart_completed(void *v) { - struct callbackdata *cbd = v; - struct prefdata *f = cbd->u.f; - - namepart_update(f->track, "display", f->p->part); +static void set_namepart_completed(void *v, const char *error) { + if(error) + popup_protocol_error(0, error); + else { + struct prefdata *f = v; + + namepart_update(f->track, "display", f->p->part); + } } /* String preferences ------------------------------------------------------ */ @@ -364,15 +373,15 @@ static void set_edited_string(struct prefdata *f, const char *value) { gtk_entry_set_text(GTK_ENTRY(f->widget), value); } +static void set_string_completed(void attribute((unused)) *v, + const char *error) { + if(error) + popup_protocol_error(0, error); +} + static void set_string(struct prefdata *f, const char *value) { - if(strcmp(f->p->default_value, value)) - /* Different from default, set it */ - disorder_eclient_set(client, 0/*completed*/, f->track, f->p->part, - value, 0/*v*/); - else - /* Same as default, just unset */ - disorder_eclient_unset(client, 0/*completed*/, f->track, f->p->part, - 0/*v*/); + disorder_eclient_set(client, set_string_completed, f->track, f->p->part, + value, 0/*v*/); } /* Boolean preferences ----------------------------------------------------- */ @@ -384,6 +393,7 @@ static void kickoff_boolean(struct prefdata *f) { static void completed_boolean(struct prefdata *f) { f->widget = gtk_check_button_new(); + gtk_widget_set_style(f->widget, tool_style); if(!f->value) /* Not set, use the default */ f->value = f->p->default_value; @@ -399,17 +409,14 @@ static void set_edited_boolean(struct prefdata *f, const char *value) { strcmp(value, "0")); } +#define set_boolean_completed set_string_completed + static void set_boolean(struct prefdata *f, const char *value) { char *s; byte_xasprintf(&s, "trackname_display_%s", f->p->part); - if(strcmp(value, f->p->default_value)) - disorder_eclient_set(client, 0/*completed*/, f->track, f->p->part, value, - 0/*v*/); - else - /* If default value then delete the pref */ - disorder_eclient_unset(client, 0/*completed*/, f->track, f->p->part, - 0/*v*/); + disorder_eclient_set(client, set_boolean_completed, + f->track, f->p->part, value, 0/*v*/); } /* Querying preferences ---------------------------------------------------- */ @@ -431,10 +438,13 @@ static void prefdata_onerror(struct callbackdata *cbd, } /* Got the value of a pref */ -static void prefdata_completed(void *v, const char *value) { - struct callbackdata *cbd = v; - - prefdata_completed_common(cbd->u.f, value); +static void prefdata_completed(void *v, const char *error, const char *value) { + if(error) { + } else { + struct callbackdata *cbd = v; + + prefdata_completed_common(cbd->u.f, value); + } } static void prefdata_completed_common(struct prefdata *f, @@ -485,13 +495,17 @@ static void properties_apply(GtkButton attribute((unused)) *button, static void properties_cancel(GtkButton attribute((unused)) *button, gpointer attribute((unused)) userdata) { gtk_widget_destroy(properties_window); + event_cancel(properties_event); + properties_event = 0; } -/** @brief Called on client reset +/** @brief Called when we've just logged in * * Destroys the current properties window. */ -void properties_reset(void) { +static void properties_logged_in(const char attribute((unused)) *event, + void attribute((unused)) *eventdata, + void attribute((unused)) *callbackdata) { if(properties_window) gtk_widget_destroy(properties_window); }