*
* 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.
*/
#include "disobedience.h"
gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(random), !!(bits & 4));
}
+/** @brief Called when the details table is destroyed */
+static void users_details_destroyed(GtkWidget attribute((unused)) *widget,
+ GtkWidget attribute((unused)) **wp) {
+ users_details_table = 0;
+ g_object_unref(users_list);
+ users_list = 0;
+ users_details_name = 0;
+ users_details_email = 0;
+ users_details_password = 0;
+ users_details_password2 = 0;
+ memset(users_details_rights, 0, sizeof users_details_rights);
+ /* also users_selection? Not AFAICT; _get_selection does upref */
+}
+
/** @brief Create or modify the user details table
* @param name User name (users_edit()) or NULL (users_add())
* @param email Email address
rights_type r = 0;
/* Create the table if it doesn't already exist */
- if(!users_details_table)
+ if(!users_details_table) {
users_details_table = gtk_table_new(4, 2, FALSE/*!homogeneous*/);
+ g_signal_connect(users_details_table, "destroy",
+ G_CALLBACK(users_details_destroyed), 0);
+ }
/* Create or update the widgets */
users_add_detail(&users_details_name, "Username", name,
/** @brief Table of buttons below the user list */
static struct button users_buttons[] = {
{
- "Add user",
+ GTK_STOCK_ADD,
users_add,
"Create a new user",
0
},
{
- "Delete user",
+ GTK_STOCK_REMOVE,
users_delete,
"Delete a user",
0
/** @brief Pop up the user management window */
void manage_users(void) {
- GtkWidget *tree, *buttons, *hbox, *vbox, *vbox2;
+ GtkWidget *tree, *buttons, *hbox, *hbox2, *vbox, *vbox2;
GtkCellRenderer *cr;
GtkTreeViewColumn *col;
users_makedetails("", "", "", "",
DETAIL_VISIBLE,
DETAIL_VISIBLE);
- /* TODO apply button is much too wide right now... */
g_signal_connect(users_apply_button, "clicked",
G_CALLBACK(users_apply), NULL);
- vbox2 = gtk_vbox_new(FALSE, 2);
+ hbox2 = gtk_hbox_new(FALSE, 0);
+ gtk_box_pack_end(GTK_BOX(hbox2), users_apply_button,
+ FALSE/*expand*/, FALSE, 0);
+
+ vbox2 = gtk_vbox_new(FALSE, 0);
gtk_box_pack_start(GTK_BOX(vbox2), users_details_table,
TRUE/*expand*/, TRUE/*fill*/, 0);
- gtk_box_pack_start(GTK_BOX(vbox2), users_apply_button,
+ 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, 0);
- gtk_box_pack_start(GTK_BOX(hbox), vbox2, TRUE/*expand*/, TRUE/*fill*/, 0);
+ 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);
}