X-Git-Url: http://git.tdb.fi/?p=ext%2Fsubsurface.git;a=blobdiff_plain;f=info.c;h=1847a49bf98ba8aef7e545d98e57ca58f2b6dfb8;hp=9ca385796298ccc85a46243aa0806eebbac4f7bc;hb=618a20ba5f2a9adc0e5a35117535f8eaa9fd34a4;hpb=bbf5f960e11982551a3fd873c4432964ec2a0717

diff --git a/info.c b/info.c
index 9ca3857..1847a49 100644
--- a/info.c
+++ b/info.c
@@ -1,297 +1,207 @@
+/* info.c */
+/* creates the UI for the info frame - 
+ * controlled through the following interfaces:
+ * 
+ * void show_dive_info(struct dive *dive)
+ *
+ * called from gtk-ui:
+ * GtkWidget *extended_dive_info_widget(void)
+ */
 #include <stdio.h>
 #include <string.h>
 #include <stdlib.h>
 #include <time.h>
+#include <ctype.h>
+#include <sys/time.h>
 
 #include "dive.h"
 #include "display.h"
+#include "display-gtk.h"
 #include "divelist.h"
 
-static GtkWidget *info_frame;
-static GtkWidget *depth, *duration, *temperature, *airconsumption;
-static GtkEntry *location, *buddy, *divemaster;
-static GtkTextBuffer *notes;
-static int location_changed = 1, notes_changed = 1;
-static int divemaster_changed = 1, buddy_changed = 1;
-
-#define EMPTY_AIRCONSUMPTION " \n "
-#define AIRCON_WIDTH 20
-
-static const char *weekday(int wday)
-{
-	static const char wday_array[7][4] = {
-		"Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat"
-	};
-	return wday_array[wday];
-}
+static GtkEntry *location, *buddy, *divemaster, *rating;
+static GtkTextView *notes;
+static GtkListStore *location_list, *people_list, *star_list;
 
-static char *get_text(GtkTextBuffer *buffer)
+static char *get_text(GtkTextView *view)
 {
+	GtkTextBuffer *buffer;
 	GtkTextIter start;
 	GtkTextIter end;
 
+	buffer = gtk_text_view_get_buffer(view);
 	gtk_text_buffer_get_start_iter(buffer, &start);
 	gtk_text_buffer_get_end_iter(buffer, &end);
 	return gtk_text_buffer_get_text(buffer, &start, &end, FALSE);
 }
 
-void update_air_info(char *buffer)
+/* old is NULL or a valid string, new is a valid string
+ * NOTW: NULL and "" need to be treated as "unchanged" */
+static int text_changed(const char *old, const char *new)
 {
-	char markup[120];
-	
-	if (! buffer)
-		buffer = EMPTY_AIRCONSUMPTION;
-	snprintf(markup, sizeof(markup), "<span font=\"8\">%s</span>",buffer);
-	gtk_label_set_markup(GTK_LABEL(airconsumption), markup);
+	return (old && strcmp(old,new)) ||
+		(!old && strcmp("",new));
 }
 
-/*
- * Return air usage (in liters).
- */
-static double calculate_airuse(struct dive *dive)
+static char *get_combo_box_entry_text(GtkComboBoxEntry *combo_box, char **textp)
 {
-	double airuse = 0;
-	int i;
-
-	for (i = 0; i < MAX_CYLINDERS; i++) {
-		cylinder_t *cyl = dive->cylinder + i;
-		int size = cyl->type.size.mliter;
-		double kilo_atm;
-
-		if (!size)
-			continue;
-
-		kilo_atm = (cyl->start.mbar - cyl->end.mbar) / 1013250.0;
-
-		/* Liters of air at 1 atm == milliliters at 1k atm*/
-		airuse += kilo_atm * size;
-	}
-	return airuse;
+	char *old = *textp;
+	const gchar *new;
+	GtkEntry *entry;
+
+	entry = GTK_ENTRY(gtk_bin_get_child(GTK_BIN(combo_box)));
+	new = gtk_entry_get_text(entry);
+	while (isspace(*new))
+		new++;
+	if (!text_changed(old,new))
+		return NULL;
+	free(old);
+	*textp = strdup(new);
+	return *textp;
 }
 
-static void update_air_info_frame(struct dive *dive)
-{
-	const double liters_per_cuft = 28.317;
-	const char *unit, *format, *desc;
-	double airuse;
-	char buffer1[80];
-	char buffer2[80];
-	int len;
-
-	airuse = calculate_airuse(dive);
-	if (!airuse) {
-		update_air_info(NULL);
-		return;
-	}
-	switch (output_units.volume) {
-	case LITER:
-		unit = "l";
-		format = "vol: %4.0f %s";
-		break;
-	case CUFT:
-		unit = "cuft";
-		format = "vol: %4.2f %s";
-		airuse /= liters_per_cuft;
-		break;
-	}
-	len = snprintf(buffer1, sizeof(buffer1), format, airuse, unit);
-	if (dive->duration.seconds) {
-		double pressure = 1 + (dive->meandepth.mm / 10000.0);
-		double sac = airuse / pressure * 60 / dive->duration.seconds;
-		snprintf(buffer1+len, sizeof(buffer1)-len, 
-				"\nSAC: %4.2f %s/min", sac, unit);
-	}
-	len = 0;
-	desc = dive->cylinder[0].type.description;
-	if (desc || dive->cylinder[0].gasmix.o2.permille) {
-		int o2 = dive->cylinder[0].gasmix.o2.permille / 10;
-		if (!desc)
-			desc = "";
-		if (!o2)
-			o2 = 21;
-		len = snprintf(buffer2, sizeof(buffer2), "%s (%d%%): used ", desc, o2);
-	}
-	snprintf(buffer2+len, sizeof(buffer2)-len, buffer1); 
-	update_air_info(buffer2);
-}
+#define SET_TEXT_VALUE(x) \
+	gtk_entry_set_text(x, dive && dive->x ? dive->x : "")
 
-void flush_dive_info_changes(struct dive *dive)
+static int divename(char *buf, size_t size, struct dive *dive)
 {
-	if (!dive)
-		return;
-
-	if (location_changed) {
-		g_free(dive->location);
-		dive->location = gtk_editable_get_chars(GTK_EDITABLE(location), 0, -1);
-	}
-
-	if (divemaster_changed) {
-		g_free(dive->divemaster);
-		dive->divemaster = gtk_editable_get_chars(GTK_EDITABLE(divemaster), 0, -1);
-	}
-
-	if (buddy_changed) {
-		g_free(dive->buddy);
-		dive->buddy = gtk_editable_get_chars(GTK_EDITABLE(buddy), 0, -1);
-	}
-
-	if (notes_changed) {
-		g_free(dive->notes);
-		dive->notes = get_text(notes);
-	}
+	struct tm *tm = gmtime(&dive->when);
+	return snprintf(buf, size, "Dive #%d - %s %02d/%02d/%04d at %d:%02d",
+		dive->number,
+		weekday(tm->tm_wday),
+		tm->tm_mon+1, tm->tm_mday,
+		tm->tm_year+1900,
+		tm->tm_hour, tm->tm_min);
 }
 
 void show_dive_info(struct dive *dive)
 {
-	struct tm *tm;
+	const char *text;
 	char buffer[80];
-	char *text;
 
-	if (!dive) {
-		gtk_label_set_text(GTK_LABEL(depth), "");
-		gtk_label_set_text(GTK_LABEL(duration), "");
-		gtk_label_set_text(GTK_LABEL(airconsumption), EMPTY_AIRCONSUMPTION);
-		gtk_label_set_width_chars(GTK_LABEL(airconsumption), AIRCON_WIDTH);
-		return;
-	}
 	/* dive number and location (or lacking that, the date) go in the window title */
-	tm = gmtime(&dive->when);
 	text = dive->location;
 	if (!text)
 		text = "";
 	if (*text) {
 		snprintf(buffer, sizeof(buffer), "Dive #%d - %s", dive->number, text);
 	} else {
-		snprintf(buffer, sizeof(buffer), "Dive #%d - %s %02d/%02d/%04d at %d:%02d",
-			dive->number,
-			weekday(tm->tm_wday),
-			tm->tm_mon+1, tm->tm_mday,
-			tm->tm_year+1900,
-			tm->tm_hour, tm->tm_min);
+		divename(buffer, sizeof(buffer), dive);
 	}
 	text = buffer;
 	if (!dive->number)
-		text += 10;	/* Skip the "Dive #0 - " part */
+		text += 10;     /* Skip the "Dive #0 - " part */
 	gtk_window_set_title(GTK_WINDOW(main_window), text);
 
-	/* the date goes in the frame label */
-	snprintf(buffer, sizeof(buffer), "%s %02d/%02d/%04d at %d:%02d",
-		weekday(tm->tm_wday),
-		tm->tm_mon+1, tm->tm_mday,
-		tm->tm_year+1900,
-		tm->tm_hour, tm->tm_min);
-	gtk_frame_set_label(GTK_FRAME(info_frame), buffer);
+	SET_TEXT_VALUE(divemaster);
+	SET_TEXT_VALUE(buddy);
+	SET_TEXT_VALUE(location);
+	gtk_entry_set_text(rating, star_strings[dive->rating]);
+	gtk_text_buffer_set_text(gtk_text_view_get_buffer(notes),
+		dive && dive->notes ? dive->notes : "", -1);
+}
 
+static int delete_dive_info(struct dive *dive)
+{
+	int success;
+	GtkWidget *dialog;
 
-	switch (output_units.length) {
-	case METERS:
-		snprintf(buffer, sizeof(buffer),
-			"%.1f m",
-			dive->maxdepth.mm / 1000.0);
-		break;
-	case FEET:
-		snprintf(buffer, sizeof(buffer),
-			"%d ft",
-			to_feet(dive->maxdepth));
-		break;
-	}
-	gtk_label_set_text(GTK_LABEL(depth), buffer);
-
-	snprintf(buffer, sizeof(buffer),
-		"%d min",
-		dive->duration.seconds / 60);
-	gtk_label_set_text(GTK_LABEL(duration), buffer);
-
-	*buffer = 0;
-	if (dive->watertemp.mkelvin) {
-		switch (output_units.temperature) {
-		case CELSIUS:
-			snprintf(buffer, sizeof(buffer),
-				"%d C",
-				to_C(dive->watertemp));
-			break;
-		case FAHRENHEIT:
-			snprintf(buffer, sizeof(buffer),
-				"%d F",
-				to_F(dive->watertemp));
-			break;
-		case KELVIN:
-			snprintf(buffer, sizeof(buffer),
-				"%d K",
-				to_K(dive->watertemp));
-			break;
-		}
+	if (!dive)
+		return 0;
+
+	dialog = gtk_dialog_new_with_buttons("Delete Dive",
+		GTK_WINDOW(main_window),
+		GTK_DIALOG_DESTROY_WITH_PARENT,
+		GTK_STOCK_OK, GTK_RESPONSE_ACCEPT,
+		GTK_STOCK_CANCEL, GTK_RESPONSE_REJECT,
+		NULL);
+
+	gtk_widget_show_all(dialog);
+	success = gtk_dialog_run(GTK_DIALOG(dialog)) == GTK_RESPONSE_ACCEPT;
+	if (success) {
+		delete_dive(dive);
+		mark_divelist_changed(TRUE);
+		dive_list_update_dives();
 	}
-	gtk_label_set_text(GTK_LABEL(temperature), buffer);
 
-	text = dive->location ? : "";
-	gtk_entry_set_text(location, text);
+	gtk_widget_destroy(dialog);
 
-	text = dive->divemaster ? : "";
-	gtk_entry_set_text(divemaster, text);
-
-	text = dive->buddy ? : "";
-	gtk_entry_set_text(buddy, text);
-
-	text = dive->notes ? : "";
-	gtk_text_buffer_set_text(notes, text, -1);
-
-	update_air_info_frame(dive);
+	return success;
 }
 
-static GtkWidget *info_label(GtkWidget *box, const char *str, GtkJustification jtype)
+static void info_menu_edit_cb(GtkMenuItem *menuitem, gpointer user_data)
 {
-	GtkWidget *label = gtk_label_new(str);
-	gtk_label_set_justify(GTK_LABEL(label), jtype);
-	gtk_box_pack_start(GTK_BOX(box), label, TRUE, TRUE, 0);
-	return label;
+	edit_dive_info(current_dive);
 }
 
-GtkWidget *dive_info_frame(void)
+static void info_menu_delete_cb(GtkMenuItem *menuitem, gpointer user_data)
 {
-	GtkWidget *frame;
-	GtkWidget *hbox;
-	GtkWidget *vbox;
-
-	frame = gtk_frame_new("Dive info");
-	info_frame = frame;
-	gtk_widget_show(frame);
+	delete_dive_info(current_dive);
+}
 
-	vbox = gtk_vbox_new(FALSE, 6);
-	gtk_container_set_border_width(GTK_CONTAINER(vbox), 3);
-	gtk_container_add(GTK_CONTAINER(frame), vbox);
+static void add_menu_item(GtkMenu *menu, const char *label, void (*cb)(GtkMenuItem *, gpointer))
+{
+	GtkWidget *item = gtk_menu_item_new_with_label(label);
+	g_signal_connect(item, "activate", G_CALLBACK(cb), NULL);
+	gtk_widget_show(item); /* Yes, really */
+	gtk_menu_prepend(menu, item);
+}
 
-	hbox = gtk_hbox_new(FALSE, 16);
-	gtk_container_set_border_width(GTK_CONTAINER(hbox), 3);
-	gtk_box_pack_start(GTK_BOX(vbox), hbox, FALSE, TRUE, 0);
+static void populate_popup_cb(GtkTextView *entry, GtkMenu *menu, gpointer user_data)
+{
+	add_menu_item(menu, "Delete", info_menu_delete_cb);
+	add_menu_item(menu, "Edit", info_menu_edit_cb);
+}
 
-	depth = info_label(hbox, "depth", GTK_JUSTIFY_RIGHT);
-	duration = info_label(hbox, "duration", GTK_JUSTIFY_RIGHT);
-	temperature = info_label(hbox, "temperature", GTK_JUSTIFY_RIGHT);
-	airconsumption = info_label(hbox, "air", GTK_JUSTIFY_RIGHT);
-	gtk_misc_set_alignment(GTK_MISC(airconsumption), 1.0, 0.5);
-	gtk_label_set_width_chars(GTK_LABEL(airconsumption), AIRCON_WIDTH);
+static GtkEntry *text_value(GtkWidget *box, const char *label)
+{
+	GtkWidget *widget;
+	GtkWidget *frame = gtk_frame_new(label);
 
-	return frame;
+	gtk_box_pack_start(GTK_BOX(box), frame, FALSE, TRUE, 0);
+	widget = gtk_entry_new();
+	gtk_widget_set_can_focus(widget, FALSE);
+	gtk_editable_set_editable(GTK_EDITABLE(widget), FALSE);
+	gtk_container_add(GTK_CONTAINER(frame), widget);
+	g_signal_connect(widget, "populate-popup", G_CALLBACK(populate_popup_cb), NULL);
+	return GTK_ENTRY(widget);
 }
 
-static GtkEntry *text_entry(GtkWidget *box, const char *label)
+static GtkComboBoxEntry *text_entry(GtkWidget *box, const char *label, GtkListStore *completions, const char *text)
 {
-	GtkWidget *entry;
+	GtkEntry *entry;
+	GtkWidget *combo_box;
 	GtkWidget *frame = gtk_frame_new(label);
+	GtkEntryCompletion *completion;
 
 	gtk_box_pack_start(GTK_BOX(box), frame, FALSE, TRUE, 0);
 
-	entry = gtk_entry_new();
-	gtk_container_add(GTK_CONTAINER(frame), entry);
+	combo_box = gtk_combo_box_entry_new_with_model(GTK_TREE_MODEL(completions), 0);
+	gtk_container_add(GTK_CONTAINER(frame), combo_box);
+
+	entry = GTK_ENTRY(gtk_bin_get_child(GTK_BIN(combo_box)));
+	if (text && *text)
+		gtk_entry_set_text(entry, text);
 
-	return GTK_ENTRY(entry);
+	completion = gtk_entry_completion_new();
+	gtk_entry_completion_set_text_column(completion, 0);
+	gtk_entry_completion_set_model(completion, GTK_TREE_MODEL(completions));
+	gtk_entry_completion_set_inline_completion(completion, TRUE);
+	gtk_entry_completion_set_inline_selection(completion, TRUE);
+	gtk_entry_completion_set_popup_single_match(completion, FALSE);
+	gtk_entry_set_completion(entry, completion);
+
+	return GTK_COMBO_BOX_ENTRY(combo_box);
 }
 
-static GtkTextBuffer *text_view(GtkWidget *box, const char *label)
+enum writable {
+	READ_ONLY,
+	READ_WRITE
+};
+
+static GtkTextView *text_view(GtkWidget *box, const char *label, enum writable writable)
 {
 	GtkWidget *view, *vbox;
-	GtkTextBuffer *buffer;
 	GtkWidget *frame = gtk_frame_new(label);
 
 	gtk_box_pack_start(GTK_BOX(box), frame, TRUE, TRUE, 0);
@@ -303,16 +213,339 @@ static GtkTextBuffer *text_view(GtkWidget *box, const char *label)
 	GtkWidget* scrolled_window = gtk_scrolled_window_new(0, 0);
 	gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scrolled_window), GTK_POLICY_AUTOMATIC, GTK_POLICY_AUTOMATIC);
 	gtk_scrolled_window_set_shadow_type(GTK_SCROLLED_WINDOW(scrolled_window), GTK_SHADOW_IN);
-	gtk_widget_show(scrolled_window);
 
 	view = gtk_text_view_new();
+	if (writable == READ_ONLY) {
+		gtk_widget_set_can_focus(view, FALSE);
+		gtk_text_view_set_editable(GTK_TEXT_VIEW(view), FALSE);
+		gtk_text_view_set_cursor_visible(GTK_TEXT_VIEW(view), FALSE);
+		g_signal_connect(view, "populate-popup", G_CALLBACK(populate_popup_cb), NULL);
+	}
 	gtk_text_view_set_wrap_mode(GTK_TEXT_VIEW(view), GTK_WRAP_WORD);
 	gtk_container_add(GTK_CONTAINER(scrolled_window), view);
+	gtk_box_pack_start(GTK_BOX(vbox), scrolled_window, TRUE, TRUE, 0);
+	return GTK_TEXT_VIEW(view);
+}
 
-	buffer = gtk_text_view_get_buffer (GTK_TEXT_VIEW (view));
+static enum {
+	MATCH_EXACT,
+	MATCH_PREPEND,
+	MATCH_AFTER
+} found_string_entry;
+static GtkTreeIter string_entry_location;
 
-	gtk_box_pack_start(GTK_BOX(vbox), scrolled_window, TRUE, TRUE, 0);
-	return buffer;
+static gboolean match_string_entry(GtkTreeModel *model, GtkTreePath *path, GtkTreeIter *iter, gpointer data)
+{
+	const char *string = data;
+	char *entry;
+	int cmp;
+
+	gtk_tree_model_get(model, iter, 0, &entry, -1);
+	cmp = strcmp(entry, string);
+
+	/* Stop. The entry is bigger than the new one */
+	if (cmp > 0)
+		return TRUE;
+
+	/* Exact match */
+	if (!cmp) {
+		found_string_entry = MATCH_EXACT;
+		return TRUE;
+	}
+
+	string_entry_location = *iter;
+	found_string_entry = MATCH_AFTER;
+	return FALSE;
+}
+
+static int match_list(GtkListStore *list, const char *string)
+{
+	found_string_entry = MATCH_PREPEND;
+	gtk_tree_model_foreach(GTK_TREE_MODEL(list), match_string_entry, (void *)string);
+	return found_string_entry;
+}
+
+static void add_string_list_entry(const char *string, GtkListStore *list)
+{
+	GtkTreeIter *iter, loc;
+
+	if (!string || !*string)
+		return;
+
+	switch (match_list(list, string)) {
+	case MATCH_EXACT:
+		return;
+	case MATCH_PREPEND:
+		iter = NULL;
+		break;
+	case MATCH_AFTER:
+		iter = &string_entry_location;
+		break;
+	}
+	gtk_list_store_insert_after(list, &loc, iter);
+	gtk_list_store_set(list, &loc, 0, string, -1);
+}
+
+void add_people(const char *string)
+{
+	add_string_list_entry(string, people_list);
+}
+
+void add_location(const char *string)
+{
+	add_string_list_entry(string, location_list);
+}
+
+static int get_rating(const char *string)
+{
+	int rating_val = 0;
+	int i;
+
+	for (i = 0; i <= 5; i++)
+		if (!strcmp(star_strings[i],string))
+			rating_val = i;
+	return rating_val;
+}
+
+struct dive_info {
+	GtkComboBoxEntry *location, *divemaster, *buddy, *rating;
+	GtkTextView *notes;
+};
+
+static void save_dive_info_changes(struct dive *dive, struct dive_info *info)
+{
+	char *old_text, *new_text;
+	char *rating_string;
+	int changed = 0;
+
+	new_text = get_combo_box_entry_text(info->location, &dive->location);
+	if (new_text) {
+		add_location(new_text);
+		changed = 1;
+	}
+
+	new_text = get_combo_box_entry_text(info->divemaster, &dive->divemaster);
+	if (new_text) {
+		add_people(new_text);
+		changed = 1;
+	}
+
+	new_text = get_combo_box_entry_text(info->buddy, &dive->buddy);
+	if (new_text) {
+		add_people(new_text);
+		changed = 1;
+	}
+
+	rating_string = strdup(star_strings[dive->rating]);
+	new_text = get_combo_box_entry_text(info->rating, &rating_string);
+	if (new_text) {
+		dive->rating = get_rating(rating_string);
+		free(rating_string);
+		changed =1;
+	}
+
+	old_text = dive->notes;
+	dive->notes = get_text(info->notes);
+	if (text_changed(old_text,dive->notes))
+		changed = 1;
+	if (old_text)
+		g_free(old_text);
+
+	if (changed) {
+		mark_divelist_changed(TRUE);
+		update_dive(dive);
+	}
+}
+
+static void dive_info_widget(GtkWidget *box, struct dive *dive, struct dive_info *info)
+{
+	GtkWidget *hbox, *label, *cylinder, *frame;
+	char buffer[80];
+
+	divename(buffer, sizeof(buffer), dive);
+	label = gtk_label_new(buffer);
+	gtk_box_pack_start(GTK_BOX(box), label, FALSE, TRUE, 0);
+
+	info->location = text_entry(box, "Location", location_list, dive->location);
+
+	hbox = gtk_hbox_new(FALSE, 3);
+	gtk_box_pack_start(GTK_BOX(box), hbox, FALSE, TRUE, 0);
+
+	info->divemaster = text_entry(hbox, "Dive master", people_list, dive->divemaster);
+	info->buddy = text_entry(hbox, "Buddy", people_list, dive->buddy);
+
+	hbox = gtk_hbox_new(FALSE, 3);
+	gtk_box_pack_start(GTK_BOX(box), hbox, FALSE, TRUE, 0);
+
+	info->rating = text_entry(hbox, "Rating", star_list, star_strings[dive->rating]);
+
+	info->notes = text_view(box, "Notes", READ_WRITE);
+	if (dive->notes && *dive->notes)
+		gtk_text_buffer_set_text(gtk_text_view_get_buffer(info->notes), dive->notes, -1);
+
+	hbox = gtk_hbox_new(FALSE, 3);
+	gtk_box_pack_start(GTK_BOX(box), hbox, FALSE, TRUE, 0);
+
+	frame = gtk_frame_new("Cylinder");
+	cylinder = cylinder_list_widget();
+	gtk_container_add(GTK_CONTAINER(frame), cylinder);
+	gtk_box_pack_start(GTK_BOX(hbox), frame, FALSE, TRUE, 0);
+}
+
+int edit_dive_info(struct dive *dive)
+{
+	int success;
+	GtkWidget *dialog, *vbox;
+	struct dive_info info;
+
+	if (!dive)
+		return 0;
+
+	dialog = gtk_dialog_new_with_buttons("Dive Info",
+		GTK_WINDOW(main_window),
+		GTK_DIALOG_DESTROY_WITH_PARENT,
+		GTK_STOCK_OK, GTK_RESPONSE_ACCEPT,
+		GTK_STOCK_CANCEL, GTK_RESPONSE_REJECT,
+		NULL);
+
+	vbox = gtk_dialog_get_content_area(GTK_DIALOG(dialog));
+	dive_info_widget(vbox, dive, &info);
+
+	gtk_widget_show_all(dialog);
+	success = gtk_dialog_run(GTK_DIALOG(dialog)) == GTK_RESPONSE_ACCEPT;
+	if (success)
+		save_dive_info_changes(dive, &info);
+
+	gtk_widget_destroy(dialog);
+
+	return success;
+}
+
+static GtkWidget *frame_box(GtkWidget *vbox, const char *fmt, ...)
+{
+	va_list ap;
+	char buffer[64];
+	GtkWidget *frame, *hbox;
+
+	va_start(ap, fmt);
+	vsnprintf(buffer, sizeof(buffer), fmt, ap);
+	va_end(ap);
+
+	frame = gtk_frame_new(buffer);
+	gtk_box_pack_start(GTK_BOX(vbox), frame, FALSE, TRUE, 0);
+	hbox = gtk_hbox_new(0, 3);
+	gtk_container_add(GTK_CONTAINER(frame), hbox);
+	return hbox;
+}
+
+/* Fixme - should do at least depths too - a dive without a depth is kind of pointless */
+static time_t dive_time_widget(struct dive *dive)
+{
+	GtkWidget *dialog;
+	GtkWidget *cal, *hbox, *vbox, *box;
+	GtkWidget *h, *m;
+	GtkWidget *duration, *depth;
+	GtkWidget *label;
+	guint yval, mval, dval;
+	struct tm tm, *tmp;
+	struct timeval tv;
+	time_t time;
+	int success;
+	double depthinterval, val;
+
+	dialog = gtk_dialog_new_with_buttons("Date and Time",
+		GTK_WINDOW(main_window),
+		GTK_DIALOG_DESTROY_WITH_PARENT,
+		GTK_STOCK_OK, GTK_RESPONSE_ACCEPT,
+		GTK_STOCK_CANCEL, GTK_RESPONSE_REJECT,
+		NULL);
+
+	vbox = gtk_dialog_get_content_area(GTK_DIALOG(dialog));
+
+	/* Calendar hbox */
+	hbox = frame_box(vbox, "Date:");
+	cal = gtk_calendar_new();
+	gtk_box_pack_start(GTK_BOX(hbox), cal, FALSE, TRUE, 0);
+
+	/* Time hbox */
+	hbox = frame_box(vbox, "Time");
+
+	h = gtk_spin_button_new_with_range (0.0, 23.0, 1.0);
+	m = gtk_spin_button_new_with_range (0.0, 59.0, 1.0);
+
+	gettimeofday(&tv, NULL);
+	time = tv.tv_sec;
+	tmp = localtime(&time);
+	gtk_spin_button_set_value(GTK_SPIN_BUTTON(h), tmp->tm_hour);
+	gtk_spin_button_set_value(GTK_SPIN_BUTTON(m), (tmp->tm_min / 5)*5);
+
+	gtk_spin_button_set_wrap(GTK_SPIN_BUTTON(h), TRUE);
+	gtk_spin_button_set_wrap(GTK_SPIN_BUTTON(m), TRUE);
+
+	gtk_box_pack_end(GTK_BOX(hbox), m, FALSE, FALSE, 0);
+	label = gtk_label_new(":");
+	gtk_box_pack_end(GTK_BOX(hbox), label, FALSE, FALSE, 0);
+	gtk_box_pack_end(GTK_BOX(hbox), h, FALSE, FALSE, 0);
+
+	hbox = gtk_hbox_new(TRUE, 3);
+	gtk_box_pack_start(GTK_BOX(vbox), hbox, FALSE, FALSE, 0);
+
+	/* Duration hbox */
+	box = frame_box(hbox, "Duration (min)");
+	duration = gtk_spin_button_new_with_range (0.0, 1000.0, 1.0);
+	gtk_box_pack_end(GTK_BOX(box), duration, FALSE, FALSE, 0);
+
+	/* Depth box */
+	box = frame_box(hbox, "Depth (%s):", output_units.length == FEET ? "ft" : "m");
+	if (output_units.length == FEET) {
+		depthinterval = 1.0;
+	} else {
+		depthinterval = 0.1;
+	}
+	depth = gtk_spin_button_new_with_range (0.0, 1000.0, depthinterval);
+	gtk_box_pack_end(GTK_BOX(box), depth, FALSE, FALSE, 0);
+
+	/* All done, show it and wait for editing */
+	gtk_widget_show_all(dialog);
+	success = gtk_dialog_run(GTK_DIALOG(dialog)) == GTK_RESPONSE_ACCEPT;
+	if (!success) {
+		gtk_widget_destroy(dialog);
+		return 0;
+	}
+
+	memset(&tm, 0, sizeof(tm));
+	gtk_calendar_get_date(GTK_CALENDAR(cal), &yval, &mval, &dval);
+	tm.tm_year = yval;
+	tm.tm_mon = mval;
+	tm.tm_mday = dval;
+
+	tm.tm_hour = gtk_spin_button_get_value(GTK_SPIN_BUTTON(h));
+	tm.tm_min = gtk_spin_button_get_value(GTK_SPIN_BUTTON(m));
+
+	val = gtk_spin_button_get_value(GTK_SPIN_BUTTON(depth));
+	if (output_units.length == FEET) {
+		dive->maxdepth.mm = feet_to_mm(val);
+	} else {
+		dive->maxdepth.mm = val * 1000 + 0.5;
+	}
+
+	dive->duration.seconds = gtk_spin_button_get_value(GTK_SPIN_BUTTON(duration))*60;
+
+	gtk_widget_destroy(dialog);
+	dive->when = utc_mktime(&tm);
+
+	return 1;
+}
+
+int add_new_dive(struct dive *dive)
+{
+	if (!dive)
+		return 0;
+
+	if (!dive_time_widget(dive))
+		return 0;
+
+	return edit_dive_info(dive);
 }
 
 GtkWidget *extended_dive_info_widget(void)
@@ -320,18 +553,30 @@ GtkWidget *extended_dive_info_widget(void)
 	GtkWidget *vbox, *hbox;
 	vbox = gtk_vbox_new(FALSE, 6);
 
+	people_list = gtk_list_store_new(1, G_TYPE_STRING);
+	location_list = gtk_list_store_new(1, G_TYPE_STRING);
+	star_list = gtk_list_store_new(1, G_TYPE_STRING);
+	add_string_list_entry(ZERO_STARS, star_list);
+	add_string_list_entry(ONE_STARS, star_list);
+	add_string_list_entry(TWO_STARS, star_list);
+	add_string_list_entry(THREE_STARS, star_list);
+	add_string_list_entry(FOUR_STARS, star_list);
+	add_string_list_entry(FIVE_STARS, star_list);
+
 	gtk_container_set_border_width(GTK_CONTAINER(vbox), 6);
-	location = text_entry(vbox, "Location");
+	location = text_value(vbox, "Location");
 
 	hbox = gtk_hbox_new(FALSE, 3);
 	gtk_box_pack_start(GTK_BOX(vbox), hbox, FALSE, TRUE, 0);
 
-	divemaster = text_entry(hbox, "Divemaster");
-	buddy = text_entry(hbox, "Buddy");
+	divemaster = text_value(hbox, "Divemaster");
+	buddy = text_value(hbox, "Buddy");
+
+	hbox = gtk_hbox_new(FALSE, 3);
+	gtk_box_pack_start(GTK_BOX(vbox), hbox, FALSE, TRUE, 0);
 
-	notes = text_view(vbox, "Notes");
+	rating = text_value(hbox, "Rating");
 
-	/* Add extended info here: name, description, yadda yadda */
-	show_dive_info(current_dive);
+	notes = text_view(vbox, "Notes", READ_ONLY);
 	return vbox;
 }