]> git.street.me.uk Git - andy/viking.git/blobdiff - src/viktrwlayer_propwin.c
Fix internal tile locking
[andy/viking.git] / src / viktrwlayer_propwin.c
index 16fd65aa955d5db1f0363be24b91875ebeee6603..1ca135e4e85c3ef80f724c1f35c9a3b4e81efffd 100644 (file)
  *
  */
 
  *
  */
 
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#ifdef HAVE_MATH_H
+#include <math.h>
+#endif
+
 #include <gtk/gtk.h>
 #include <gtk/gtk.h>
+#include <glib/gi18n.h>
 #include <time.h>
 #include <time.h>
+#ifdef HAVE_STRING_H
 #include <string.h>
 #include <string.h>
+#endif
 #include "coords.h"
 #include "vikcoord.h"
 #include "viktrack.h"
 #include "coords.h"
 #include "vikcoord.h"
 #include "viktrack.h"
+#include "viktrwlayer.h"
 #include "viktrwlayer_propwin.h"
 #include "vikwaypoint.h"
 #include "dialog.h"
 #include "globals.h"
 #include "viktrwlayer_propwin.h"
 #include "vikwaypoint.h"
 #include "dialog.h"
 #include "globals.h"
+#include "dems.h"
+
+#include "vikviewport.h" /* ugh */
+#include "viktreeview.h" /* ugh */
+#include <gdk-pixbuf/gdk-pixdata.h>
+#include "viklayer.h" /* ugh */
+#include "vikaggregatelayer.h"
+#include "viklayerspanel.h" /* ugh */
 
 #define PROFILE_WIDTH 600
 #define PROFILE_HEIGHT 300
 
 #define PROFILE_WIDTH 600
 #define PROFILE_HEIGHT 300
+#define MIN_ALT_DIFF 100.0
+#define MIN_SPEED_DIFF 20.0
+
+typedef struct _propsaved {
+  gboolean saved;
+  gint pos;
+  GdkImage *img;
+} PropSaved;
+
+typedef struct _propwidgets {
+  VikTrwLayer *vtl;
+  VikTrack *tr;
+  VikLayersPanel *vlp;
+  gchar *track_name;
+  GtkWidget *dialog;
+  GtkWidget *w_comment;
+  GtkWidget *w_track_length;
+  GtkWidget *w_tp_count;
+  GtkWidget *w_segment_count;
+  GtkWidget *w_duptp_count;
+  GtkWidget *w_max_speed;
+  GtkWidget *w_avg_speed;
+  GtkWidget *w_avg_dist;
+  GtkWidget *w_elev_range;
+  GtkWidget *w_elev_gain;
+  GtkWidget *w_time_start;
+  GtkWidget *w_time_end;
+  GtkWidget *w_time_dur;
+  GtkWidget *w_cur_dist; /*< Current distance */
+  GtkWidget *w_cur_time; /*< Current time */
+  gdouble   track_length;
+  PropSaved elev_graph_saved_img;
+  PropSaved speed_graph_saved_img;
+  GtkWidget *elev_box;
+  GtkWidget *speed_box;
+  VikTrackpoint *marker_tp;
+} PropWidgets;
+
+static PropWidgets *prop_widgets_new()
+{
+  PropWidgets *widgets = g_malloc0(sizeof(PropWidgets));
+
+  return widgets;
+}
+
+static void prop_widgets_free(PropWidgets *widgets)
+{
+
+  if (widgets->elev_graph_saved_img.img)
+    g_object_unref(widgets->elev_graph_saved_img.img);
+  if (widgets->speed_graph_saved_img.img)
+    g_object_unref(widgets->speed_graph_saved_img.img);
+  g_free(widgets);
+}
 
 static void minmax_alt(const gdouble *altitudes, gdouble *min, gdouble *max)
 {
 
 static void minmax_alt(const gdouble *altitudes, gdouble *min, gdouble *max)
 {
@@ -46,148 +120,884 @@ static void minmax_alt(const gdouble *altitudes, gdouble *min, gdouble *max)
         *min = altitudes[i];
     }
   }
         *min = altitudes[i];
     }
   }
+}
 
 
+#define MARGIN 70
+#define LINES 5
+static VikTrackpoint *set_center_at_graph_position(gdouble event_x, gint img_width, VikLayersPanel *vlp, VikTrack *tr, gboolean time_base)
+{
+  VikTrackpoint *trackpoint;
+  gdouble x = event_x - img_width / 2 + PROFILE_WIDTH / 2 - MARGIN / 2;
+  if (x < 0)
+    x = 0;
+  if (x > PROFILE_WIDTH)
+    x = PROFILE_WIDTH;
+
+  if (time_base)
+    trackpoint = vik_track_get_closest_tp_by_percentage_time ( tr, (gdouble) x / PROFILE_WIDTH, NULL );
+  else
+    trackpoint = vik_track_get_closest_tp_by_percentage_dist ( tr, (gdouble) x / PROFILE_WIDTH, NULL );
+
+  if ( trackpoint ) {
+    VikCoord coord = trackpoint->coord;
+    vik_viewport_set_center_coord ( vik_layers_panel_get_viewport(vlp), &coord );
+    vik_layers_panel_emit_update ( vlp );
+  }
+  return trackpoint;
 }
 
 }
 
-GtkWidget *vik_trw_layer_create_profile ( GtkWidget *window, VikTrack *tr, gdouble *min_alt, gdouble *max_alt )
+static void draw_graph_mark(GtkWidget *image, gdouble event_x, gint img_width, GdkGC *gc, PropSaved *saved_img)
 {
 {
-  GdkPixmap *pix = gdk_pixmap_new( window->window, PROFILE_WIDTH, PROFILE_HEIGHT, -1 );
-  GtkWidget *image = gtk_image_new_from_pixmap ( pix, NULL );
-  gdouble *altitudes = vik_track_make_elevation_map ( tr, PROFILE_WIDTH );
+  GdkPixmap *pix = NULL;
+  /* the pixmap = margin + graph area */
+  gdouble x = event_x - img_width/2 + PROFILE_WIDTH/2 + MARGIN/2;
+
+  // fprintf(stderr, "event_x=%f img_width=%d x=%f\n", event_x, img_width, x);
+
+  gtk_image_get_pixmap(GTK_IMAGE(image), &pix, NULL);
+       /* Restore previously saved image */
+  if (saved_img->saved) {
+    gdk_draw_image(GDK_DRAWABLE(pix), gc, saved_img->img, 0, 0,
+        saved_img->pos, 0, -1, -1);
+    saved_img->saved = FALSE;
+    gtk_widget_queue_draw_area(image,
+        saved_img->pos + img_width/2 - PROFILE_WIDTH/2 - MARGIN/2, 0,
+        saved_img->img->width, saved_img->img->height);
+  }
+  if ((x >= MARGIN) && (x < (PROFILE_WIDTH + MARGIN))) {
+               /* Save part of the image */
+    if (saved_img->img)
+      gdk_drawable_copy_to_image(GDK_DRAWABLE(pix), saved_img->img,
+          x, 0, 0, 0, saved_img->img->width, saved_img->img->height);
+    else
+      saved_img->img = gdk_drawable_copy_to_image(GDK_DRAWABLE(pix),
+          saved_img->img, x, 0, 0, 0, 1, PROFILE_HEIGHT);
+    saved_img->pos = x;
+    saved_img->saved = TRUE;
+    gdk_draw_line (GDK_DRAWABLE(pix), gc, x, 0, x, image->allocation.height);
+    /* redraw the area which contains the line, saved_width is just convenient */
+    gtk_widget_queue_draw_area(image, event_x, 0, 1, PROFILE_HEIGHT);
+  }
+}
+
+static void track_graph_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along, gboolean is_vt_graph )
+{
+  VikTrack *tr = pass_along[0];
+  VikLayersPanel *vlp = pass_along[1];
+  PropWidgets *widgets = pass_along[2];
+  GList *child = gtk_container_get_children(GTK_CONTAINER(event_box));
+  GtkWidget *image = GTK_WIDGET(child->data);
+  GtkWidget *window = gtk_widget_get_toplevel(GTK_WIDGET(event_box));
+
+
+  VikTrackpoint *trackpoint = set_center_at_graph_position(event->x, event_box->allocation.width, vlp, tr, is_vt_graph);
+  draw_graph_mark(image, event->x, event_box->allocation.width, window->style->black_gc,
+      is_vt_graph ? &widgets->speed_graph_saved_img : &widgets->elev_graph_saved_img);
+  g_list_free(child);
+  widgets->marker_tp = trackpoint;
+  gtk_dialog_set_response_sensitive(GTK_DIALOG(widgets->dialog), VIK_TRW_LAYER_PROPWIN_SPLIT_MARKER, TRUE);
+
+  /* draw on the other graph */
+  if (trackpoint == NULL || widgets->elev_box == NULL || widgets->speed_box == NULL)
+    /* This test assumes we have only 2 graphs */
+    return;
+
+  gdouble pc = NAN;
+  gdouble x2;
+  GList *other_child = gtk_container_get_children(GTK_CONTAINER(
+                         is_vt_graph ? widgets->elev_box : widgets->speed_box));
+  GtkWidget *other_image = GTK_WIDGET(other_child->data);
+  if (is_vt_graph) {
+    gdouble dist = 0.0;
+    GList *iter;
+    for (iter = tr->trackpoints->next; iter != NULL; iter = iter->next) {
+      dist += vik_coord_diff(&(VIK_TRACKPOINT(iter->data)->coord),
+                                 &(VIK_TRACKPOINT(iter->prev->data)->coord));
+      /* Assuming trackpoint is not a copy */
+      if (trackpoint == VIK_TRACKPOINT(iter->data))
+        break;
+    }
+    if (iter != NULL)
+      pc = dist/widgets->track_length;
+  } else {
+    time_t t_start, t_end, t_total;
+    t_start = VIK_TRACKPOINT(tr->trackpoints->data)->timestamp;
+    t_end = VIK_TRACKPOINT(g_list_last(tr->trackpoints)->data)->timestamp;
+    t_total = t_end - t_start;
+    pc = (gdouble)(trackpoint->timestamp - t_start)/t_total;
+  }
+  if (!isnan(pc)) {
+    x2 = pc * PROFILE_WIDTH + MARGIN + (event_box->allocation.width/2 - PROFILE_WIDTH/2 - MARGIN/2);
+    draw_graph_mark(other_image, x2, event_box->allocation.width, window->style->black_gc,
+      is_vt_graph ? &widgets->elev_graph_saved_img : &widgets->speed_graph_saved_img);
+  }
+
+  g_list_free(other_child);
+
+}
 
 
+static gboolean track_profile_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along )
+{
+  track_graph_click(event_box, event, pass_along, FALSE);
+  return TRUE;  /* don't call other (further) callbacks */
+}
+
+static gboolean track_vt_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along )
+{
+  track_graph_click(event_box, event, pass_along, TRUE);
+  return TRUE;  /* don't call other (further) callbacks */
+}
+
+void track_profile_move( GtkWidget *image, GdkEventMotion *event, gpointer *pass_along )
+{
+  VikTrack *tr = pass_along[0];
+  PropWidgets *widgets = pass_along[2];
+  int mouse_x, mouse_y;
+  GdkModifierType state;
+
+  if (event->is_hint)
+    gdk_window_get_pointer (event->window, &mouse_x, &mouse_y, &state);
+  else
+    mouse_x = event->x;
+
+  gdouble x = mouse_x - image->allocation.width / 2 + PROFILE_WIDTH / 2 - MARGIN / 2;
+  if (x < 0)
+    x = 0;
+  if (x > PROFILE_WIDTH)
+    x = PROFILE_WIDTH;
+
+  gdouble meters_from_start;
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_dist ( tr, (gdouble) x / PROFILE_WIDTH, &meters_from_start );
+  if (trackpoint && widgets->w_cur_dist) {
+    static gchar tmp_buf[20];
+    vik_units_distance_t dist_units = a_vik_get_units_distance ();
+    switch (dist_units) {
+    case VIK_UNITS_DISTANCE_KILOMETRES:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f km", meters_from_start/1000.0);
+      break;
+    case VIK_UNITS_DISTANCE_MILES:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f miles", meters_from_start/1600.0);
+      break;
+    default:
+      g_critical("Houston, we've had a problem. distance=%d", dist_units);
+    }
+    gtk_label_set_text(GTK_LABEL(widgets->w_cur_dist), tmp_buf);
+  }
+}
+
+void track_vt_move( GtkWidget *image, GdkEventMotion *event, gpointer *pass_along )
+{
+  VikTrack *tr = pass_along[0];
+  PropWidgets *widgets = pass_along[2];
+  int mouse_x, mouse_y;
+  GdkModifierType state;
+
+  if (event->is_hint)
+    gdk_window_get_pointer (event->window, &mouse_x, &mouse_y, &state);
+  else
+    mouse_x = event->x;
+
+  gdouble x = mouse_x - image->allocation.width / 2 + PROFILE_WIDTH / 2 - MARGIN / 2;
+  if (x < 0)
+    x = 0;
+  if (x > PROFILE_WIDTH)
+    x = PROFILE_WIDTH;
+
+  time_t seconds_from_start;
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_time ( tr, (gdouble) x / PROFILE_WIDTH, &seconds_from_start );
+  if (trackpoint && widgets->w_cur_time) {
+    static gchar tmp_buf[20];
+    guint h, m, s;
+    h = seconds_from_start/3600;
+    m = (seconds_from_start - h*3600)/60;
+    s = seconds_from_start - (3600*h) - (60*m);
+    g_snprintf(tmp_buf, sizeof(tmp_buf), "%02d:%02d:%02d", h, m, s);
+
+    gtk_label_set_text(GTK_LABEL(widgets->w_cur_time), tmp_buf);
+  }
+}
+
+static void draw_dem_alt_speed_dist(VikTrack *tr, GdkDrawable *pix, GdkGC *alt_gc, GdkGC *speed_gc, gdouble alt_offset, gdouble alt_diff, gint width, gint height, gint margin)
+{
+  GList *iter;
+  gdouble dist = 0;
+  gdouble max_speed = 0;
+  gdouble total_length = vik_track_get_length_including_gaps(tr);
+
+  for (iter = tr->trackpoints->next; iter; iter = iter->next) {
+    if (!isnan(VIK_TRACKPOINT(iter->data)->speed))
+      max_speed = MAX(max_speed, VIK_TRACKPOINT(iter->data)->speed);
+  }
+  max_speed = max_speed * 110 / 100;
+
+  for (iter = tr->trackpoints->next; iter; iter = iter->next) {
+    int x, y_alt, y_speed;
+    gint16 elev = a_dems_get_elev_by_coord(&(VIK_TRACKPOINT(iter->data)->coord), VIK_DEM_INTERPOL_BEST);
+    elev -= alt_offset;
+    dist += vik_coord_diff ( &(VIK_TRACKPOINT(iter->data)->coord),
+      &(VIK_TRACKPOINT(iter->prev->data)->coord) );
+    x = (width * dist)/total_length + margin;
+    if ( elev != VIK_DEM_INVALID_ELEVATION ) {
+      y_alt = height - ((height * elev)/alt_diff);
+      gdk_draw_rectangle(GDK_DRAWABLE(pix), alt_gc, TRUE, x-2, y_alt-2, 4, 4);
+    }
+    if (!isnan(VIK_TRACKPOINT(iter->data)->speed)) {
+      y_speed = height - (height * VIK_TRACKPOINT(iter->data)->speed)/max_speed;
+      gdk_draw_rectangle(GDK_DRAWABLE(pix), speed_gc, TRUE, x-2, y_speed-2, 4, 4);
+    }
+  }
+}
+
+GtkWidget *vik_trw_layer_create_profile ( GtkWidget *window, VikTrack *tr, gpointer vlp, PropWidgets *widgets, gdouble *min_alt, gdouble *max_alt)
+{
+  GdkPixmap *pix;
+  GtkWidget *image;
+  gdouble *altitudes = vik_track_make_elevation_map ( tr, PROFILE_WIDTH );
+  gdouble mina, maxa;
+  GtkWidget *eventbox;
+  gpointer *pass_along;
   guint i;
 
   guint i;
 
+  if ( altitudes == NULL ) {
+    *min_alt = *max_alt = VIK_DEFAULT_ALTITUDE;
+    return NULL;
+  }
+
+  pix = gdk_pixmap_new( window->window, PROFILE_WIDTH + MARGIN, PROFILE_HEIGHT, -1 );
+  image = gtk_image_new_from_pixmap ( pix, NULL );
+
   GdkGC *no_alt_info = gdk_gc_new ( window->window );
   GdkGC *no_alt_info = gdk_gc_new ( window->window );
+  GdkGC *dem_alt_gc = gdk_gc_new ( window->window );
+  GdkGC *gps_speed_gc = gdk_gc_new ( window->window );
   GdkColor color;
 
   GdkColor color;
 
-  gdk_color_parse ( "red", &color );
+  gdk_color_parse ( "yellow", &color );
   gdk_gc_set_rgb_fg_color ( no_alt_info, &color);
   gdk_gc_set_rgb_fg_color ( no_alt_info, &color);
+  gdk_color_parse ( "green", &color );
+  gdk_gc_set_rgb_fg_color ( dem_alt_gc, &color);
+  gdk_color_parse ( "red", &color );
+  gdk_gc_set_rgb_fg_color ( gps_speed_gc, &color);
+
 
   minmax_alt(altitudes, min_alt, max_alt);
 
   minmax_alt(altitudes, min_alt, max_alt);
+  mina = *min_alt;
+  maxa = *max_alt + ((*max_alt - *min_alt) * 0.25);
   
   
+  /* clear the image */
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->bg_gc[0], 
+                    TRUE, 0, 0, MARGIN, PROFILE_HEIGHT);
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->mid_gc[0], 
+                    TRUE, MARGIN, 0, PROFILE_WIDTH, PROFILE_HEIGHT);
+
+  /* draw grid */
+#define LABEL_FONT "Sans 7"
+  vik_units_height_t height_units = a_vik_get_units_height ();
+  for (i=0; i<=LINES; i++) {
+    PangoFontDescription *pfd;
+    PangoLayout *pl = gtk_widget_create_pango_layout (GTK_WIDGET(image), NULL);
+    gchar s[32];
+    int w, h;
+
+    pango_layout_set_alignment (pl, PANGO_ALIGN_RIGHT);
+    pfd = pango_font_description_from_string (LABEL_FONT);
+    pango_layout_set_font_description (pl, pfd);
+    pango_font_description_free (pfd);
+    switch (height_units) {
+    case VIK_UNITS_HEIGHT_METRES:
+      sprintf(s, "%8dm", (int)(mina + (LINES-i)*(maxa-mina)/LINES));
+      break;
+    case VIK_UNITS_HEIGHT_FEET:
+      sprintf(s, "%8dft", (int)((mina + (LINES-i)*(maxa-mina)/LINES)*3.2808399));
+      break;
+    default:
+      sprintf(s, "--");
+      g_critical("Houston, we've had a problem. height=%d", height_units);
+    }
+    pango_layout_set_text(pl, s, -1);
+    pango_layout_get_pixel_size (pl, &w, &h);
+    gdk_draw_layout(GDK_DRAWABLE(pix), window->style->fg_gc[0], MARGIN-w-3, 
+                   CLAMP((int)i*PROFILE_HEIGHT/LINES - h/2, 0, PROFILE_HEIGHT-h), pl);
+
+    gdk_draw_line (GDK_DRAWABLE(pix), window->style->dark_gc[0], 
+                  MARGIN, PROFILE_HEIGHT/LINES * i, MARGIN + PROFILE_WIDTH, PROFILE_HEIGHT/LINES * i);
+    g_object_unref ( G_OBJECT ( pl ) );
+    pl = NULL;
+  }
+
+  /* draw elevations */
   for ( i = 0; i < PROFILE_WIDTH; i++ )
   for ( i = 0; i < PROFILE_WIDTH; i++ )
-    if ( altitudes[i] == VIK_DEFAULT_ALTITUDE )
-      gdk_draw_line ( GDK_DRAWABLE(pix), no_alt_info, i, 0, i, PROFILE_HEIGHT );
-    else
-      gdk_draw_line ( GDK_DRAWABLE(pix), window->style->white_gc, i, 0, i, PROFILE_HEIGHT-PROFILE_HEIGHT*(altitudes[i]-*min_alt)/(*max_alt-*min_alt) );
+    if ( altitudes[i] == VIK_DEFAULT_ALTITUDE ) 
+      gdk_draw_line ( GDK_DRAWABLE(pix), no_alt_info, 
+                     i + MARGIN, 0, i + MARGIN, PROFILE_HEIGHT );
+    else 
+      gdk_draw_line ( GDK_DRAWABLE(pix), window->style->dark_gc[3], 
+                     i + MARGIN, PROFILE_HEIGHT, i + MARGIN, PROFILE_HEIGHT-PROFILE_HEIGHT*(altitudes[i]-mina)/(maxa-mina) );
+
+  draw_dem_alt_speed_dist(tr, GDK_DRAWABLE(pix), dem_alt_gc, gps_speed_gc, mina, maxa - mina, PROFILE_WIDTH, PROFILE_HEIGHT, MARGIN);
+
+  /* draw border */
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->black_gc, FALSE, MARGIN, 0, PROFILE_WIDTH-1, PROFILE_HEIGHT-1);
+
+
 
   g_object_unref ( G_OBJECT(pix) );
   g_free ( altitudes );
   g_object_unref ( G_OBJECT(no_alt_info) );
 
   g_object_unref ( G_OBJECT(pix) );
   g_free ( altitudes );
   g_object_unref ( G_OBJECT(no_alt_info) );
-  return image;
-}
-
-gint vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrack *tr )
-{
-  GtkWidget *dialog = gtk_dialog_new_with_buttons ("Track Properties",
-                                                  parent,
-                                                  GTK_DIALOG_MODAL | GTK_DIALOG_DESTROY_WITH_PARENT,
-                                                  GTK_STOCK_CANCEL,
-                                                  GTK_RESPONSE_REJECT,
-                                                  "Split Segments",
-                                                  VIK_TRW_LAYER_PROPWIN_SPLIT,
-                                                  "Reverse",
-                                                  VIK_TRW_LAYER_PROPWIN_REVERSE,
-                                                  "Delete Dupl.",
-                                                  VIK_TRW_LAYER_PROPWIN_DEL_DUP,
-                                                  GTK_STOCK_OK,
-                                                  GTK_RESPONSE_ACCEPT,
-                                                  NULL);
-  GtkWidget *left_vbox, *right_vbox, *hbox;
-  GtkWidget *e_cmt;
-  GtkWidget *l_len, *l_tps, *l_segs, *l_dups, *l_maxs, *l_avgs, *l_avgd, *l_elev, *l_galo, *l_begin, *l_end, *l_dur;
+  g_object_unref ( G_OBJECT(dem_alt_gc) );
+  g_object_unref ( G_OBJECT(gps_speed_gc) );
+
+  pass_along = g_malloc ( sizeof(gpointer) * 3 ); /* FIXME: mem leak -- never be freed */
+  pass_along[0] = tr;
+  pass_along[1] = vlp;
+  pass_along[2] = widgets;
+
+  eventbox = gtk_event_box_new ();
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_profile_click), pass_along );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_profile_move), pass_along );
+  g_signal_connect_swapped ( G_OBJECT(eventbox), "destroy", G_CALLBACK(g_free), pass_along );
+  gtk_container_add ( GTK_CONTAINER(eventbox), image );
+  gtk_widget_set_events (eventbox, GDK_BUTTON_PRESS_MASK
+                         | GDK_POINTER_MOTION_MASK
+                         | GDK_POINTER_MOTION_HINT_MASK);
+
+  return eventbox;
+}
+
+
+GtkWidget *vik_trw_layer_create_vtdiag ( GtkWidget *window, VikTrack *tr, gpointer vlp, PropWidgets *widgets)
+{
+  GdkPixmap *pix;
+  GtkWidget *image;
+  gdouble mins, maxs;
+  guint i;
+  GtkWidget *eventbox;
+  gpointer *pass_along;
+
+  pass_along = g_malloc ( sizeof(gpointer) * 3 ); /* FIXME: mem leak -- never be freed */
+  pass_along[0] = tr;
+  pass_along[1] = vlp;
+  pass_along[2] = widgets;
+
+  gdouble *speeds = vik_track_make_speed_map ( tr, PROFILE_WIDTH );
+  if ( speeds == NULL ) {
+    g_free(pass_along);
+    return NULL;
+  }
+
+  pix = gdk_pixmap_new( window->window, PROFILE_WIDTH + MARGIN, PROFILE_HEIGHT, -1 );
+  image = gtk_image_new_from_pixmap ( pix, NULL );
+
+  minmax_alt(speeds, &mins, &maxs);
+  if (mins < 0.0)
+    mins = 0; /* splines sometimes give negative speeds */
+  maxs = maxs + ((maxs - mins) * 0.1);
+  if  (maxs-mins < MIN_SPEED_DIFF) {
+    maxs = mins + MIN_SPEED_DIFF;
+  }
+  
+  /* clear the image */
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->bg_gc[0], 
+                    TRUE, 0, 0, MARGIN, PROFILE_HEIGHT);
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->mid_gc[0], 
+                    TRUE, MARGIN, 0, PROFILE_WIDTH, PROFILE_HEIGHT);
+
+#if 0
+  /* XXX this can go out, it's just a helpful dev tool */
+  {
+    int j;
+    GdkGC **colors[8] = { window->style->bg_gc, window->style->fg_gc, 
+                        window->style->light_gc, 
+                        window->style->dark_gc, window->style->mid_gc, 
+                        window->style->text_gc, window->style->base_gc,
+                        window->style->text_aa_gc };
+    for (i=0; i<5; i++) {
+      for (j=0; j<8; j++) {
+       gdk_draw_rectangle(GDK_DRAWABLE(pix), colors[j][i],
+                          TRUE, i*20, j*20, 20, 20);
+       gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->black_gc,
+                          FALSE, i*20, j*20, 20, 20);
+      }
+    }
+  }
+#else
+
+  /* draw grid */
+#define LABEL_FONT "Sans 7"
+  for (i=0; i<=LINES; i++) {
+    PangoFontDescription *pfd;
+    PangoLayout *pl = gtk_widget_create_pango_layout (GTK_WIDGET(image), NULL);
+    gchar s[32];
+    int w, h;
+
+    pango_layout_set_alignment (pl, PANGO_ALIGN_RIGHT);
+    pfd = pango_font_description_from_string (LABEL_FONT);
+    pango_layout_set_font_description (pl, pfd);
+    pango_font_description_free (pfd);
+    vik_units_speed_t speed_units = a_vik_get_units_speed ();
+    switch (speed_units) {
+    case VIK_UNITS_SPEED_KILOMETRES_PER_HOUR:
+      sprintf(s, "%8dkm/h", (int)((mins + (LINES-i)*(maxs-mins)/LINES)*3.6));
+      break;
+    case VIK_UNITS_SPEED_MILES_PER_HOUR:
+      sprintf(s, "%8dmph", (int)((mins + (LINES-i)*(maxs-mins)/LINES)*2.23693629));
+      break;
+    case VIK_UNITS_SPEED_METRES_PER_SECOND:
+      sprintf(s, "%8dm/s", (int)(mins + (LINES-i)*(maxs-mins)/LINES));
+      break;
+    case VIK_UNITS_SPEED_KNOTS:
+      sprintf(s, "%8dknots", (int)((mins + (LINES-i)*(maxs-mins)/LINES)*1.94384449));
+      break;
+    default:
+      sprintf(s, "--");
+      g_critical("Houston, we've had a problem. speed=%d", speed_units);
+    }
+
+    pango_layout_set_text(pl, s, -1);
+    pango_layout_get_pixel_size (pl, &w, &h);
+    gdk_draw_layout(GDK_DRAWABLE(pix), window->style->fg_gc[0], MARGIN-w-3, 
+                   CLAMP((int)i*PROFILE_HEIGHT/LINES - h/2, 0, PROFILE_HEIGHT-h), pl);
+
+    gdk_draw_line (GDK_DRAWABLE(pix), window->style->dark_gc[0], 
+                  MARGIN, PROFILE_HEIGHT/LINES * i, MARGIN + PROFILE_WIDTH, PROFILE_HEIGHT/LINES * i);
+    g_object_unref ( G_OBJECT ( pl ) );
+    pl = NULL;
+  }
+  
+
+  /* draw speeds */
+  for ( i = 0; i < PROFILE_WIDTH; i++ )
+      gdk_draw_line ( GDK_DRAWABLE(pix), window->style->dark_gc[3], 
+                     i + MARGIN, PROFILE_HEIGHT, i + MARGIN, PROFILE_HEIGHT-PROFILE_HEIGHT*(speeds[i]-mins)/(maxs-mins) );
+#endif
+
+
+  GdkGC *gps_speed_gc = gdk_gc_new ( window->window );
+  GdkColor color;
+
+  gdk_color_parse ( "red", &color );
+  gdk_gc_set_rgb_fg_color ( gps_speed_gc, &color);
+
+  time_t beg_time = VIK_TRACKPOINT(tr->trackpoints->data)->timestamp;
+  time_t dur =  VIK_TRACKPOINT(g_list_last(tr->trackpoints)->data)->timestamp - beg_time;
+  GList *iter;
+  for (iter = tr->trackpoints; iter; iter = iter->next) {
+    gdouble gps_speed = VIK_TRACKPOINT(iter->data)->speed;
+    if (isnan(gps_speed))
+        continue;
+    int x = MARGIN + PROFILE_WIDTH * (VIK_TRACKPOINT(iter->data)->timestamp - beg_time) / dur;
+    int y = PROFILE_HEIGHT - PROFILE_HEIGHT*(gps_speed - mins)/(maxs - mins);
+    gdk_draw_rectangle(GDK_DRAWABLE(pix), gps_speed_gc, TRUE, x-2, y-2, 4, 4);
+  }
+
+  /* draw border */
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->black_gc, FALSE, MARGIN, 0, PROFILE_WIDTH-1, PROFILE_HEIGHT-1);
+
+  g_object_unref ( G_OBJECT(pix) );
+  g_object_unref ( G_OBJECT(gps_speed_gc) );
+  g_free ( speeds );
+
+  eventbox = gtk_event_box_new ();
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_vt_click), pass_along );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_vt_move), pass_along );
+  g_signal_connect_swapped ( G_OBJECT(eventbox), "destroy", G_CALLBACK(g_free), pass_along );
+  gtk_container_add ( GTK_CONTAINER(eventbox), image );
+  gtk_widget_set_events (eventbox, GDK_BUTTON_PRESS_MASK
+                         | GDK_POINTER_MOTION_MASK
+                         | GDK_POINTER_MOTION_HINT_MASK);
+
+  return eventbox;
+}
+#undef MARGIN
+#undef LINES
+
+static void propwin_response_cb( GtkDialog *dialog, gint resp, PropWidgets *widgets)
+{
+  VikTrack *tr = widgets->tr;
+  VikTrwLayer *vtl = widgets->vtl;
+  gboolean keep_dialog = FALSE;
+
+  /* FIXME: check and make sure the track still exists before doing anything to it */
+  /* Note: destroying diaglog (eg, parent window exit) won't give "response" */
+  switch (resp) {
+    case GTK_RESPONSE_DELETE_EVENT: /* received delete event (not from buttons) */
+    case GTK_RESPONSE_REJECT:
+      break;
+    case GTK_RESPONSE_ACCEPT:
+      vik_track_set_comment(tr, gtk_entry_get_text(GTK_ENTRY(widgets->w_comment)));
+      break;
+    case VIK_TRW_LAYER_PROPWIN_REVERSE:
+      vik_track_reverse(tr);
+      vik_layer_emit_update ( VIK_LAYER(vtl) );
+      break;
+    case VIK_TRW_LAYER_PROPWIN_DEL_DUP:
+      vik_track_remove_dup_points(tr);
+      /* above operation could have deleted current_tp or last_tp */
+      trw_layer_cancel_tps_of_track ( vtl, widgets->track_name );
+      vik_layer_emit_update ( VIK_LAYER(vtl) );
+      break;
+    case VIK_TRW_LAYER_PROPWIN_SPLIT:
+      {
+        /* get new tracks, add them, resolve naming conflicts (free if cancel), and delete old. old can still exist on clipboard. */
+        guint ntracks;
+        VikTrack **tracks = vik_track_split_into_segments(tr, &ntracks);
+        gchar *new_tr_name;
+        guint i;
+        for ( i = 0; i < ntracks; i++ )
+        {
+          g_assert ( tracks[i] );
+          new_tr_name = g_strdup_printf("%s #%d", widgets->track_name, i+1);
+          /* if ( (wp_exists) && (! overwrite) ) */
+          /* don't need to upper case new_tr_name because old tr name was uppercase */
+          if ( vik_trw_layer_get_track(vtl, new_tr_name ) && 
+             ( ! a_dialog_overwrite ( VIK_GTK_WINDOW_FROM_LAYER(vtl), "The track \"%s\" exists, do you wish to overwrite it?", new_tr_name ) ) )
+          {
+            gchar *new_new_tr_name = a_dialog_new_track ( VIK_GTK_WINDOW_FROM_LAYER(vtl), vik_trw_layer_get_tracks(vtl) );
+            g_free ( new_tr_name );
+            if (new_new_tr_name)
+              new_tr_name = new_new_tr_name;
+            else
+            {
+              new_tr_name = NULL;
+              vik_track_free ( tracks[i] );
+            }
+          }
+          if ( new_tr_name )
+            vik_trw_layer_add_track ( vtl, new_tr_name, tracks[i] );
+        }
+        if ( tracks )
+        {
+          g_free ( tracks );
+          /* Don't let track destroy this dialog */
+          vik_track_clear_property_dialog(tr);
+          vik_trw_layer_delete_track ( vtl, widgets->track_name );
+          vik_layer_emit_update ( VIK_LAYER(vtl) ); /* chase thru the hoops */
+        }
+      }
+      break;
+    case VIK_TRW_LAYER_PROPWIN_SPLIT_MARKER:
+      {
+        GList *iter = tr->trackpoints;
+        while ((iter = iter->next)) {
+          if (widgets->marker_tp == VIK_TRACKPOINT(iter->data))
+            break;
+        }
+        if (iter == NULL) {
+          a_dialog_msg(VIK_GTK_WINDOW_FROM_LAYER(vtl), GTK_MESSAGE_ERROR,
+                  _("Failed spliting track. Track unchanged"), NULL);
+          keep_dialog = TRUE;
+          break;
+        }
+
+        gchar *r_name = g_strdup_printf("%s #R", widgets->track_name);
+        if (vik_trw_layer_get_track(vtl, r_name ) && 
+             ( ! a_dialog_overwrite( VIK_GTK_WINDOW_FROM_LAYER(vtl),
+              "The track \"%s\" exists, do you wish to overwrite it?", r_name)))
+        {
+            gchar *new_r_name = a_dialog_new_track( VIK_GTK_WINDOW_FROM_LAYER(vtl), vik_trw_layer_get_tracks(vtl) );
+            if (new_r_name) {
+              g_free( r_name );
+              r_name = new_r_name;
+            }
+            else {
+              a_dialog_msg(VIK_GTK_WINDOW_FROM_LAYER(vtl), GTK_MESSAGE_WARNING,
+                  _("Operation Aborted. Track unchanged"), NULL);
+              keep_dialog = TRUE;
+              break;
+            }
+        }
+        iter->prev->next = NULL;
+        iter->prev = NULL;
+        VikTrack *tr_right = vik_track_new();
+        if ( tr->comment )
+          vik_track_set_comment ( tr_right, tr->comment );
+        tr_right->visible = tr->visible;
+        tr_right->trackpoints = iter;
+
+        vik_trw_layer_add_track(vtl, r_name, tr_right);
+        vik_layer_emit_update ( VIK_LAYER(vtl) );
+      }
+      break;
+    default:
+      fprintf(stderr, "DEBUG: unknown response\n");
+      return;
+  }
+
+  /* Keep same behaviour for now: destroy dialog if click on any button */
+  if (!keep_dialog) {
+    prop_widgets_free(widgets);
+    vik_track_clear_property_dialog(tr);
+    gtk_widget_destroy ( GTK_WIDGET(dialog) );
+  }
+}
+
+static GtkWidget *create_graph_page ( GtkWidget *graph,
+                                     const gchar *markup,
+                                     GtkWidget *value)
+{
+  GtkWidget *hbox = gtk_hbox_new ( FALSE, 10 );
+  GtkWidget *vbox = gtk_vbox_new ( FALSE, 10 );
+  GtkWidget *label = gtk_label_new (NULL);
+  gtk_box_pack_start (GTK_BOX(vbox), graph, FALSE, FALSE, 0);
+  gtk_label_set_markup ( GTK_LABEL(label), markup );
+  gtk_box_pack_start (GTK_BOX(hbox), label, FALSE, FALSE, 0);
+  gtk_box_pack_start (GTK_BOX(hbox), value, FALSE, FALSE, 0);
+  gtk_box_pack_start (GTK_BOX(vbox), hbox, FALSE, FALSE, 0);
+  
+  return vbox;
+}
+
+void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *tr, gpointer vlp, gchar *track_name )
+{
+  /* FIXME: free widgets when destroy signal received */
+  PropWidgets *widgets = prop_widgets_new();
+  widgets->vtl = vtl;
+  widgets->tr = tr;
+  widgets->vlp = vlp;
+  widgets->track_name = track_name;
+  gchar *title = g_strdup_printf(_("%s - Track Properties"), track_name);
+  GtkWidget *dialog = gtk_dialog_new_with_buttons (title,
+                         parent,
+                         GTK_DIALOG_DESTROY_WITH_PARENT | GTK_DIALOG_NO_SEPARATOR,
+                         GTK_STOCK_CANCEL, GTK_RESPONSE_REJECT,
+                         _("Split at Marker"), VIK_TRW_LAYER_PROPWIN_SPLIT_MARKER,
+                         _("Split Segments"), VIK_TRW_LAYER_PROPWIN_SPLIT,
+                         _("Reverse"),        VIK_TRW_LAYER_PROPWIN_REVERSE,
+                         _("Delete Dupl."),   VIK_TRW_LAYER_PROPWIN_DEL_DUP,
+                         GTK_STOCK_OK,     GTK_RESPONSE_ACCEPT,
+                         NULL);
+  widgets->dialog = dialog;
+  g_free(title);
+  g_signal_connect(dialog, "response", G_CALLBACK(propwin_response_cb), widgets);
+  //fprintf(stderr, "DEBUG: dialog=0x%p\n", dialog);
+  GtkTable *table;
   gdouble tr_len;
   guint32 tp_count, seg_count;
   gdouble tr_len;
   guint32 tp_count, seg_count;
-  gint resp;
 
   gdouble min_alt, max_alt;
 
   gdouble min_alt, max_alt;
-  GtkWidget *profile = vik_trw_layer_create_profile(GTK_WIDGET(parent),tr,&min_alt,&max_alt);
+  GtkWidget *profile = vik_trw_layer_create_profile(GTK_WIDGET(parent),tr, vlp, widgets, &min_alt,&max_alt);
+  GtkWidget *vtdiag = vik_trw_layer_create_vtdiag(GTK_WIDGET(parent), tr, vlp, widgets);
+  GtkWidget *graphs = gtk_notebook_new();
 
 
-  static gchar *label_texts[] = { "<b>Comment:</b>", "<b>Track Length:</b>", "<b>Trackpoints:</b>", "<b>Segments:</b>", "<b>Duplicate Points:</b>", "<b>Max Speed:</b>", "<b>Avg. Speed:</b>", "<b>Avg. Dist. Between TPs:</b>", "<b>Elevation Range:</b>", "<b>Total Elevation Gain/Loss:</b>", "<b>Start:</b>",  "<b>End:</b>",  "<b>Duration:</b>" };
-  static gchar tmp_buf[20];
+  widgets->elev_box = profile;
+  widgets->speed_box = vtdiag;
 
 
-  left_vbox = a_dialog_create_label_vbox ( label_texts, sizeof(label_texts) / sizeof(label_texts[0]) );
-  right_vbox = gtk_vbox_new ( TRUE, 3 );
+  GtkWidget *content[20];
+  int cnt;
+  int i;
 
 
-  e_cmt = gtk_entry_new ();
-  if ( tr->comment )
-    gtk_entry_set_text ( GTK_ENTRY(e_cmt), tr->comment );
-  g_signal_connect_swapped ( e_cmt, "activate", G_CALLBACK(a_dialog_response_accept), GTK_DIALOG(dialog) );
+  static gchar *label_texts[] = { N_("<b>Comment:</b>"), N_("<b>Track Length:</b>"), N_("<b>Trackpoints:</b>"), N_("<b>Segments:</b>"), N_("<b>Duplicate Points:</b>"), N_("<b>Max Speed:</b>"), N_("<b>Avg. Speed:</b>"), N_("<b>Avg. Dist. Between TPs:</b>"), N_("<b>Elevation Range:</b>"), N_("<b>Total Elevation Gain/Loss:</b>"), N_("<b>Start:</b>"), N_("<b>End:</b>"), N_("<b>Duration:</b>") };
+  static gchar tmp_buf[50];
+  gdouble tmp_speed;
 
 
-  tr_len = vik_track_get_length(tr);
-  g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f m", tr_len );
-  l_len = gtk_label_new ( tmp_buf );
+  cnt = 0;
+  widgets->w_comment = gtk_entry_new ();
+  if ( tr->comment )
+    gtk_entry_set_text ( GTK_ENTRY(widgets->w_comment), tr->comment );
+  g_signal_connect_swapped ( widgets->w_comment, "activate", G_CALLBACK(a_dialog_response_accept), GTK_DIALOG(dialog) );
+  content[cnt++] = widgets->w_comment;
+
+  vik_units_distance_t dist_units = a_vik_get_units_distance ();
+
+  tr_len = widgets->track_length = vik_track_get_length(tr);
+  switch (dist_units) {
+  case VIK_UNITS_DISTANCE_KILOMETRES:
+    g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f km", tr_len/1000.0 );
+    break;
+  case VIK_UNITS_DISTANCE_MILES:
+    g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f miles", tr_len/1600.0 );
+    break;
+  default:
+    g_critical("Houston, we've had a problem. distance=%d", dist_units);
+  }
+  widgets->w_track_length = content[cnt++] = gtk_label_new ( tmp_buf );
 
   tp_count = vik_track_get_tp_count(tr);
   g_snprintf(tmp_buf, sizeof(tmp_buf), "%u", tp_count );
 
   tp_count = vik_track_get_tp_count(tr);
   g_snprintf(tmp_buf, sizeof(tmp_buf), "%u", tp_count );
-  l_tps = gtk_label_new ( tmp_buf );
+  widgets->w_tp_count = content[cnt++] = gtk_label_new ( tmp_buf );
 
   seg_count = vik_track_get_segment_count(tr) ;
   g_snprintf(tmp_buf, sizeof(tmp_buf), "%u", seg_count );
 
   seg_count = vik_track_get_segment_count(tr) ;
   g_snprintf(tmp_buf, sizeof(tmp_buf), "%u", seg_count );
-  l_segs = gtk_label_new ( tmp_buf );
+  widgets->w_segment_count = content[cnt++] = gtk_label_new ( tmp_buf );
 
   g_snprintf(tmp_buf, sizeof(tmp_buf), "%lu", vik_track_get_dup_point_count(tr) );
 
   g_snprintf(tmp_buf, sizeof(tmp_buf), "%lu", vik_track_get_dup_point_count(tr) );
-  l_dups = gtk_label_new ( tmp_buf );
-
-  g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f m/s", vik_track_get_max_speed(tr) );
-  l_maxs = gtk_label_new ( tmp_buf );
-
-  g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f m/s", vik_track_get_average_speed(tr) );
-  l_avgs = gtk_label_new ( tmp_buf );
-
-  g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f m", (tp_count - seg_count) == 0 ? 0 : tr_len / ( tp_count - seg_count ) );
-  l_avgd = gtk_label_new ( tmp_buf );
-
-  g_snprintf(tmp_buf, sizeof(tmp_buf), "%.0f m - %.0f m", min_alt, max_alt );
-  l_elev = gtk_label_new ( tmp_buf );
+  widgets->w_duptp_count = content[cnt++] = gtk_label_new ( tmp_buf );
+
+  vik_units_speed_t speed_units = a_vik_get_units_speed ();
+  tmp_speed = vik_track_get_max_speed(tr);
+  if ( tmp_speed == 0 )
+    g_snprintf(tmp_buf, sizeof(tmp_buf), _("No Data"));
+  else {
+    switch (speed_units) {
+    case VIK_UNITS_SPEED_KILOMETRES_PER_HOUR:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f km/h", tmp_speed*3.6 );
+      break;
+    case VIK_UNITS_SPEED_MILES_PER_HOUR:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f mph", tmp_speed*2.23693629 );
+      break;
+    case VIK_UNITS_SPEED_METRES_PER_SECOND:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f m/s", tmp_speed );
+      break;
+    case VIK_UNITS_SPEED_KNOTS:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f knots", tmp_speed*1.94384449 );
+      break;
+    default:
+      g_snprintf (tmp_buf, sizeof(tmp_buf), "--" );
+      g_critical("Houston, we've had a problem. speed=%d", speed_units);
+    }
+  }
+  widgets->w_max_speed = content[cnt++] = gtk_label_new ( tmp_buf );
+
+  tmp_speed = vik_track_get_average_speed(tr);
+  if ( tmp_speed == 0 )
+    g_snprintf(tmp_buf, sizeof(tmp_buf), _("No Data"));
+  else {
+    switch (speed_units) {
+    case VIK_UNITS_SPEED_KILOMETRES_PER_HOUR:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f km/h", tmp_speed*3.6 );
+      break;
+    case VIK_UNITS_SPEED_MILES_PER_HOUR:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f mph", tmp_speed* 2.23693629 );
+      break;
+    case VIK_UNITS_SPEED_METRES_PER_SECOND:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f m/s", tmp_speed );
+      break;
+    case VIK_UNITS_SPEED_KNOTS:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f knots", tmp_speed*1.94384449 );
+      break;
+    default:
+      g_snprintf (tmp_buf, sizeof(tmp_buf), "--" );
+      g_critical("Houston, we've had a problem. speed=%d", speed_units);
+    }
+  }
+  widgets->w_avg_speed = content[cnt++] = gtk_label_new ( tmp_buf );
+
+  switch (dist_units) {
+  case VIK_UNITS_DISTANCE_KILOMETRES:
+    // Even though kilometres, the average distance between points is going to be quite small so keep in metres
+    g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f m", (tp_count - seg_count) == 0 ? 0 : tr_len / ( tp_count - seg_count ) );
+    break;
+  case VIK_UNITS_DISTANCE_MILES:
+    g_snprintf(tmp_buf, sizeof(tmp_buf), "%.3f miles", (tp_count - seg_count) == 0 ? 0 : (tr_len / ( tp_count - seg_count )) / 1600.0 );
+    break;
+  default:
+    g_critical("Houston, we've had a problem. distance=%d", dist_units);
+  }
+  widgets->w_avg_dist = content[cnt++] = gtk_label_new ( tmp_buf );
+
+  vik_units_height_t height_units = a_vik_get_units_height ();
+  if ( min_alt == VIK_DEFAULT_ALTITUDE )
+    g_snprintf(tmp_buf, sizeof(tmp_buf), _("No Data"));
+  else {
+    switch (height_units) {
+    case VIK_UNITS_HEIGHT_METRES:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.0f m - %.0f m", min_alt, max_alt );
+      break;
+    case VIK_UNITS_HEIGHT_FEET:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.0f feet - %.0f feet", min_alt*3.2808399, max_alt*3.2808399 );
+      break;
+    default:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "--" );
+      g_critical("Houston, we've had a problem. height=%d", height_units);
+    }
+  }
+  widgets->w_elev_range = content[cnt++] = gtk_label_new ( tmp_buf );
 
   vik_track_get_total_elevation_gain(tr, &max_alt, &min_alt );
 
   vik_track_get_total_elevation_gain(tr, &max_alt, &min_alt );
-  g_snprintf(tmp_buf, sizeof(tmp_buf), "%.0f m / %.0f m", max_alt, min_alt );
-  l_galo = gtk_label_new ( tmp_buf );
-
+  if ( min_alt == VIK_DEFAULT_ALTITUDE )
+    g_snprintf(tmp_buf, sizeof(tmp_buf), _("No Data"));
+  else {
+    switch (height_units) {
+    case VIK_UNITS_HEIGHT_METRES:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.0f m / %.0f m", max_alt, min_alt );
+      break;
+    case VIK_UNITS_HEIGHT_FEET:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.0f feet / %.0f feet", max_alt*3.2808399, min_alt*3.2808399 );
+      break;
+    default:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "--" );
+      g_critical("Houston, we've had a problem. height=%d", height_units);
+    }
+  }
+  widgets->w_elev_gain = content[cnt++] = gtk_label_new ( tmp_buf );
+
+#if 0
+#define PACK(w) gtk_box_pack_start (GTK_BOX(right_vbox), w, FALSE, FALSE, 0);
+  gtk_box_pack_start (GTK_BOX(right_vbox), e_cmt, FALSE, FALSE, 0); 
+  PACK(l_len);
+  PACK(l_tps);
+  PACK(l_segs);
+  PACK(l_dups);
+  PACK(l_maxs);
+  PACK(l_avgs);
+  PACK(l_avgd);
+  PACK(l_elev);
+  PACK(l_galo);
+#undef PACK;
+#endif
+
+  if ( tr->trackpoints && VIK_TRACKPOINT(tr->trackpoints->data)->timestamp )
   {
     time_t t1, t2;
     t1 = VIK_TRACKPOINT(tr->trackpoints->data)->timestamp;
     t2 = VIK_TRACKPOINT(g_list_last(tr->trackpoints)->data)->timestamp;
 
     strncpy(tmp_buf, ctime(&t1), sizeof(tmp_buf));
   {
     time_t t1, t2;
     t1 = VIK_TRACKPOINT(tr->trackpoints->data)->timestamp;
     t2 = VIK_TRACKPOINT(g_list_last(tr->trackpoints)->data)->timestamp;
 
     strncpy(tmp_buf, ctime(&t1), sizeof(tmp_buf));
-    tmp_buf[strlen(tmp_buf)-1] = 0;
-    l_begin = gtk_label_new(tmp_buf);
+    tmp_buf[sizeof(tmp_buf)-1] = 0;
+    g_strchomp(tmp_buf);
+    widgets->w_time_start = content[cnt++] = gtk_label_new(tmp_buf);
 
     strncpy(tmp_buf, ctime(&t2), sizeof(tmp_buf));
 
     strncpy(tmp_buf, ctime(&t2), sizeof(tmp_buf));
-    tmp_buf[strlen(tmp_buf)-1] = 0;
-    l_end = gtk_label_new(tmp_buf);
-
-    g_snprintf(tmp_buf, sizeof(tmp_buf), "%d minutes", (int)(t2-t1)/60);
-    l_dur = gtk_label_new(tmp_buf);
-  }
-
-  gtk_box_pack_start (GTK_BOX(right_vbox), e_cmt, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_len, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_tps, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_segs, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_dups, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_maxs, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_avgs, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_avgd, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_elev, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_galo, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_begin, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_end, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(right_vbox), l_dur, FALSE, FALSE, 0);
-
-  hbox = gtk_hbox_new ( TRUE, 0 );
-  gtk_box_pack_start (GTK_BOX(hbox), left_vbox, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(hbox), right_vbox, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), hbox, FALSE, FALSE, 0);
-  gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), profile, FALSE, FALSE, 0);
+    tmp_buf[sizeof(tmp_buf)-1] = 0;
+    g_strchomp(tmp_buf);
+    widgets->w_time_end = content[cnt++] = gtk_label_new(tmp_buf);
+
+    g_snprintf(tmp_buf, sizeof(tmp_buf), _("%d minutes"), (int)(t2-t1)/60);
+    widgets->w_time_dur = content[cnt++] = gtk_label_new(tmp_buf);
+  } else {
+    widgets->w_time_start = content[cnt++] = gtk_label_new(_("No Data"));
+    widgets->w_time_end = content[cnt++] = gtk_label_new(_("No Data"));
+    widgets->w_time_dur = content[cnt++] = gtk_label_new(_("No Data"));
+  }
+
+  table = GTK_TABLE(gtk_table_new (cnt, 2, FALSE));
+  gtk_table_set_col_spacing (table, 0, 10);
+  for (i=0; i<cnt; i++) {
+    GtkWidget *label;
+
+    label = gtk_label_new(NULL);
+    gtk_misc_set_alignment ( GTK_MISC(label), 1, 0 );
+    gtk_label_set_markup ( GTK_LABEL(label), _(label_texts[i]) );
+    gtk_table_attach_defaults ( table, label, 0, 1, i, i+1 );
+    if (GTK_IS_MISC(content[i])) {
+      gtk_misc_set_alignment ( GTK_MISC(content[i]), 0, 0 );
+    }
+    gtk_table_attach_defaults ( table, content[i], 1, 2, i, i+1 );
+  }
 
 
+  gtk_notebook_append_page(GTK_NOTEBOOK(graphs), GTK_WIDGET(table), gtk_label_new(_("Statistics")));
 
 
-  gtk_widget_show_all ( dialog );
-  resp = gtk_dialog_run (GTK_DIALOG (dialog));
-  if ( resp == GTK_RESPONSE_ACCEPT )
-    vik_track_set_comment ( tr, gtk_entry_get_text ( GTK_ENTRY(e_cmt) ) );
+  if ( profile ) {
+    GtkWidget *page = NULL;
+    widgets->w_cur_dist = gtk_label_new(_("No Data"));
+    page = create_graph_page (profile, _("<b>Track Distance:</b>"), widgets->w_cur_dist );
+    gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Elevation-distance")));
+  }
+
+  if ( vtdiag ) {
+    GtkWidget *page = NULL;
+    widgets->w_cur_time = gtk_label_new(_("No Data"));
+    page = create_graph_page (vtdiag, _("<b>Track Time:</b>"), widgets->w_cur_time );
+    gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Speed-time")));
+  }
+
+  gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), graphs, FALSE, FALSE, 0);
 
 
-  gtk_widget_destroy ( dialog );
-  return resp;
+  gtk_dialog_set_response_sensitive(GTK_DIALOG(dialog), VIK_TRW_LAYER_PROPWIN_SPLIT_MARKER, FALSE);
+  if (seg_count <= 1)
+    gtk_dialog_set_response_sensitive(GTK_DIALOG(dialog), VIK_TRW_LAYER_PROPWIN_SPLIT, FALSE);
+  if (vik_track_get_dup_point_count(tr) <= 0)
+    gtk_dialog_set_response_sensitive(GTK_DIALOG(dialog), VIK_TRW_LAYER_PROPWIN_DEL_DUP, FALSE);
+
+  vik_track_set_property_dialog(tr, dialog);
+  gtk_widget_show_all ( dialog );
 }
 }