* When you select 'add' a new empty set of details are displayed to be edited.
* Again Apply will commit them.
*
- * TODO: @ref RIGHT_ADMIN and @ref RIGHT_USERINFO should be applied here, so we
- * can give decent error messages.
- *
* TODO: it would be really nice if the Username entry could be removed and new
* user names entered in the list, rather off in the details panel. This may
* be possible with a sufficiently clever GtkCellRenderer.
GtkWidget **checkp = &users_details_rights[leftmost_bit(right)];
if(!(check = *checkp)) {
- *checkp = check = gtk_check_button_new();
+ *checkp = check = gtk_check_button_new_with_label("");
users_detail_generic(title, check);
}
gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(check), !!(value & right));
rights_type mask) {
const uint32_t first = mask / 7;
const int bit = leftmost_bit(first);
- GtkWidget **widgets = &users_details_rights[bit], *any, *mine, *random;
+ GtkWidget **widgets = &users_details_rights[bit], *any, *mine, *rnd;
if(!*widgets) {
GtkWidget *hbox = gtk_hbox_new(FALSE, 2);
any = widgets[0] = gtk_check_button_new_with_label("Any");
mine = widgets[1] = gtk_check_button_new_with_label("Own");
- random = widgets[2] = gtk_check_button_new_with_label("Random");
+ rnd = widgets[2] = gtk_check_button_new_with_label("Random");
gtk_box_pack_start(GTK_BOX(hbox), any, FALSE, FALSE, 0);
gtk_box_pack_start(GTK_BOX(hbox), mine, FALSE, FALSE, 0);
- gtk_box_pack_start(GTK_BOX(hbox), random, FALSE, FALSE, 0);
+ gtk_box_pack_start(GTK_BOX(hbox), rnd, FALSE, FALSE, 0);
users_detail_generic(title, hbox);
g_signal_connect(any, "toggled", G_CALLBACK(users_any_toggled), NULL);
users_details_rights[bit] = any;
users_details_rights[bit + 1] = mine;
- users_details_rights[bit + 2] = random;
+ users_details_rights[bit + 2] = rnd;
} else {
any = widgets[0];
mine = widgets[1];
- random = widgets[2];
+ rnd = widgets[2];
}
/* Discard irrelevant bits */
bits &= mask;
bits >>= bit;
gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(any), !!(bits & 1));
gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(mine), !!(bits & 2));
- gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(random), !!(bits & 4));
+ gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(rnd), !!(bits & 4));
}
/** @brief Called when the details table is destroyed */
users_delete_button = users_buttons[1].widget;
/* Buttons live below the list */
- vbox = gtk_vbox_new(FALSE, 2);
- gtk_box_pack_start(GTK_BOX(vbox), tree, TRUE/*expand*/, TRUE/*fill*/, 0);
+ vbox = gtk_vbox_new(FALSE, 0);
+ gtk_box_pack_start(GTK_BOX(vbox), scroll_widget(tree), TRUE/*expand*/, TRUE/*fill*/, 0);
gtk_box_pack_start(GTK_BOX(vbox), buttons, FALSE/*expand*/, FALSE, 0);
/* Create an empty user details table, and put an apply button below it */
gtk_box_pack_start(GTK_BOX(vbox2), hbox2,
FALSE/*expand*/, FALSE, 0);
- /* User details are to the right of the list */
- hbox = gtk_hbox_new(FALSE, 2);
- gtk_box_pack_start(GTK_BOX(hbox), vbox, FALSE/*expand*/, FALSE, 2);
- gtk_box_pack_start(GTK_BOX(hbox), vbox2, TRUE/*expand*/, TRUE/*fill*/, 2);
- gtk_container_add(GTK_CONTAINER(users_window), hbox);
+ /* User details are to the right of the list. We put in a pointless event
+ * box as as spacer, so that the longest label in the user details isn't
+ * cuddled up to the user list. */
+ hbox = gtk_hbox_new(FALSE, 0);
+ gtk_box_pack_start(GTK_BOX(hbox), vbox, FALSE/*expand*/, FALSE, 0);
+ gtk_box_pack_start(GTK_BOX(hbox), gtk_event_box_new(), FALSE/*expand*/, FALSE, 2);
+ gtk_box_pack_start(GTK_BOX(hbox), vbox2, TRUE/*expand*/, TRUE/*fill*/, 0);
+ gtk_container_add(GTK_CONTAINER(users_window), frame_widget(hbox, NULL));
}
/*