]> git.street.me.uk Git - andy/viking.git/blobdiff - src/viktrwlayer_propwin.c
Import Launchpad translation updates
[andy/viking.git] / src / viktrwlayer_propwin.c
index 87edaf87a835e8dc8e7ab9f6a2b2104f028ba8bb..000f49e06d9d98dd6fae58b5cf3f20b14ebf091c 100644 (file)
@@ -4,7 +4,7 @@
  * Copyright (C) 2003-2005, Evan Battaglia <gtoevan@gmx.net>
  * Copyright (C) 2005-2007, Alex Foobarian <foobarian@gmail.com>
  * Copyright (C) 2007-2008, Quy Tonthat <qtonthat@gmail.com>
- * Copyright (C) 2011, Rob Norris <rw_norris@hotmail.com>
+ * Copyright (C) 2012, Rob Norris <rw_norris@hotmail.com>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
 #define PROPWIN_PROFILE_WIDTH 600
 #define PROPWIN_PROFILE_HEIGHT 300
 
-#define PROPWIN_LABEL_FONT "Sans 7"
-
 typedef enum {
   PROPWIN_GRAPH_TYPE_ELEVATION_DISTANCE,
+  PROPWIN_GRAPH_TYPE_GRADIENT_DISTANCE,
   PROPWIN_GRAPH_TYPE_SPEED_TIME,
   PROPWIN_GRAPH_TYPE_DISTANCE_TIME,
   PROPWIN_GRAPH_TYPE_ELEVATION_TIME,
@@ -73,6 +72,12 @@ static const gdouble chunksa[] = {2.0, 5.0, 10.0, 15.0, 20.0,
                                  150.0, 200.0, 250.0, 375.0, 500.0,
                                  750.0, 1000.0, 2000.0, 5000.0, 10000.0, 100000.0};
 
+/* (Hopefully!) Human friendly gradient grid sizes - note no fixed 'ratio' just numbers that look nice...*/
+static const gdouble chunksg[] = {1.0, 2.0, 3.0, 4.0, 5.0, 8.0, 10.0,
+                                 12.0, 15.0, 20.0, 25.0, 30.0, 35.0, 40.0, 45.0, 50.0, 75.0,
+                                 100.0, 150.0, 200.0, 250.0, 375.0, 500.0,
+                                 750.0, 1000.0, 10000.0};
+
 /* (Hopefully!) Human friendly grid sizes - note no fixed 'ratio' just numbers that look nice...*/
 /* As need to cover walking speeds - have many low numbers (but also may go up to airplane speeds!) */
 static const gdouble chunkss[] = {1.0, 2.0, 3.0, 4.0, 5.0, 8.0, 10.0,
@@ -95,7 +100,9 @@ typedef struct _propwidgets {
   gboolean  configure_dialog;
   VikTrwLayer *vtl;
   VikTrack *tr;
-  gchar *track_name;
+  gpointer  trk_id;
+  VikViewport *vvp;
+  VikLayersPanel *vlp;
   gint      profile_width;
   gint      profile_height;
   gint      profile_width_old;
@@ -104,20 +111,25 @@ typedef struct _propwidgets {
   gint      profile_height_offset;
   GtkWidget *dialog;
   GtkWidget *w_comment;
+  GtkWidget *w_description;
   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_mvg_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_color;
   GtkWidget *w_cur_dist; /*< Current distance */
   GtkWidget *w_cur_elevation;
+  GtkWidget *w_cur_gradient_dist; /*< Current distance on gradient graph */
+  GtkWidget *w_cur_gradient_gradient; /*< Current gradient on gradient graph */
   GtkWidget *w_cur_time; /*< Current time */
   GtkWidget *w_cur_speed;
   GtkWidget *w_cur_dist_dist; /*< Current distance on distance graph */
@@ -129,17 +141,20 @@ typedef struct _propwidgets {
   GtkWidget *w_show_dem;
   GtkWidget *w_show_alt_gps_speed;
   GtkWidget *w_show_gps_speed;
+  GtkWidget *w_show_gradient_gps_speed;
   GtkWidget *w_show_dist_speed;
   GtkWidget *w_show_elev_speed;
   GtkWidget *w_show_sd_gps_speed;
   gdouble   track_length;
   gdouble   track_length_inc_gaps;
   PropSaved elev_graph_saved_img;
+  PropSaved gradient_graph_saved_img;
   PropSaved speed_graph_saved_img;
   PropSaved dist_graph_saved_img;
   PropSaved elev_time_graph_saved_img;
   PropSaved speed_dist_graph_saved_img;
   GtkWidget *elev_box;
+  GtkWidget *gradient_box;
   GtkWidget *speed_box;
   GtkWidget *dist_box;
   GtkWidget *elev_time_box;
@@ -149,7 +164,16 @@ typedef struct _propwidgets {
   gdouble   min_altitude;
   gdouble   max_altitude;
   gdouble   draw_min_altitude;
+  gdouble   draw_min_altitude_time;
   gint      cia; // Chunk size Index into Altitudes
+  gint      ciat; // Chunk size Index into Altitudes / Time
+  // NB cia & ciat are normally same value but sometimes not due to differing methods of altitude array creation
+  //    thus also have draw_min_altitude for each altitude graph type
+  gdouble   *gradients;
+  gdouble   min_gradient;
+  gdouble   max_gradient;
+  gdouble   draw_min_gradient;
+  gint      cig; // Chunk size Index into Gradients
   gdouble   *speeds;
   gdouble   *speeds_dist;
   gdouble   min_speed;
@@ -177,6 +201,8 @@ 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->gradient_graph_saved_img.img)
+    g_object_unref(widgets->gradient_graph_saved_img.img);
   if (widgets->speed_graph_saved_img.img)
     g_object_unref(widgets->speed_graph_saved_img.img);
   if (widgets->dist_graph_saved_img.img)
@@ -187,6 +213,8 @@ static void prop_widgets_free(PropWidgets *widgets)
     g_object_unref(widgets->speed_dist_graph_saved_img.img);
   if (widgets->altitudes)
     g_free(widgets->altitudes);
+  if (widgets->gradients)
+    g_free(widgets->gradients);
   if (widgets->speeds)
     g_free(widgets->speeds);
   if (widgets->distances)
@@ -258,6 +286,49 @@ static void get_new_min_and_chunk_index_altitude (gdouble mina, gdouble maxa, gd
   }
 }
 
+/**
+ * Returns via pointers:
+ *   the new minimum value to be used for the altitude graph
+ *   the index in to the altitudes chunk sizes array (ci = Chunk Index)
+ */
+static void get_new_min_and_chunk_index_gradient (gdouble mina, gdouble maxa, gdouble *new_min, gint *ci)
+{
+  /* Get unitized chunk */
+  /* Find suitable chunk index */
+  *ci = 0;
+  gdouble diffa_chunk = (maxa - mina)/LINES;
+
+  /* Loop through to find best match */
+  while (diffa_chunk > chunksg[*ci]) {
+    (*ci)++;
+    /* Last Resort Check */
+    if ( *ci == sizeof(chunksg)/sizeof(chunksg[0]) )
+      break;
+  }
+
+  /* Ensure adjusted minimum .. maximum covers mina->maxa */
+
+  // Now work out adjusted minimum point to the nearest lowest chunk divisor value
+  // When negative ensure logic uses lowest value
+  if ( mina < 0 )
+    *new_min = (gdouble) ( ( (gint)(mina - chunksg[*ci]) / (gint)chunksg[*ci] ) * (gint)chunksg[*ci] );
+  else
+    *new_min = (gdouble) ( ( (gint)mina / (gint)chunksg[*ci] ) * (gint)chunksg[*ci] );
+
+  // Range not big enough - as new minimum has lowered
+  if ((*new_min + (chunksg[*ci] * LINES) < maxa)) {
+    // Next chunk should cover it
+    if ( *ci < sizeof(chunksg)/sizeof(chunksg[0]) ) {
+      (*ci)++;
+      // Remember to adjust the minimum too...
+      if ( mina < 0 )
+       *new_min = (gdouble) ( ( (gint)(mina - chunksg[*ci]) / (gint)chunksg[*ci] ) * (gint)chunksg[*ci] );
+      else
+       *new_min = (gdouble) ( ( (gint)mina / (gint)chunksg[*ci] ) * (gint)chunksg[*ci] );
+    }
+  }
+}
+
 /**
  * Returns via pointers:
  *   the new minimum value to be used for the appropriate graph
@@ -311,7 +382,7 @@ static VikTrackpoint *set_center_at_graph_position(gdouble event_x,
       /* since vlp not set, vvp should be valid instead! */
       if ( vvp )
        vik_viewport_set_center_coord ( vvp, &coord );
-      vik_layer_emit_update ( VIK_LAYER(vtl), FALSE );
+      vik_layer_emit_update ( VIK_LAYER(vtl) );
     }
   }
   return trackpoint;
@@ -406,19 +477,14 @@ static gdouble tp_percentage_by_distance ( VikTrack *tr, VikTrackpoint *trackpoi
   return pc;
 }
 
-static void track_graph_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along, VikPropWinGraphType_t graph_type )
+static void track_graph_click( GtkWidget *event_box, GdkEventButton *event, PropWidgets *widgets, VikPropWinGraphType_t graph_type )
 {
-  VikTrack *tr = pass_along[0];
-  VikLayersPanel *vlp = pass_along[1];
-  VikViewport *vvp = pass_along[2];
-  PropWidgets *widgets = pass_along[3];
-
   gboolean is_time_graph =
     ( graph_type == PROPWIN_GRAPH_TYPE_SPEED_TIME ||
       graph_type == PROPWIN_GRAPH_TYPE_DISTANCE_TIME ||
       graph_type == PROPWIN_GRAPH_TYPE_ELEVATION_TIME );
 
-  VikTrackpoint *trackpoint = set_center_at_graph_position(event->x, event_box->allocation.width, widgets->vtl, vlp, vvp, tr, is_time_graph, widgets->profile_width);
+  VikTrackpoint *trackpoint = set_center_at_graph_position(event->x, event_box->allocation.width, widgets->vtl, widgets->vlp, widgets->vvp, widgets->tr, is_time_graph, widgets->profile_width);
   // Unable to get the point so give up
   if ( trackpoint == NULL ) {
     gtk_dialog_set_response_sensitive(GTK_DIALOG(widgets->dialog), VIK_TRW_LAYER_PROPWIN_SPLIT_MARKER, FALSE);
@@ -448,6 +514,11 @@ static void track_graph_click( GtkWidget *event_box, GdkEventButton *event, gpoi
       graph_saved_img = &widgets->elev_graph_saved_img;
       is_time_graph   = FALSE;
       break;
+    case PROPWIN_GRAPH_TYPE_GRADIENT_DISTANCE:
+      graph_box       = widgets->gradient_box;
+      graph_saved_img = &widgets->gradient_graph_saved_img;
+      is_time_graph   = FALSE;
+      break;
     case PROPWIN_GRAPH_TYPE_SPEED_TIME:
       graph_box       = widgets->speed_box;
       graph_saved_img = &widgets->speed_graph_saved_img;
@@ -477,9 +548,9 @@ static void track_graph_click( GtkWidget *event_box, GdkEventButton *event, gpoi
       image = GTK_WIDGET(child->data);
 
       if (is_time_graph)
-       pc = tp_percentage_by_time ( tr, trackpoint );
+       pc = tp_percentage_by_time ( widgets->tr, trackpoint );
       else
-       pc = tp_percentage_by_distance ( tr, trackpoint, widgets->track_length_inc_gaps );
+       pc = tp_percentage_by_distance ( widgets->tr, trackpoint, widgets->track_length_inc_gaps );
 
       if (!isnan(pc)) {
        gdouble marker_x = (pc * widgets->profile_width) + MARGIN;
@@ -501,33 +572,39 @@ static void track_graph_click( GtkWidget *event_box, GdkEventButton *event, gpoi
   gtk_dialog_set_response_sensitive(GTK_DIALOG(widgets->dialog), VIK_TRW_LAYER_PROPWIN_SPLIT_MARKER, widgets->is_marker_drawn);
 }
 
-static gboolean track_profile_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along )
+static gboolean track_profile_click( GtkWidget *event_box, GdkEventButton *event, gpointer ptr )
+{
+  track_graph_click(event_box, event, ptr, PROPWIN_GRAPH_TYPE_ELEVATION_DISTANCE);
+  return TRUE;  /* don't call other (further) callbacks */
+}
+
+static gboolean track_gradient_click( GtkWidget *event_box, GdkEventButton *event, gpointer ptr )
 {
-  track_graph_click(event_box, event, pass_along, PROPWIN_GRAPH_TYPE_ELEVATION_DISTANCE);
+  track_graph_click(event_box, event, ptr, PROPWIN_GRAPH_TYPE_GRADIENT_DISTANCE);
   return TRUE;  /* don't call other (further) callbacks */
 }
 
-static gboolean track_vt_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along )
+static gboolean track_vt_click( GtkWidget *event_box, GdkEventButton *event, gpointer ptr )
 {
-  track_graph_click(event_box, event, pass_along, PROPWIN_GRAPH_TYPE_SPEED_TIME);
+  track_graph_click(event_box, event, ptr, PROPWIN_GRAPH_TYPE_SPEED_TIME);
   return TRUE;  /* don't call other (further) callbacks */
 }
 
-static gboolean track_dt_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along )
+static gboolean track_dt_click( GtkWidget *event_box, GdkEventButton *event, gpointer ptr )
 {
-  track_graph_click(event_box, event, pass_along, PROPWIN_GRAPH_TYPE_DISTANCE_TIME);
+  track_graph_click(event_box, event, ptr, PROPWIN_GRAPH_TYPE_DISTANCE_TIME);
   return TRUE;  /* don't call other (further) callbacks */
 }
 
-static gboolean track_et_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along )
+static gboolean track_et_click( GtkWidget *event_box, GdkEventButton *event, gpointer ptr )
 {
-  track_graph_click(event_box, event, pass_along, PROPWIN_GRAPH_TYPE_ELEVATION_TIME);
+  track_graph_click(event_box, event, ptr, PROPWIN_GRAPH_TYPE_ELEVATION_TIME);
   return TRUE;  /* don't call other (further) callbacks */
 }
 
-static gboolean track_sd_click( GtkWidget *event_box, GdkEventButton *event, gpointer *pass_along )
+static gboolean track_sd_click( GtkWidget *event_box, GdkEventButton *event, gpointer ptr )
 {
-  track_graph_click(event_box, event, pass_along, PROPWIN_GRAPH_TYPE_SPEED_DISTANCE);
+  track_graph_click(event_box, event, ptr, PROPWIN_GRAPH_TYPE_SPEED_DISTANCE);
   return TRUE;  /* don't call other (further) callbacks */
 }
 
@@ -546,6 +623,21 @@ static gint blobby_altitude ( gdouble x_blob, PropWidgets *widgets )
   return y_blob;
 }
 
+/**
+ * Calculate y position for blob on gradient graph
+ */
+static gint blobby_gradient ( gdouble x_blob, PropWidgets *widgets )
+{
+  gint ix = (gint)x_blob;
+  // Ensure ix is inbounds
+  if (ix == widgets->profile_width)
+    ix--;
+
+  gint y_blob = widgets->profile_height-widgets->profile_height*(widgets->gradients[ix]-widgets->draw_min_gradient)/(chunksg[widgets->cig]*LINES);
+
+  return y_blob;
+}
+
 /**
  * Calculate y position for blob on speed graph
  */
@@ -587,8 +679,7 @@ static gint blobby_altitude_time ( gdouble x_blob, PropWidgets *widgets )
   if (ix == widgets->profile_width)
     ix--;
 
-  gint y_blob = widgets->profile_height-widgets->profile_height*(widgets->ats[ix]-widgets->draw_min_altitude)/(chunksa[widgets->cia]*LINES);
-  // only difference here [could refactor]?          _____________________/
+  gint y_blob = widgets->profile_height-widgets->profile_height*(widgets->ats[ix]-widgets->draw_min_altitude_time)/(chunksa[widgets->ciat]*LINES);
   return y_blob;
 }
 
@@ -608,10 +699,8 @@ static gint blobby_speed_dist ( gdouble x_blob, PropWidgets *widgets )
 }
 
 
-void track_profile_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_along )
+void track_profile_move( GtkWidget *event_box, GdkEventMotion *event, PropWidgets *widgets )
 {
-  VikTrack *tr = pass_along[0];
-  PropWidgets *widgets = pass_along[3];
   int mouse_x, mouse_y;
   GdkModifierType state;
 
@@ -627,7 +716,7 @@ void track_profile_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *
     x = widgets->profile_width;
 
   gdouble meters_from_start;
-  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_dist ( tr, (gdouble) x / widgets->profile_width, &meters_from_start );
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_dist ( widgets->tr, (gdouble) x / widgets->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 ();
@@ -667,7 +756,7 @@ void track_profile_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *
 
   gdouble marker_x = -1.0; // i.e. Don't draw unless we get a valid value
   if (widgets->is_marker_drawn) {
-    gdouble pc = tp_percentage_by_distance ( tr, widgets->marker_tp, widgets->track_length_inc_gaps );
+    gdouble pc = tp_percentage_by_distance ( widgets->tr, widgets->marker_tp, widgets->track_length_inc_gaps );
     if (!isnan(pc)) {
       marker_x = (pc * widgets->profile_width) + MARGIN;
     }
@@ -687,10 +776,85 @@ void track_profile_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *
   g_list_free(child);
 }
 
-void track_vt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_along )
+void track_gradient_move( GtkWidget *event_box, GdkEventMotion *event, PropWidgets *widgets )
+{
+  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 - event_box->allocation.width / 2 + widgets->profile_width / 2 - MARGIN / 2;
+  if (x < 0)
+    x = 0;
+  if (x > widgets->profile_width)
+    x = widgets->profile_width;
+
+  gdouble meters_from_start;
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_dist ( widgets->tr, (gdouble) x / widgets->profile_width, &meters_from_start );
+  if (trackpoint && widgets->w_cur_gradient_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", VIK_METERS_TO_MILES(meters_from_start) );
+      break;
+    default:
+      g_critical("Houston, we've had a problem. distance=%d", dist_units);
+    }
+    gtk_label_set_text(GTK_LABEL(widgets->w_cur_gradient_dist), tmp_buf);
+  }
+
+  // Show track gradient for this position - to the nearest whole number
+  if (trackpoint && widgets->w_cur_gradient_gradient) {
+    static gchar tmp_buf[20];
+    
+    double gradient = widgets->gradients[(int) x];
+
+    g_snprintf(tmp_buf, sizeof(tmp_buf), "%d%%", (int)gradient);
+    gtk_label_set_text(GTK_LABEL(widgets->w_cur_gradient_gradient), tmp_buf);
+  }
+
+  widgets->blob_tp = trackpoint;
+
+  if ( widgets->gradients == NULL )
+    return;
+
+  GtkWidget *window = gtk_widget_get_toplevel (event_box);
+  GList *child = gtk_container_get_children(GTK_CONTAINER(event_box));
+  GtkWidget *image = GTK_WIDGET(child->data);
+
+  gint y_blob = blobby_gradient (x, widgets);
+
+  gdouble marker_x = -1.0; // i.e. Don't draw unless we get a valid value
+  if (widgets->is_marker_drawn) {
+    gdouble pc = tp_percentage_by_distance ( widgets->tr, widgets->marker_tp, widgets->track_length_inc_gaps );
+    if (!isnan(pc)) {
+      marker_x = (pc * widgets->profile_width) + MARGIN;
+    }
+  }
+
+  save_image_and_draw_graph_marks (image,
+                                  marker_x,
+                                  window->style->black_gc,
+                                  MARGIN+x,
+                                  y_blob,
+                                  &widgets->gradient_graph_saved_img,
+                                  widgets->profile_width,
+                                  widgets->profile_height,
+                                  &widgets->is_marker_drawn,
+                                  &widgets->is_blob_drawn);
+
+  g_list_free(child);
+}
+
+void track_vt_move( GtkWidget *event_box, GdkEventMotion *event, PropWidgets *widgets )
 {
-  VikTrack *tr = pass_along[0];
-  PropWidgets *widgets = pass_along[3];
   int mouse_x, mouse_y;
   GdkModifierType state;
 
@@ -706,7 +870,7 @@ void track_vt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
     x = widgets->profile_width;
 
   time_t seconds_from_start;
-  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_time ( tr, (gdouble) x / widgets->profile_width, &seconds_from_start );
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_time ( widgets->tr, (gdouble) x / widgets->profile_width, &seconds_from_start );
   if (trackpoint && widgets->w_cur_time) {
     static gchar tmp_buf[20];
     guint h, m, s;
@@ -760,7 +924,7 @@ void track_vt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
 
   gdouble marker_x = -1.0; // i.e. Don't draw unless we get a valid value
   if (widgets->is_marker_drawn) {
-    gdouble pc = tp_percentage_by_time ( tr, widgets->marker_tp );
+    gdouble pc = tp_percentage_by_time ( widgets->tr, widgets->marker_tp );
     if (!isnan(pc)) {
       marker_x = (pc * widgets->profile_width) + MARGIN;
     }
@@ -783,10 +947,8 @@ void track_vt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
 /**
  * Update labels and blob marker on mouse moves in the distance/time graph
  */
-void track_dt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_along )
+void track_dt_move( GtkWidget *event_box, GdkEventMotion *event, PropWidgets *widgets )
 {
-  VikTrack *tr = pass_along[0];
-  PropWidgets *widgets = pass_along[3];
   int mouse_x, mouse_y;
   GdkModifierType state;
 
@@ -802,7 +964,7 @@ void track_dt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
     x = widgets->profile_width;
 
   time_t seconds_from_start;
-  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_time ( tr, (gdouble) x / widgets->profile_width, &seconds_from_start );
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_time ( widgets->tr, (gdouble) x / widgets->profile_width, &seconds_from_start );
   if (trackpoint && widgets->w_cur_dist_time) {
     static gchar tmp_buf[20];
     guint h, m, s;
@@ -841,7 +1003,7 @@ void track_dt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
 
   gdouble marker_x = -1.0; // i.e. Don't draw unless we get a valid value
   if (widgets->is_marker_drawn) {
-    gdouble pc = tp_percentage_by_time ( tr, widgets->marker_tp );
+    gdouble pc = tp_percentage_by_time ( widgets->tr, widgets->marker_tp );
     if (!isnan(pc)) {
       marker_x = (pc * widgets->profile_width) + MARGIN;
     }
@@ -864,10 +1026,8 @@ void track_dt_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
 /**
  * Update labels and blob marker on mouse moves in the elevation/time graph
  */
-void track_et_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_along )
+void track_et_move( GtkWidget *event_box, GdkEventMotion *event, PropWidgets *widgets )
 {
-  VikTrack *tr = pass_along[0];
-  PropWidgets *widgets = pass_along[3];
   int mouse_x, mouse_y;
   GdkModifierType state;
 
@@ -883,7 +1043,7 @@ void track_et_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
     x = widgets->profile_width;
 
   time_t seconds_from_start;
-  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_time ( tr, (gdouble) x / widgets->profile_width, &seconds_from_start );
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_time ( widgets->tr, (gdouble) x / widgets->profile_width, &seconds_from_start );
   if (trackpoint && widgets->w_cur_elev_time) {
     static gchar tmp_buf[20];
     guint h, m, s;
@@ -922,7 +1082,7 @@ void track_et_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
 
   gdouble marker_x = -1.0; // i.e. Don't draw unless we get a valid value
   if (widgets->is_marker_drawn) {
-    gdouble pc = tp_percentage_by_time ( tr, widgets->marker_tp );
+    gdouble pc = tp_percentage_by_time ( widgets->tr, widgets->marker_tp );
     if (!isnan(pc)) {
       marker_x = (pc * widgets->profile_width) + MARGIN;
     }
@@ -942,10 +1102,8 @@ void track_et_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
   g_list_free(child);
 }
 
-void track_sd_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_along )
+void track_sd_move( GtkWidget *event_box, GdkEventMotion *event, PropWidgets *widgets )
 {
-  VikTrack *tr = pass_along[0];
-  PropWidgets *widgets = pass_along[3];
   int mouse_x, mouse_y;
   GdkModifierType state;
 
@@ -961,7 +1119,7 @@ void track_sd_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
     x = widgets->profile_width;
 
   gdouble meters_from_start;
-  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_dist ( tr, (gdouble) x / widgets->profile_width, &meters_from_start );
+  VikTrackpoint *trackpoint = vik_track_get_closest_tp_by_percentage_dist ( widgets->tr, (gdouble) x / widgets->profile_width, &meters_from_start );
   if (trackpoint && widgets->w_cur_speed_dist) {
     static gchar tmp_buf[20];
     vik_units_distance_t dist_units = a_vik_get_units_distance ();
@@ -1020,7 +1178,7 @@ void track_sd_move( GtkWidget *event_box, GdkEventMotion *event, gpointer *pass_
 
   gdouble marker_x = -1.0; // i.e. Don't draw unless we get a valid value
   if (widgets->is_marker_drawn) {
-    gdouble pc = tp_percentage_by_distance ( tr, widgets->marker_tp, widgets->track_length_inc_gaps );
+    gdouble pc = tp_percentage_by_distance ( widgets->tr, widgets->marker_tp, widgets->track_length_inc_gaps );
     if (!isnan(pc)) {
       marker_x = (pc * widgets->profile_width) + MARGIN;
     }
@@ -1154,15 +1312,13 @@ static void draw_elevations (GtkWidget *image, VikTrack *tr, PropWidgets *widget
   
   /* draw grid */
   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 (PROPWIN_LABEL_FONT);
-    pango_layout_set_font_description (pl, pfd);
-    pango_font_description_free (pfd);
+    pango_layout_set_font_description (pl, window->style->font_desc);
+
     switch (height_units) {
     case VIK_UNITS_HEIGHT_METRES:
       sprintf(s, "%8dm", (int)(mina + (LINES-i)*chunksa[widgets->cia]));
@@ -1236,6 +1392,137 @@ static void draw_elevations (GtkWidget *image, VikTrack *tr, PropWidgets *widget
   g_object_unref ( G_OBJECT(no_alt_info) );
 }
 
+/**
+ * Draws representative speed on the supplied pixmap
+ *   (which is the gradients graph)
+ */
+static void draw_speed_dist(VikTrack *tr,
+                                   GdkDrawable *pix,
+                                   GdkGC *speed_gc,
+                                   gdouble max_speed_in,
+                                   gint width,
+                                   gint height,
+                                   gint margin,
+                                   gboolean do_speed)
+{
+  GList *iter;
+  gdouble max_speed = 0;
+  gdouble total_length = vik_track_get_length_including_gaps(tr);
+
+  // Calculate the max speed factor
+  if (do_speed)
+    max_speed = max_speed_in * 110 / 100;
+
+  gdouble dist = 0;
+  for (iter = tr->trackpoints->next; iter; iter = iter->next) {
+    int x;
+    dist += vik_coord_diff ( &(VIK_TRACKPOINT(iter->data)->coord),
+                            &(VIK_TRACKPOINT(iter->prev->data)->coord) );
+    x = (width * dist)/total_length + margin;
+    if (do_speed) {
+      // This is just a speed indicator - no actual values can be inferred by user
+      if (!isnan(VIK_TRACKPOINT(iter->data)->speed)) {
+       int 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);
+      }
+    }
+  }
+}
+
+/**
+ * Draw just the gradient image
+ */
+static void draw_gradients (GtkWidget *image, VikTrack *tr, PropWidgets *widgets )
+{
+  GtkWidget *window;
+  GdkPixmap *pix;
+  gdouble mina;
+  guint i;
+
+  GdkColor color;
+
+  // Free previous allocation
+  if ( widgets->gradients )
+    g_free ( widgets->gradients );
+
+  widgets->gradients = vik_track_make_gradient_map ( tr, widgets->profile_width );
+
+  if ( widgets->gradients == NULL )
+    return;
+
+  minmax_array(widgets->gradients, &widgets->min_gradient, &widgets->max_gradient, TRUE, widgets->profile_width);
+
+  get_new_min_and_chunk_index_gradient (widgets->min_gradient, widgets->max_gradient, &widgets->draw_min_gradient, &widgets->cig);
+
+  // Assign locally
+  mina = widgets->draw_min_gradient;
+
+  window = gtk_widget_get_toplevel (widgets->gradient_box);
+
+  pix = gdk_pixmap_new( window->window, widgets->profile_width + MARGIN, widgets->profile_height, -1 );
+
+  gtk_image_set_from_pixmap ( GTK_IMAGE(image), pix, NULL );
+
+  /* clear the image */
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->bg_gc[0], 
+                    TRUE, 0, 0, MARGIN, widgets->profile_height);
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->mid_gc[0], 
+                    TRUE, MARGIN, 0, widgets->profile_width, widgets->profile_height);
+  
+  /* draw grid */
+  for (i=0; i<=LINES; i++) {
+    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);
+    pango_layout_set_font_description (pl, window->style->font_desc);
+
+    sprintf(s, "%8d%%", (int)(mina + (LINES-i)*chunksg[widgets->cig]));
+    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*widgets->profile_height/LINES - h/2, 0, widgets->profile_height-h), pl);
+
+    gdk_draw_line (GDK_DRAWABLE(pix), window->style->dark_gc[0], 
+                  MARGIN, widgets->profile_height/LINES * i, MARGIN + widgets->profile_width, widgets->profile_height/LINES * i);
+    g_object_unref ( G_OBJECT ( pl ) );
+    pl = NULL;
+  }
+
+  /* draw gradients */
+  for ( i = 0; i < widgets->profile_width; i++ )
+      gdk_draw_line ( GDK_DRAWABLE(pix), window->style->dark_gc[3], 
+                     i + MARGIN, widgets->profile_height, i + MARGIN, widgets->profile_height-widgets->profile_height*(widgets->gradients[i]-mina)/(chunksg[widgets->cig]*LINES) );
+
+  if ( gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON(widgets->w_show_gradient_gps_speed)) ) {
+    GdkGC *gps_speed_gc = gdk_gc_new ( window->window );
+
+    gdk_color_parse ( "red", &color );
+    gdk_gc_set_rgb_fg_color ( gps_speed_gc, &color);
+
+    // Ensure somekind of max speed when not set
+    if ( widgets->max_speed < 0.01 )
+      widgets->max_speed = vik_track_get_max_speed(tr);
+
+    draw_speed_dist(tr,
+                           GDK_DRAWABLE(pix),
+                           gps_speed_gc,
+                           widgets->max_speed,
+                           widgets->profile_width,
+                           widgets->profile_height,
+                           MARGIN,
+                           gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON(widgets->w_show_alt_gps_speed)));
+    
+    g_object_unref ( G_OBJECT(gps_speed_gc) );
+  }
+
+  /* draw border */
+  gdk_draw_rectangle(GDK_DRAWABLE(pix), window->style->black_gc, FALSE, MARGIN, 0, widgets->profile_width-1, widgets->profile_height-1);
+
+  g_object_unref ( G_OBJECT(pix) );
+}
+
 /**
  * Draw just the speed (velocity)/time image
  */
@@ -1302,15 +1589,13 @@ static void draw_vt ( GtkWidget *image, VikTrack *tr, PropWidgets *widgets)
 
   /* draw grid */
   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 (PROPWIN_LABEL_FONT);
-    pango_layout_set_font_description (pl, pfd);
-    pango_font_description_free (pfd);
+    pango_layout_set_font_description (pl, window->style->font_desc);
+
     // NB: No need to convert here anymore as numbers are in the appropriate units
     switch (speed_units) {
     case VIK_UNITS_SPEED_KILOMETRES_PER_HOUR:
@@ -1449,15 +1734,13 @@ static void draw_dt ( GtkWidget *image, VikTrack *tr, PropWidgets *widgets )
 
   /* draw grid */
   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 (PROPWIN_LABEL_FONT);
-    pango_layout_set_font_description (pl, pfd);
-    pango_font_description_free (pfd);
+    pango_layout_set_font_description (pl, window->style->font_desc);
+
     if ( dist_units == VIK_UNITS_DISTANCE_MILES )
       sprintf(s, _("%.1f miles"), ((LINES-i)*chunksd[widgets->cid]));
     else
@@ -1536,10 +1819,10 @@ static void draw_et ( GtkWidget *image, VikTrack *tr, PropWidgets *widgets )
 
   minmax_array(widgets->ats, &widgets->min_altitude, &widgets->max_altitude, TRUE, widgets->profile_width);
 
-  get_new_min_and_chunk_index_altitude (widgets->min_altitude, widgets->max_altitude, &widgets->draw_min_altitude, &widgets->cia);
+  get_new_min_and_chunk_index_altitude (widgets->min_altitude, widgets->max_altitude, &widgets->draw_min_altitude_time, &widgets->ciat);
 
   // Assign locally
-  mina = widgets->draw_min_altitude;
+  mina = widgets->draw_min_altitude_time;
 
   window = gtk_widget_get_toplevel (widgets->elev_time_box);
 
@@ -1555,22 +1838,20 @@ static void draw_et ( GtkWidget *image, VikTrack *tr, PropWidgets *widgets )
 
   /* draw grid */
   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 (PROPWIN_LABEL_FONT);
-    pango_layout_set_font_description (pl, pfd);
-    pango_font_description_free (pfd);
+    pango_layout_set_font_description (pl, window->style->font_desc);
+
     switch (height_units) {
     case VIK_UNITS_HEIGHT_METRES:
-      sprintf(s, "%8dm", (int)(mina + (LINES-i)*chunksa[widgets->cia]));
+      sprintf(s, "%8dm", (int)(mina + (LINES-i)*chunksa[widgets->ciat]));
       break;
     case VIK_UNITS_HEIGHT_FEET:
       // NB values already converted into feet
-      sprintf(s, "%8dft", (int)(mina + (LINES-i)*chunksa[widgets->cia]));
+      sprintf(s, "%8dft", (int)(mina + (LINES-i)*chunksa[widgets->ciat]));
       break;
     default:
       sprintf(s, "--");
@@ -1590,7 +1871,7 @@ static void draw_et ( GtkWidget *image, VikTrack *tr, PropWidgets *widgets )
   /* draw elevations */
   for ( i = 0; i < widgets->profile_width; i++ )
       gdk_draw_line ( GDK_DRAWABLE(pix), window->style->dark_gc[3],
-                     i + MARGIN, widgets->profile_height, i + MARGIN, widgets->profile_height-widgets->profile_height*(widgets->ats[i]-mina)/(chunksa[widgets->cia]*LINES) );
+                     i + MARGIN, widgets->profile_height, i + MARGIN, widgets->profile_height-widgets->profile_height*(widgets->ats[i]-mina)/(chunksa[widgets->ciat]*LINES) );
 
   // Show speed indicator
   if ( gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widgets->w_show_elev_speed)) ) {
@@ -1685,15 +1966,13 @@ static void draw_sd ( GtkWidget *image, VikTrack *tr, PropWidgets *widgets)
 
   /* draw grid */
   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 (PROPWIN_LABEL_FONT);
-    pango_layout_set_font_description (pl, pfd);
-    pango_font_description_free (pfd);
+    pango_layout_set_font_description (pl, window->style->font_desc);
+
     // NB: No need to convert here anymore as numbers are in the appropriate units
     switch (speed_units) {
     case VIK_UNITS_SPEED_KILOMETRES_PER_HOUR:
@@ -1779,11 +2058,8 @@ static void draw_sd ( GtkWidget *image, VikTrack *tr, PropWidgets *widgets)
 /**
  * Draw all graphs
  */
-static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean resized )
+static void draw_all_graphs ( GtkWidget *widget, PropWidgets *widgets, gboolean resized )
 {
-  VikTrack *tr = pass_along[0];
-  PropWidgets *widgets = pass_along[3];
-
   // Draw graphs even if they are not visible
 
   GList *child = NULL;
@@ -1803,7 +2079,7 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     }
 
     child = gtk_container_get_children(GTK_CONTAINER(widgets->elev_box));
-    draw_elevations (GTK_WIDGET(child->data), tr, widgets );
+    draw_elevations (GTK_WIDGET(child->data), widgets->tr, widgets );
 
     image = GTK_WIDGET(child->data);
     g_list_free(child);
@@ -1811,11 +2087,11 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     // Ensure marker or blob are redrawn if necessary
     if (widgets->is_marker_drawn || widgets->is_blob_drawn) {
 
-      pc = tp_percentage_by_distance ( tr, widgets->marker_tp, widgets->track_length_inc_gaps );
+      pc = tp_percentage_by_distance ( widgets->tr, widgets->marker_tp, widgets->track_length_inc_gaps );
       gdouble x_blob = -MARGIN - 1.0; // i.e. Don't draw unless we get a valid value
       gint y_blob = 0;
       if (widgets->is_blob_drawn) {
-       pc_blob = tp_percentage_by_distance ( tr, widgets->blob_tp, widgets->track_length_inc_gaps );
+       pc_blob = tp_percentage_by_distance ( widgets->tr, widgets->blob_tp, widgets->track_length_inc_gaps );
        if (!isnan(pc_blob)) {
          x_blob = (pc_blob * widgets->profile_width);
        }
@@ -1840,6 +2116,54 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     }
   }
 
+  // Draw gradients
+  if (widgets->gradient_box != NULL) {
+
+    // Saved image no longer any good as we've resized, so we remove it here
+    if (resized && widgets->gradient_graph_saved_img.img) {
+      g_object_unref(widgets->gradient_graph_saved_img.img);
+      widgets->gradient_graph_saved_img.img = NULL;
+      widgets->gradient_graph_saved_img.saved = FALSE;
+    }
+
+    child = gtk_container_get_children(GTK_CONTAINER(widgets->gradient_box));
+    draw_gradients (GTK_WIDGET(child->data), widgets->tr, widgets );
+
+    image = GTK_WIDGET(child->data);
+    g_list_free(child);
+
+    // Ensure marker or blob are redrawn if necessary
+    if (widgets->is_marker_drawn || widgets->is_blob_drawn) {
+
+      pc = tp_percentage_by_distance ( widgets->tr, widgets->marker_tp, widgets->track_length_inc_gaps );
+      gdouble x_blob = -MARGIN - 1.0; // i.e. Don't draw unless we get a valid value
+      gint y_blob = 0;
+      if (widgets->is_blob_drawn) {
+       pc_blob = tp_percentage_by_distance ( widgets->tr, widgets->blob_tp, widgets->track_length_inc_gaps );
+       if (!isnan(pc_blob)) {
+         x_blob = (pc_blob * widgets->profile_width);
+       }
+       y_blob = blobby_gradient (x_blob, widgets);
+      }
+
+      gdouble marker_x = -1.0; // i.e. Don't draw unless we get a valid value
+      if (!isnan(pc)) {
+       marker_x = (pc * widgets->profile_width) + MARGIN;
+      }
+
+      save_image_and_draw_graph_marks (image,
+                                      marker_x,
+                                      window->style->black_gc,
+                                      x_blob+MARGIN,
+                                      y_blob,
+                                      &widgets->gradient_graph_saved_img,
+                                      widgets->profile_width,
+                                      widgets->profile_height,
+                                      &widgets->is_marker_drawn,
+                                      &widgets->is_blob_drawn);
+    }
+  }
+
   // Draw speeds
   if (widgets->speed_box != NULL) {
 
@@ -1851,7 +2175,7 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     }
 
     child = gtk_container_get_children(GTK_CONTAINER(widgets->speed_box));
-    draw_vt (GTK_WIDGET(child->data), tr, widgets );
+    draw_vt (GTK_WIDGET(child->data), widgets->tr, widgets );
 
     image = GTK_WIDGET(child->data);
     g_list_free(child);
@@ -1859,12 +2183,12 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     // Ensure marker or blob are redrawn if necessary
     if (widgets->is_marker_drawn || widgets->is_blob_drawn) {
 
-      pc = tp_percentage_by_time ( tr, widgets->marker_tp );
+      pc = tp_percentage_by_time ( widgets->tr, widgets->marker_tp );
 
       gdouble x_blob = -MARGIN - 1.0; // i.e. Don't draw unless we get a valid value
       gint    y_blob = 0;
       if (widgets->is_blob_drawn) {
-       pc_blob = tp_percentage_by_time ( tr, widgets->blob_tp );
+       pc_blob = tp_percentage_by_time ( widgets->tr, widgets->blob_tp );
        if (!isnan(pc_blob)) {
          x_blob = (pc_blob * widgets->profile_width);
        }
@@ -1901,7 +2225,7 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     }
 
     child = gtk_container_get_children(GTK_CONTAINER(widgets->dist_box));
-    draw_dt (GTK_WIDGET(child->data), tr, widgets );
+    draw_dt (GTK_WIDGET(child->data), widgets->tr, widgets );
 
     image = GTK_WIDGET(child->data);
     g_list_free(child);
@@ -1909,12 +2233,12 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     // Ensure marker or blob are redrawn if necessary
     if (widgets->is_marker_drawn || widgets->is_blob_drawn) {
 
-      pc = tp_percentage_by_time ( tr, widgets->marker_tp );
+      pc = tp_percentage_by_time ( widgets->tr, widgets->marker_tp );
 
       gdouble x_blob = -MARGIN - 1.0; // i.e. Don't draw unless we get a valid value
       gint    y_blob = 0;
       if (widgets->is_blob_drawn) {
-       pc_blob = tp_percentage_by_time ( tr, widgets->blob_tp );
+       pc_blob = tp_percentage_by_time ( widgets->tr, widgets->blob_tp );
        if (!isnan(pc_blob)) {
          x_blob = (pc_blob * widgets->profile_width);
        }
@@ -1951,7 +2275,7 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     }
 
     child = gtk_container_get_children(GTK_CONTAINER(widgets->elev_time_box));
-    draw_et (GTK_WIDGET(child->data), tr, widgets );
+    draw_et (GTK_WIDGET(child->data), widgets->tr, widgets );
 
     image = GTK_WIDGET(child->data);
     g_list_free(child);
@@ -1959,12 +2283,12 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     // Ensure marker or blob are redrawn if necessary
     if (widgets->is_marker_drawn || widgets->is_blob_drawn) {
 
-      pc = tp_percentage_by_time ( tr, widgets->marker_tp );
+      pc = tp_percentage_by_time ( widgets->tr, widgets->marker_tp );
 
       gdouble x_blob = -MARGIN - 1.0; // i.e. Don't draw unless we get a valid value
       gint    y_blob = 0;
       if (widgets->is_blob_drawn) {
-       pc_blob = tp_percentage_by_time ( tr, widgets->blob_tp );
+       pc_blob = tp_percentage_by_time ( widgets->tr, widgets->blob_tp );
        if (!isnan(pc_blob)) {
          x_blob = (pc_blob * widgets->profile_width);
        }
@@ -2000,7 +2324,7 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     }
 
     child = gtk_container_get_children(GTK_CONTAINER(widgets->speed_dist_box));
-    draw_sd (GTK_WIDGET(child->data), tr, widgets );
+    draw_sd (GTK_WIDGET(child->data), widgets->tr, widgets );
 
     image = GTK_WIDGET(child->data);
     g_list_free(child);
@@ -2008,11 +2332,11 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
     // Ensure marker or blob are redrawn if necessary
     if (widgets->is_marker_drawn || widgets->is_blob_drawn) {
 
-      pc = tp_percentage_by_distance ( tr, widgets->marker_tp, widgets->track_length_inc_gaps );
+      pc = tp_percentage_by_distance ( widgets->tr, widgets->marker_tp, widgets->track_length_inc_gaps );
       gdouble x_blob = -MARGIN - 1.0; // i.e. Don't draw unless we get a valid value
       gint y_blob = 0;
       if (widgets->is_blob_drawn) {
-       pc_blob = tp_percentage_by_distance ( tr, widgets->blob_tp, widgets->track_length_inc_gaps );
+       pc_blob = tp_percentage_by_distance ( widgets->tr, widgets->blob_tp, widgets->track_length_inc_gaps );
        if (!isnan(pc_blob)) {
          x_blob = (pc_blob * widgets->profile_width);
        }
@@ -2042,10 +2366,8 @@ static void draw_all_graphs ( GtkWidget *widget, gpointer *pass_along, gboolean
 /**
  * Configure/Resize the profile & speed/time images
  */
-static gboolean configure_event ( GtkWidget *widget, GdkEventConfigure *event, gpointer *pass_along )
+static gboolean configure_event ( GtkWidget *widget, GdkEventConfigure *event, PropWidgets *widgets )
 {
-  PropWidgets *widgets = pass_along[3];
-
   if (widgets->configure_dialog) {
     // Determine size offsets between dialog size and size for images
     // Only on the initialisation of the dialog
@@ -2072,7 +2394,7 @@ static gboolean configure_event ( GtkWidget *widget, GdkEventConfigure *event, g
     return FALSE;
 
   // Draw stuff
-  draw_all_graphs ( widget, pass_along, TRUE );
+  draw_all_graphs ( widget, widgets, TRUE );
 
   return FALSE;
 }
@@ -2080,15 +2402,14 @@ static gboolean configure_event ( GtkWidget *widget, GdkEventConfigure *event, g
 /**
  * Create height profile widgets including the image and callbacks
  */
-GtkWidget *vik_trw_layer_create_profile ( GtkWidget *window, VikTrack *tr, gpointer vlp, VikViewport *vvp, PropWidgets *widgets, gdouble *min_alt, gdouble *max_alt)
+GtkWidget *vik_trw_layer_create_profile ( GtkWidget *window, PropWidgets *widgets, gdouble *min_alt, gdouble *max_alt)
 {
   GdkPixmap *pix;
   GtkWidget *image;
   GtkWidget *eventbox;
-  gpointer *pass_along;
 
   // First allocation
-  widgets->altitudes = vik_track_make_elevation_map ( tr, widgets->profile_width );
+  widgets->altitudes = vik_track_make_elevation_map ( widgets->tr, widgets->profile_width );
 
   if ( widgets->altitudes == NULL ) {
     *min_alt = *max_alt = VIK_DEFAULT_ALTITUDE;
@@ -2102,16 +2423,39 @@ GtkWidget *vik_trw_layer_create_profile ( GtkWidget *window, VikTrack *tr, gpoin
 
   g_object_unref ( G_OBJECT(pix) );
 
-  pass_along = g_malloc ( sizeof(gpointer) * 4 ); /* FIXME: mem leak -- never be freed */
-  pass_along[0] = tr;
-  pass_along[1] = vlp;
-  pass_along[2] = vvp;
-  pass_along[3] = widgets;
+  eventbox = gtk_event_box_new ();
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_profile_click), widgets );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_profile_move), widgets );
+  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 | GDK_STRUCTURE_MASK);
+
+  return eventbox;
+}
+
+/**
+ * Create height profile widgets including the image and callbacks
+ */
+GtkWidget *vik_trw_layer_create_gradient ( GtkWidget *window, PropWidgets *widgets)
+{
+  GdkPixmap *pix;
+  GtkWidget *image;
+  GtkWidget *eventbox;
+
+  // First allocation
+  widgets->gradients = vik_track_make_gradient_map ( widgets->tr, widgets->profile_width );
+
+  if ( widgets->gradients == NULL ) {
+    return NULL;
+  }
+
+  pix = gdk_pixmap_new( window->window, widgets->profile_width + MARGIN, widgets->profile_height, -1 );
+  image = gtk_image_new_from_pixmap ( pix, NULL );
+
+  g_object_unref ( G_OBJECT(pix) );
 
   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 );
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_gradient_click), widgets );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_gradient_move), widgets );
   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 | GDK_STRUCTURE_MASK);
 
@@ -2121,24 +2465,17 @@ GtkWidget *vik_trw_layer_create_profile ( GtkWidget *window, VikTrack *tr, gpoin
 /**
  * Create speed/time widgets including the image and callbacks
  */
-GtkWidget *vik_trw_layer_create_vtdiag ( GtkWidget *window, VikTrack *tr, gpointer vlp, VikViewport *vvp, PropWidgets *widgets)
+GtkWidget *vik_trw_layer_create_vtdiag ( GtkWidget *window, PropWidgets *widgets)
 {
   GdkPixmap *pix;
   GtkWidget *image;
   GtkWidget *eventbox;
-  gpointer *pass_along;
 
   // First allocation
-  widgets->speeds = vik_track_make_speed_map ( tr, widgets->profile_width );
+  widgets->speeds = vik_track_make_speed_map ( widgets->tr, widgets->profile_width );
   if ( widgets->speeds == NULL )
     return NULL;
 
-  pass_along = g_malloc ( sizeof(gpointer) * 4 ); /* FIXME: mem leak -- never be freed */
-  pass_along[0] = tr;
-  pass_along[1] = vlp;
-  pass_along[2] = vvp;
-  pass_along[3] = widgets;
-
   pix = gdk_pixmap_new( window->window, widgets->profile_width + MARGIN, widgets->profile_height, -1 );
   image = gtk_image_new_from_pixmap ( pix, NULL );
 
@@ -2168,9 +2505,8 @@ GtkWidget *vik_trw_layer_create_vtdiag ( GtkWidget *window, VikTrack *tr, gpoint
   g_object_unref ( G_OBJECT(pix) );
 
   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 );
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_vt_click), widgets );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_vt_move), widgets );
   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);
 
@@ -2180,33 +2516,26 @@ GtkWidget *vik_trw_layer_create_vtdiag ( GtkWidget *window, VikTrack *tr, gpoint
 /**
  * Create distance / time widgets including the image and callbacks
  */
-GtkWidget *vik_trw_layer_create_dtdiag ( GtkWidget *window, VikTrack *tr, gpointer vlp, VikViewport *vvp, PropWidgets *widgets)
+GtkWidget *vik_trw_layer_create_dtdiag ( GtkWidget *window, PropWidgets *widgets)
 {
   GdkPixmap *pix;
   GtkWidget *image;
   GtkWidget *eventbox;
-  gpointer *pass_along;
 
   // First allocation
-  widgets->distances = vik_track_make_distance_map ( tr, widgets->profile_width );
+  widgets->distances = vik_track_make_distance_map ( widgets->tr, widgets->profile_width );
   if ( widgets->distances == NULL )
     return NULL;
 
-  pass_along = g_malloc ( sizeof(gpointer) * 4 ); /* FIXME: mem leak -- never be freed */
-  pass_along[0] = tr;
-  pass_along[1] = vlp;
-  pass_along[2] = vvp;
-  pass_along[3] = widgets;
-
   pix = gdk_pixmap_new( window->window, widgets->profile_width + MARGIN, widgets->profile_height, -1 );
   image = gtk_image_new_from_pixmap ( pix, NULL );
 
   g_object_unref ( G_OBJECT(pix) );
 
   eventbox = gtk_event_box_new ();
-  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_dt_click), pass_along );
-  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_dt_move), pass_along );
-  g_signal_connect_swapped ( G_OBJECT(eventbox), "destroy", G_CALLBACK(g_free), pass_along );
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_dt_click), widgets );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_dt_move), widgets );
+  //g_signal_connect_swapped ( G_OBJECT(eventbox), "destroy", G_CALLBACK(g_free), widgets );
   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);
 
@@ -2216,33 +2545,25 @@ GtkWidget *vik_trw_layer_create_dtdiag ( GtkWidget *window, VikTrack *tr, gpoint
 /**
  * Create elevation / time widgets including the image and callbacks
  */
-GtkWidget *vik_trw_layer_create_etdiag ( GtkWidget *window, VikTrack *tr, gpointer vlp, VikViewport *vvp, PropWidgets *widgets)
+GtkWidget *vik_trw_layer_create_etdiag ( GtkWidget *window, PropWidgets *widgets)
 {
   GdkPixmap *pix;
   GtkWidget *image;
   GtkWidget *eventbox;
-  gpointer *pass_along;
 
   // First allocation
-  widgets->ats = vik_track_make_elevation_time_map ( tr, widgets->profile_width );
+  widgets->ats = vik_track_make_elevation_time_map ( widgets->tr, widgets->profile_width );
   if ( widgets->ats == NULL )
     return NULL;
 
-  pass_along = g_malloc ( sizeof(gpointer) * 4 ); /* FIXME: mem leak -- never be freed */
-  pass_along[0] = tr;
-  pass_along[1] = vlp;
-  pass_along[2] = vvp;
-  pass_along[3] = widgets;
-
   pix = gdk_pixmap_new( window->window, widgets->profile_width + MARGIN, widgets->profile_height, -1 );
   image = gtk_image_new_from_pixmap ( pix, NULL );
 
   g_object_unref ( G_OBJECT(pix) );
 
   eventbox = gtk_event_box_new ();
-  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_et_click), pass_along );
-  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_et_move), pass_along );
-  g_signal_connect_swapped ( G_OBJECT(eventbox), "destroy", G_CALLBACK(g_free), pass_along );
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_et_click), widgets );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_et_move), widgets );
   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);
 
@@ -2252,33 +2573,25 @@ GtkWidget *vik_trw_layer_create_etdiag ( GtkWidget *window, VikTrack *tr, gpoint
 /**
  * Create speed/distance widgets including the image and callbacks
  */
-GtkWidget *vik_trw_layer_create_sddiag ( GtkWidget *window, VikTrack *tr, gpointer vlp, VikViewport *vvp, PropWidgets *widgets)
+GtkWidget *vik_trw_layer_create_sddiag ( GtkWidget *window, PropWidgets *widgets)
 {
   GdkPixmap *pix;
   GtkWidget *image;
   GtkWidget *eventbox;
-  gpointer *pass_along;
 
   // First allocation
-  widgets->speeds_dist = vik_track_make_speed_dist_map ( tr, widgets->profile_width );
+  widgets->speeds_dist = vik_track_make_speed_dist_map ( widgets->tr, widgets->profile_width );
   if ( widgets->speeds_dist == NULL )
     return NULL;
 
-  pass_along = g_malloc ( sizeof(gpointer) * 4 ); /* FIXME: mem leak -- never be freed */
-  pass_along[0] = tr;
-  pass_along[1] = vlp;
-  pass_along[2] = vvp;
-  pass_along[3] = widgets;
-
   pix = gdk_pixmap_new( window->window, widgets->profile_width + MARGIN, widgets->profile_height, -1 );
   image = gtk_image_new_from_pixmap ( pix, NULL );
 
   g_object_unref ( G_OBJECT(pix) );
 
   eventbox = gtk_event_box_new ();
-  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_sd_click), pass_along );
-  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_sd_move), pass_along );
-  g_signal_connect_swapped ( G_OBJECT(eventbox), "destroy", G_CALLBACK(g_free), pass_along );
+  g_signal_connect ( G_OBJECT(eventbox), "button_press_event", G_CALLBACK(track_sd_click), widgets );
+  g_signal_connect ( G_OBJECT(eventbox), "motion_notify_event", G_CALLBACK(track_sd_move), widgets );
   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);
 
@@ -2286,7 +2599,12 @@ GtkWidget *vik_trw_layer_create_sddiag ( GtkWidget *window, VikTrack *tr, gpoint
 }
 #undef MARGIN
 
-static void propwin_response_cb( GtkDialog *dialog, gint resp, PropWidgets *widgets)
+static void destroy_cb ( GtkDialog *dialog, PropWidgets *widgets )
+{
+  prop_widgets_free(widgets);
+}
+
+static void propwin_response_cb( GtkDialog *dialog, gint resp, PropWidgets *widgets )
 {
   VikTrack *tr = widgets->tr;
   VikTrwLayer *vtl = widgets->vtl;
@@ -2300,53 +2618,54 @@ static void propwin_response_cb( GtkDialog *dialog, gint resp, PropWidgets *widg
       break;
     case GTK_RESPONSE_ACCEPT:
       vik_track_set_comment(tr, gtk_entry_get_text(GTK_ENTRY(widgets->w_comment)));
+      vik_track_set_description(tr, gtk_entry_get_text(GTK_ENTRY(widgets->w_description)));
+      gtk_color_button_get_color ( GTK_COLOR_BUTTON(widgets->w_color), &(tr->color) );
+      trw_layer_update_treeview ( widgets->vtl, widgets->tr, widgets->trk_id );
+      vik_layer_emit_update ( VIK_LAYER(vtl) );
       break;
     case VIK_TRW_LAYER_PROPWIN_REVERSE:
       vik_track_reverse(tr);
-      vik_layer_emit_update ( VIK_LAYER(vtl), FALSE );
+      vik_layer_emit_update ( VIK_LAYER(vtl) );
       break;
     case VIK_TRW_LAYER_PROPWIN_DEL_DUP:
-      vik_track_remove_dup_points(tr);
+      vik_track_remove_dup_points(tr); // NB ignore the returned answer
+      // As we could have seen the nuber of dulplicates that would be deleted in the properties statistics tab,
+      //   choose not to inform the user unnecessarily
+
       /* 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), FALSE );
+      trw_layer_cancel_tps_of_track ( vtl, tr );
+      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. */
+        /* get new tracks, add them and then the delete old one. 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_yes_or_no ( 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), NULL );
-            g_free ( new_tr_name );
-            if (new_new_tr_name)
-              new_tr_name = new_new_tr_name;
+          if ( tracks[i] ) {
+           new_tr_name = trw_layer_new_unique_sublayer_name ( vtl,
+                                                               widgets->tr->is_route ? VIK_TRW_LAYER_SUBLAYER_ROUTE : VIK_TRW_LAYER_SUBLAYER_TRACK,
+                                                               widgets->tr->name);
+            if ( widgets->tr->is_route )
+              vik_trw_layer_add_route ( vtl, new_tr_name, tracks[i] );
             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] );
+              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), FALSE ); /* chase thru the hoops */
+          if ( widgets->tr->is_route )
+            vik_trw_layer_delete_route ( vtl, tr );
+          else
+            vik_trw_layer_delete_track ( vtl, tr );
+          vik_layer_emit_update ( VIK_LAYER(vtl) ); /* chase thru the hoops */
         }
       }
       break;
@@ -2364,33 +2683,23 @@ static void propwin_response_cb( GtkDialog *dialog, gint resp, PropWidgets *widg
           break;
         }
 
-        gchar *r_name = g_strdup_printf("%s #2", widgets->track_name);
-        if (vik_trw_layer_get_track(vtl, r_name ) && 
-             ( ! a_dialog_yes_or_no( 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), NULL );
-            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;
-            }
-        }
+        gchar *r_name = trw_layer_new_unique_sublayer_name(vtl,
+                                                           widgets->tr->is_route ? VIK_TRW_LAYER_SUBLAYER_ROUTE : VIK_TRW_LAYER_SUBLAYER_TRACK,
+                                                           widgets->tr->name);
         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->is_route = tr->is_route;
         tr_right->trackpoints = iter;
 
-        vik_trw_layer_add_track(vtl, r_name, tr_right);
-        vik_layer_emit_update ( VIK_LAYER(vtl), FALSE );
+        if ( widgets->tr->is_route )
+          vik_trw_layer_add_route(vtl, r_name, tr_right);
+        else
+          vik_trw_layer_add_track(vtl, r_name, tr_right);
+        vik_layer_emit_update ( VIK_LAYER(vtl) );
       }
       break;
     default:
@@ -2400,7 +2709,6 @@ static void propwin_response_cb( GtkDialog *dialog, gint resp, PropWidgets *widg
 
   /* 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) );
   }
@@ -2409,12 +2717,11 @@ static void propwin_response_cb( GtkDialog *dialog, gint resp, PropWidgets *widg
 /**
  * Force a redraw when checkbutton has been toggled to show/hide that information
  */
-static void checkbutton_toggle_cb ( GtkToggleButton *togglebutton, gpointer *pass_along, gpointer dummy)
+static void checkbutton_toggle_cb ( GtkToggleButton *togglebutton, PropWidgets *widgets, gpointer dummy )
 {
-  PropWidgets *widgets = pass_along[3];
   // Even though not resized, we'll pretend it is -
   //  as this invalidates the saved images (since the image may have changed)
-  draw_all_graphs ( widgets->dialog, pass_along, TRUE);
+  draw_all_graphs ( widgets->dialog, widgets, TRUE );
 }
 
 /**
@@ -2454,16 +2761,22 @@ static GtkWidget *create_graph_page ( GtkWidget *graph,
   return vbox;
 }
 
-void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *tr, gpointer vlp, gchar *track_name, VikViewport *vvp )
+void vik_trw_layer_propwin_run ( GtkWindow *parent,
+                                 VikTrwLayer *vtl,
+                                 VikTrack *tr,
+                                 gpointer vlp,
+                                 VikViewport *vvp,
+                                 gpointer *trk_id )
 {
-  /* FIXME: free widgets when destroy signal received */
   PropWidgets *widgets = prop_widgets_new();
   widgets->vtl = vtl;
+  widgets->vvp = vvp;
+  widgets->vlp = vlp;
   widgets->tr = tr;
+  widgets->trk_id = trk_id;
   widgets->profile_width  = PROPWIN_PROFILE_WIDTH;
   widgets->profile_height = PROPWIN_PROFILE_HEIGHT;
-  widgets->track_name = track_name;
-  gchar *title = g_strdup_printf(_("%s - Track Properties"), track_name);
+  gchar *title = g_strdup_printf(_("%s - Track Properties"), tr->name);
   GtkWidget *dialog = gtk_dialog_new_with_buttons (title,
                          parent,
                          GTK_DIALOG_DESTROY_WITH_PARENT | GTK_DIALOG_NO_SEPARATOR,
@@ -2475,25 +2788,44 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
                          GTK_STOCK_OK,     GTK_RESPONSE_ACCEPT,
                          NULL);
   widgets->dialog = dialog;
+  g_signal_connect( G_OBJECT(dialog), "response", G_CALLBACK(propwin_response_cb), widgets);
+
   g_free(title);
-  g_signal_connect(dialog, "response", G_CALLBACK(propwin_response_cb), widgets);
   GtkTable *table;
   gdouble tr_len;
-  guint32 tp_count, seg_count;
+  gulong tp_count;
+  guint seg_count;
 
   gdouble min_alt, max_alt;
-  widgets->elev_box = vik_trw_layer_create_profile(GTK_WIDGET(parent), tr, vlp, vvp, widgets, &min_alt, &max_alt);
-  widgets->speed_box = vik_trw_layer_create_vtdiag(GTK_WIDGET(parent), tr, vlp, vvp, widgets);
-  widgets->dist_box = vik_trw_layer_create_dtdiag(GTK_WIDGET(parent), tr, vlp, vvp, widgets);
-  widgets->elev_time_box = vik_trw_layer_create_etdiag(GTK_WIDGET(parent), tr, vlp, vvp, widgets);
-  widgets->speed_dist_box = vik_trw_layer_create_sddiag(GTK_WIDGET(parent), tr, vlp, vvp, widgets);
+  widgets->elev_box = vik_trw_layer_create_profile(GTK_WIDGET(parent), widgets, &min_alt, &max_alt);
+  widgets->gradient_box = vik_trw_layer_create_gradient(GTK_WIDGET(parent), widgets);
+  widgets->speed_box = vik_trw_layer_create_vtdiag(GTK_WIDGET(parent), widgets);
+  widgets->dist_box = vik_trw_layer_create_dtdiag(GTK_WIDGET(parent), widgets);
+  widgets->elev_time_box = vik_trw_layer_create_etdiag(GTK_WIDGET(parent), widgets);
+  widgets->speed_dist_box = vik_trw_layer_create_sddiag(GTK_WIDGET(parent), widgets);
   GtkWidget *graphs = gtk_notebook_new();
 
   GtkWidget *content[20];
   int cnt;
   int i;
 
-  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 *label_texts[] = {
+    N_("<b>Comment:</b>"),
+    N_("<b>Description:</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>Moving 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>"),
+    N_("<b>Color:</b>") };
   static gchar tmp_buf[50];
   gdouble tmp_speed;
 
@@ -2504,6 +2836,12 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
   g_signal_connect_swapped ( widgets->w_comment, "activate", G_CALLBACK(a_dialog_response_accept), GTK_DIALOG(dialog) );
   content[cnt++] = widgets->w_comment;
 
+  widgets->w_description = gtk_entry_new ();
+  if ( tr->description )
+    gtk_entry_set_text ( GTK_ENTRY(widgets->w_description), tr->description );
+  g_signal_connect_swapped ( widgets->w_description, "activate", G_CALLBACK(a_dialog_response_accept), GTK_DIALOG(dialog) );
+  content[cnt++] = widgets->w_description;
+
   vik_units_distance_t dist_units = a_vik_get_units_distance ();
 
   // NB This value not shown yet - but is used by internal calculations
@@ -2523,7 +2861,7 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
   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 );
+  g_snprintf(tmp_buf, sizeof(tmp_buf), "%lu", tp_count );
   widgets->w_tp_count = content[cnt++] = gtk_label_new ( tmp_buf );
 
   seg_count = vik_track_get_segment_count(tr) ;
@@ -2582,6 +2920,34 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
   }
   widgets->w_avg_speed = content[cnt++] = gtk_label_new ( tmp_buf );
 
+  // Use 60sec as the default period to be considered stopped
+  //  this is the TrackWaypoint draw stops default value 'vtl->stop_length'
+  //  however this variable is not directly accessible - and I don't expect it's often changed from the default
+  //  so ATM just put in the number
+  tmp_speed = vik_track_get_average_speed_moving(tr, 60);
+  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", VIK_MPS_TO_KPH(tmp_speed));
+      break;
+    case VIK_UNITS_SPEED_MILES_PER_HOUR:
+      g_snprintf(tmp_buf, sizeof(tmp_buf), "%.2f mph", VIK_MPS_TO_MPH(tmp_speed));
+      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", VIK_MPS_TO_KNOTS(tmp_speed));
+      break;
+    default:
+      g_snprintf (tmp_buf, sizeof(tmp_buf), "--" );
+      g_critical("Houston, we've had a problem. speed=%d", speed_units);
+    }
+  }
+  widgets->w_mvg_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
@@ -2670,6 +3036,8 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
     widgets->w_time_dur = content[cnt++] = gtk_label_new(_("No Data"));
   }
 
+  widgets->w_color = content[cnt++] = gtk_color_button_new_with_color ( &(tr->color) );
+
   table = GTK_TABLE(gtk_table_new (cnt, 2, FALSE));
   gtk_table_set_col_spacing (table, 0, 10);
   for (i=0; i<cnt; i++) {
@@ -2689,13 +3057,6 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
 
   gtk_notebook_append_page(GTK_NOTEBOOK(graphs), GTK_WIDGET(table), gtk_label_new(_("Statistics")));
 
-  gpointer *pass_along;
-  pass_along = g_malloc ( sizeof(gpointer) * 4 ); /* FIXME: mem leak -- never be freed */
-  pass_along[0] = tr;
-  pass_along[1] = vlp;
-  pass_along[2] = vvp;
-  pass_along[3] = widgets;
-
   if ( widgets->elev_box ) {
     GtkWidget *page = NULL;
     widgets->w_cur_dist = gtk_label_new(_("No Data"));
@@ -2707,11 +3068,25 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
                              _("<b>Track Height:</b>"), widgets->w_cur_elevation,
                              widgets->w_show_dem, TRUE,
                              widgets->w_show_alt_gps_speed, TRUE);
-    g_signal_connect (widgets->w_show_dem, "toggled", G_CALLBACK (checkbutton_toggle_cb), pass_along);
-    g_signal_connect (widgets->w_show_alt_gps_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), pass_along);
+    g_signal_connect (widgets->w_show_dem, "toggled", G_CALLBACK (checkbutton_toggle_cb), widgets);
+    g_signal_connect (widgets->w_show_alt_gps_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), widgets);
     gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Elevation-distance")));
   }
 
+  if ( widgets->gradient_box ) {
+    GtkWidget *page = NULL;
+    widgets->w_cur_gradient_dist = gtk_label_new(_("No Data"));
+    widgets->w_cur_gradient_gradient = gtk_label_new(_("No Data"));
+    widgets->w_show_gradient_gps_speed = gtk_check_button_new_with_mnemonic(_("Show _GPS Speed"));
+    page = create_graph_page (widgets->gradient_box,
+                             _("<b>Track Distance:</b>"), widgets->w_cur_gradient_dist,
+                             _("<b>Track Gradient:</b>"), widgets->w_cur_gradient_gradient,
+                             widgets->w_show_gradient_gps_speed, TRUE,
+                  NULL, FALSE);
+    g_signal_connect (widgets->w_show_gradient_gps_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), widgets);
+    gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Gradient-distance")));
+  }
+
   if ( widgets->speed_box ) {
     GtkWidget *page = NULL;
     widgets->w_cur_time = gtk_label_new(_("No Data"));
@@ -2722,7 +3097,7 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
                              _("<b>Track Speed:</b>"), widgets->w_cur_speed,
                              widgets->w_show_gps_speed, TRUE,
                              NULL, FALSE);
-    g_signal_connect (widgets->w_show_gps_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), pass_along);
+    g_signal_connect (widgets->w_show_gps_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), widgets);
     gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Speed-time")));
   }
 
@@ -2736,7 +3111,7 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
                              _("<b>Track Time:</b>"), widgets->w_cur_dist_time,
                              widgets->w_show_dist_speed, FALSE,
                              NULL, FALSE);
-    g_signal_connect (widgets->w_show_dist_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), pass_along);
+    g_signal_connect (widgets->w_show_dist_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), widgets);
     gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Distance-time")));
   }
 
@@ -2750,7 +3125,7 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
                              _("<b>Track Height:</b>"), widgets->w_cur_elev_elev,
                              widgets->w_show_elev_speed, FALSE,
                              NULL, FALSE);
-    g_signal_connect (widgets->w_show_elev_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), pass_along);
+    g_signal_connect (widgets->w_show_elev_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), widgets);
     gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Elevation-time")));
   }
 
@@ -2764,7 +3139,7 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
                              _("<b>Track Speed:</b>"), widgets->w_cur_speed_speed,
                              widgets->w_show_sd_gps_speed, TRUE,
                              NULL, FALSE);
-    g_signal_connect (widgets->w_show_sd_gps_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), pass_along);
+    g_signal_connect (widgets->w_show_sd_gps_speed, "toggled", G_CALLBACK (checkbutton_toggle_cb), widgets);
     gtk_notebook_append_page(GTK_NOTEBOOK(graphs), page, gtk_label_new(_("Speed-distance")));
   }
 
@@ -2778,9 +3153,31 @@ void vik_trw_layer_propwin_run ( GtkWindow *parent, VikTrwLayer *vtl, VikTrack *
 
   // On dialog realization configure_event casues the graphs to be initially drawn
   widgets->configure_dialog = TRUE;
-  g_signal_connect ( G_OBJECT(dialog), "configure-event", G_CALLBACK (configure_event), pass_along);
+  g_signal_connect ( G_OBJECT(dialog), "configure-event", G_CALLBACK (configure_event), widgets );
+
+  g_signal_connect ( G_OBJECT(dialog), "destroy", G_CALLBACK (destroy_cb), widgets );
 
   vik_track_set_property_dialog(tr, dialog);
   gtk_dialog_set_default_response ( GTK_DIALOG(dialog), GTK_RESPONSE_ACCEPT );
   gtk_widget_show_all ( dialog );
 }
+
+
+/**
+ * Update this property dialog
+ * e.g. if the track has been renamed
+ */
+void vik_trw_layer_propwin_update ( VikTrack *trk )
+{
+  // If not displayed do nothing
+  if ( !trk->property_dialog )
+    return;
+
+  // Update title with current name
+  if ( trk->name ) {
+    gchar *title = g_strdup_printf ( _("%s - Track Properties"), trk->name );
+    gtk_window_set_title ( GTK_WINDOW(trk->property_dialog), title );
+    g_free(title);
+  }
+
+}