src/viktrwlayer_waypointlist.c

   1 /* -*- Mode: C; indent-tabs-mode: t; c-basic-offset: 4; tab-width: 4 -*- */
   2 /*
   3  * viking -- GPS Data and Topo Analyzer, Explorer, and Manager
   4  *
   5  * Copyright (C) 2013, Rob Norris <rw_norris@hotmail.com>
   6  *
   7  * This program is free software; you can redistribute it and/or modify
   8  * it under the terms of the GNU General Public License as published by
   9  * the Free Software Foundation; either version 2 of the License, or
  10  * (at your option) any later version.
  11  *
  12  * This program is distributed in the hope that it will be useful,
  13  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  14  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  15  * GNU General Public License for more details.
  16  *
  17  * You should have received a copy of the GNU General Public License
  18  * along with this program; if not, write to the Free Software
  19  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  20  *
  21  */
  22 #include <math.h>
  23 #include <string.h>
  24 #include <stdlib.h>
  25 #include <stdio.h>
  26 #include <glib.h>
  27 #include <glib/gstdio.h>
  28 #include <glib/gi18n.h>
  29
  30 #include "viking.h"
  31 #include "viktrwlayer_waypointlist.h"
  32
  33 // Long formatted date+basic time - listing this way ensures the string comparison sort works - so no local type format %x or %c here!
  34 #define WAYPOINT_LIST_DATE_FORMAT "%Y-%m-%d %H:%M"
  35
  36 /**
  37  * waypoint_close_cb:
  38  *
  39  */
  40 static void waypoint_close_cb ( GtkWidget *dialog, gint resp, GList *data )
  41 {
  42         g_list_foreach ( data, (GFunc) g_free, NULL );
  43         g_list_free ( data );
  44
  45         gtk_widget_destroy (dialog);
  46 }
  47
  48 /**
  49  * format_1f_cell_data_func:
  50  *
  51  * General purpose column double formatting
  52  *
  53 static void format_1f_cell_data_func ( GtkTreeViewColumn *col,
  54                                        GtkCellRenderer   *renderer,
  55                                        GtkTreeModel      *model,
  56                                        GtkTreeIter       *iter,
  57                                        gpointer           user_data )
  58 {
  59         gdouble value;
  60         gchar buf[20];
  61         gint column = GPOINTER_TO_INT (user_data);
  62         gtk_tree_model_get ( model, iter, column, &value, -1 );
  63         g_snprintf ( buf, sizeof(buf), "%.1f", value );
  64         g_object_set ( renderer, "text", buf, NULL );
  65 }
  66  */
  67
  68 #define WPT_LIST_COLS 9
  69 #define WPT_COL_NUM WPT_LIST_COLS-1
  70 #define TRW_COL_NUM WPT_COL_NUM-1
  71
  72 /*
  73  * trw_layer_waypoint_tooltip_cb:
  74  *
  75  * Show a tooltip when the mouse is over a waypoint list entry.
  76  * The tooltip contains the description.
  77  */
  78 static gboolean trw_layer_waypoint_tooltip_cb ( GtkWidget  *widget,
  79                                                 gint        x,
  80                                                 gint        y,
  81                                                 gboolean    keyboard_tip,
  82                                                 GtkTooltip *tooltip,
  83                                                 gpointer    data )
  84 {
  85         GtkTreeIter iter;
  86         GtkTreePath *path = NULL;
  87         GtkTreeView *tree_view = GTK_TREE_VIEW (widget);
  88         GtkTreeModel *model = gtk_tree_view_get_model (tree_view);
  89
  90         if ( !gtk_tree_view_get_tooltip_context ( tree_view, &x, &y,
  91                                                   keyboard_tip,
  92                                                   &model, &path, &iter ) )
  93                 return FALSE;
  94
  95         VikWaypoint *wpt;
  96         gtk_tree_model_get ( model, &iter, WPT_COL_NUM, &wpt, -1 );
  97         if ( !wpt ) return FALSE;
  98
  99         gboolean tooltip_set = TRUE;
 100         if ( wpt->description )
 101                 gtk_tooltip_set_text ( tooltip, wpt->description );
 102         else
 103                 tooltip_set = FALSE;
 104
 105         if ( tooltip_set )
 106                 gtk_tree_view_set_tooltip_row ( tree_view, tooltip, path );
 107
 108         gtk_tree_path_free ( path );
 109
 110         return tooltip_set;
 111 }
 112
 113 /*
 114 static void trw_layer_waypoint_select_cb ( GtkTreeSelection *selection, gpointer data )
 115 {
 116         GtkTreeIter iter;
 117         if ( !gtk_tree_selection_get_selected (selection, NULL, &iter) )
 118                 return;
 119
 120         GtkTreeView *tree_view = GTK_TREE_VIEW ( data );
 121         GtkTreeModel *model = gtk_tree_view_get_model (tree_view);
 122
 123         VikWaypoint *wpt;
 124         gtk_tree_model_get ( model, &iter, WPT_COL_NUM, &wpt, -1 );
 125         if ( !wpt ) return;
 126
 127         VikTrwLayer *vtl;
 128         gtk_tree_model_get ( model, &iter, TRW_COL_NUM, &vtl, -1 );
 129         if ( !IS_VIK_TRW_LAYER(vtl) ) return;
 130
 131         //vik_treeview_select_iter ( VIK_LAYER(vtl)->vt, g_hash_table_lookup ( vtl->waypoint_iters, uuid ), TRUE );
 132 }
 133 */
 134
 135 // A slightly better way of defining the menu callback information
 136 // This should be much easier to extend/rework compared to the current trw_layer menus
 137 typedef enum {
 138   MA_VTL = 0,
 139   MA_WPT,
 140   MA_WPT_UUID,
 141   MA_VVP,
 142   MA_TREEVIEW,
 143   MA_WPTS_LIST,
 144   MA_LAST
 145 } menu_array_index;
 146
 147 typedef gpointer menu_array_values[MA_LAST];
 148
 149 // Instead of hooking automatically on treeview item selection
 150 // This is performed on demand via the specific menu request
 151 static void trw_layer_waypoint_select ( menu_array_values values )
 152 {
 153         VikTrwLayer *vtl = VIK_TRW_LAYER(values[MA_VTL]);
 154
 155         if ( values[MA_WPT_UUID] ) {
 156                 GtkTreeIter *iter = NULL;
 157                 iter = g_hash_table_lookup ( vik_trw_layer_get_waypoints_iters(vtl), values[MA_WPT_UUID] );
 158
 159                 if ( iter )
 160                         vik_treeview_select_iter ( VIK_LAYER(vtl)->vt, iter, TRUE );
 161         }
 162 }
 163
 164 static void trw_layer_waypoint_properties ( menu_array_values values )
 165 {
 166         VikTrwLayer *vtl = VIK_TRW_LAYER(values[MA_VTL]);
 167         VikWaypoint *wpt = VIK_WAYPOINT(values[MA_WPT]);
 168
 169         if ( wpt && wpt->name ) {
 170                 // Kill off this dialog to allow interaction with properties window
 171                 //  since the properties also allows waypoint manipulations it won't cause conflicts here.
 172                 GtkWidget *gw = gtk_widget_get_toplevel ( values[MA_TREEVIEW] );
 173                 waypoint_close_cb ( gw, 0, values[MA_WPTS_LIST] );
 174
 175                 gboolean updated = FALSE;
 176                 gchar *new_name = a_dialog_waypoint ( VIK_GTK_WINDOW_FROM_LAYER(vtl), wpt->name, vtl, wpt, vik_trw_layer_get_coord_mode(vtl), FALSE, &updated );
 177                 if ( new_name )
 178                         trw_layer_waypoint_rename ( vtl, wpt, new_name );
 179
 180                 if ( updated )
 181                         trw_layer_waypoint_reset_icon ( vtl, wpt );
 182
 183                 if ( updated && VIK_LAYER(vtl)->visible )
 184                         vik_layer_emit_update ( VIK_LAYER(vtl) );
 185         }
 186 }
 187
 188 static void trw_layer_waypoint_view ( menu_array_values values )
 189 {
 190         VikTrwLayer *vtl = VIK_TRW_LAYER(values[MA_VTL]);
 191         VikWaypoint *wpt = VIK_WAYPOINT(values[MA_WPT]);
 192         VikViewport *vvp = VIK_VIEWPORT(values[MA_VVP]);
 193
 194         vik_viewport_set_center_coord ( vvp, &(wpt->coord) );
 195
 196         trw_layer_waypoint_select (values);
 197
 198         vik_layer_emit_update ( VIK_LAYER(vtl) );
 199 }
 200
 201 static void trw_layer_show_picture ( menu_array_values values )
 202 {
 203         VikWaypoint *wpt = VIK_WAYPOINT(values[MA_WPT]);
 204 #ifdef WINDOWS
 205         ShellExecute(NULL, "open", wpt->image, NULL, NULL, SW_SHOWNORMAL);
 206 #else
 207         VikTrwLayer *vtl = VIK_TRW_LAYER(values[MA_VTL]);
 208         GError *err = NULL;
 209         gchar *quoted_file = g_shell_quote ( wpt->image );
 210         gchar *cmd = g_strdup_printf ( "%s %s", a_vik_get_image_viewer(), quoted_file );
 211         g_free ( quoted_file );
 212         if ( ! g_spawn_command_line_async ( cmd, &err ) ) {
 213                 a_dialog_error_msg_extra ( VIK_GTK_WINDOW_FROM_LAYER(vtl), _("Could not launch %s to open file."), a_vik_get_image_viewer() );
 214                 g_error_free ( err );
 215         }
 216         g_free ( cmd );
 217 #endif
 218 }
 219
 220
 221 static gboolean add_menu_items ( GtkMenu *menu, VikTrwLayer *vtl, VikWaypoint *wpt, gpointer wpt_uuid, VikViewport *vvp, GtkWidget *tree_view, gpointer data )
 222 {
 223         static menu_array_values values;
 224         GtkWidget *item;
 225
 226         values[MA_VTL]       = vtl;
 227         values[MA_WPT]       = wpt;
 228         values[MA_WPT_UUID]  = wpt_uuid;
 229         values[MA_VVP]       = vvp;
 230         values[MA_TREEVIEW]  = tree_view;
 231         values[MA_WPTS_LIST] = data;
 232
 233         /*
 234         item = gtk_image_menu_item_new_with_mnemonic ( _("_Select") );
 235         gtk_image_menu_item_set_image ( (GtkImageMenuItem*)item, gtk_image_new_from_stock (GTK_STOCK_FIND, GTK_ICON_SIZE_MENU) );
 236         g_signal_connect_swapped ( G_OBJECT(item), "activate", G_CALLBACK(trw_layer_waypoint_select), values );
 237         gtk_menu_shell_append (GTK_MENU_SHELL (menu), item);
 238         gtk_widget_show ( item );
 239         */
 240
 241         // AUTO SELECT NOT TRUE YET...
 242         // ATM view auto selects, so don't bother with separate select menu entry
 243         item = gtk_image_menu_item_new_with_mnemonic ( _("_View") );
 244         gtk_image_menu_item_set_image ( (GtkImageMenuItem*)item, gtk_image_new_from_stock (GTK_STOCK_ZOOM_FIT, GTK_ICON_SIZE_MENU) );
 245         g_signal_connect_swapped ( G_OBJECT(item), "activate", G_CALLBACK(trw_layer_waypoint_view), values );
 246         gtk_menu_shell_append (GTK_MENU_SHELL (menu), item);
 247         gtk_widget_show ( item );
 248
 249         item = gtk_image_menu_item_new_from_stock ( GTK_STOCK_PROPERTIES, NULL );
 250         g_signal_connect_swapped ( G_OBJECT(item), "activate", G_CALLBACK(trw_layer_waypoint_properties), values );
 251         gtk_menu_shell_append ( GTK_MENU_SHELL(menu), item );
 252         gtk_widget_show ( item );
 253
 254         item = gtk_image_menu_item_new_with_mnemonic ( _("_Show Picture...") );
 255         gtk_image_menu_item_set_image ( (GtkImageMenuItem*)item, gtk_image_new_from_stock ("vik-icon-Show Picture", GTK_ICON_SIZE_MENU) ); // Own icon - see stock_icons in vikwindow.c
 256         g_signal_connect_swapped ( G_OBJECT(item), "activate", G_CALLBACK(trw_layer_show_picture), values );
 257         gtk_menu_shell_append ( GTK_MENU_SHELL(menu), item );
 258         gtk_widget_show ( item );
 259         gtk_widget_set_sensitive ( item, GPOINTER_TO_INT(wpt->image) );
 260
 261         return TRUE;
 262 }
 263
 264 static gboolean trw_layer_waypoint_menu_popup ( GtkWidget *tree_view,
 265                                                 GdkEventButton *event,
 266                                                 gpointer data )
 267 {
 268         static GtkTreeIter iter;
 269
 270         // Use selected item to get a single iterator ref
 271         // This relies on an row being selected as part of the right click
 272         GtkTreeSelection *selection = gtk_tree_view_get_selection ( GTK_TREE_VIEW(tree_view) );
 273         if ( gtk_tree_selection_count_selected_rows (selection) != 1 )
 274                 return FALSE;
 275
 276         GtkTreePath *path;
 277         GtkTreeModel *model = gtk_tree_view_get_model ( GTK_TREE_VIEW(tree_view) );
 278
 279         // All this just to get the iter
 280         if ( gtk_tree_view_get_path_at_pos ( GTK_TREE_VIEW(tree_view),
 281                                              (gint) event->x,
 282                                              (gint) event->y,
 283                                              &path, NULL, NULL, NULL)) {
 284                 gtk_tree_model_get_iter_from_string ( model, &iter, gtk_tree_path_to_string (path) );
 285                 gtk_tree_path_free ( path );
 286         }
 287         else
 288                 return FALSE;
 289
 290         VikWaypoint *wpt;
 291         gtk_tree_model_get ( model, &iter, WPT_COL_NUM, &wpt, -1 );
 292         if ( !wpt ) return FALSE;
 293
 294         VikTrwLayer *vtl;
 295         gtk_tree_model_get ( model, &iter, TRW_COL_NUM, &vtl, -1 );
 296         if ( !IS_VIK_TRW_LAYER(vtl) ) return FALSE;
 297
 298         wpu_udata udataU;
 299         udataU.wp   = wpt;
 300         udataU.uuid = NULL;
 301
 302         gpointer *wptf;
 303         wptf = g_hash_table_find ( vik_trw_layer_get_waypoints(vtl), (GHRFunc) trw_layer_waypoint_find_uuid, &udataU );
 304
 305         if ( wptf && udataU.uuid ) {
 306                 VikViewport *vvp = vik_window_viewport((VikWindow *)(VIK_GTK_WINDOW_FROM_LAYER(vtl)));
 307
 308                 GtkWidget *menu = gtk_menu_new();
 309
 310                 // Originally started to reuse the trw_layer menu items
 311                 //  however these offer too many ways to edit the waypoint data
 312                 //  so without an easy way to distinguish read only operations,
 313                 //  create a very minimal new set of operations
 314                 add_menu_items ( GTK_MENU(menu),
 315                                  vtl,
 316                                  wpt,
 317                                  udataU.uuid,
 318                                  vvp,
 319                                  tree_view,
 320                                  data );
 321
 322                 gtk_menu_popup ( GTK_MENU(menu), NULL, NULL, NULL, NULL, event->button, gtk_get_current_event_time() );
 323                 return TRUE;
 324         }
 325         return FALSE;
 326 }
 327
 328 static gboolean trw_layer_waypoint_button_pressed ( GtkWidget *tree_view,
 329                                                     GdkEventButton *event,
 330                                                     gpointer data )
 331 {
 332         // Only on right clicks...
 333         if ( ! (event->type == GDK_BUTTON_PRESS && event->button == 3) )
 334                 return FALSE;
 335
 336         // ATM Force a selection...
 337         GtkTreeSelection *selection = gtk_tree_view_get_selection ( GTK_TREE_VIEW(tree_view) );
 338         if ( gtk_tree_selection_count_selected_rows (selection) <= 1 ) {
 339                 GtkTreePath *path;
 340                 /* Get tree path for row that was clicked */
 341                 if ( gtk_tree_view_get_path_at_pos ( GTK_TREE_VIEW(tree_view),
 342                                                      (gint) event->x,
 343                                                     (gint) event->y,
 344                                                      &path, NULL, NULL, NULL)) {
 345                         gtk_tree_selection_unselect_all ( selection );
 346                         gtk_tree_selection_select_path ( selection, path );
 347                         gtk_tree_path_free ( path );
 348                 }
 349         }
 350         return trw_layer_waypoint_menu_popup ( tree_view, event, data );
 351 }
 352
 353 /*
 354  * Foreach entry we copy the various individual waypoint properties into the tree store
 355  *  formatting & converting the internal values into something for display
 356  */
 357 static void trw_layer_waypoint_list_add ( vik_trw_waypoint_list_t *vtdl,
 358                                           GtkTreeStore *store,
 359                                           vik_units_distance_t dist_units,
 360                                           vik_units_height_t height_units )
 361 {
 362         GtkTreeIter t_iter;
 363         VikWaypoint *wpt = vtdl->wpt;
 364         VikTrwLayer *vtl = vtdl->vtl;
 365
 366         // Get start date
 367         gchar time_buf[32];
 368         time_buf[0] = '\0';
 369         if ( wpt->has_timestamp ) {
 370
 371 #if GLIB_CHECK_VERSION(2,26,0)
 372                 GDateTime* gdt = g_date_time_new_from_unix_utc ( wpt->timestamp );
 373                 gchar *time = g_date_time_format ( gdt, WAYPOINT_LIST_DATE_FORMAT );
 374                 strncpy ( time_buf, time, sizeof(time_buf) );
 375                 g_free ( time );
 376                 g_date_time_unref ( gdt);
 377 #else
 378                 GDate* gdate_start = g_date_new ();
 379                 g_date_set_time_t ( gdate_start, wpt->timestamp );
 380                 g_date_strftime ( time_buf, sizeof(time_buf), WAYPOINT_LIST_DATE_FORMAT, gdate_start );
 381                 g_date_free ( gdate_start );
 382 #endif
 383         }
 384
 385         // NB: doesn't include aggegrate visibility
 386         gboolean visible = VIK_LAYER(vtl)->visible && wpt->visible;
 387         visible = visible && vik_trw_layer_get_waypoints_visibility(vtl);
 388
 389         gdouble alt = wpt->altitude;
 390         switch (height_units) {
 391         case VIK_UNITS_HEIGHT_FEET: alt = VIK_METERS_TO_FEET(alt); break;
 392         default:
 393                 // VIK_UNITS_HEIGHT_METRES: no need to convert
 394                 break;
 395         }
 396
 397         gtk_tree_store_append ( store, &t_iter, NULL );
 398         gtk_tree_store_set ( store, &t_iter,
 399                              0, VIK_LAYER(vtl)->name,
 400                              1, wpt->name,
 401                              2, time_buf,
 402                              3, visible,
 403                              4, wpt->comment,
 404                              5, (gint)round(alt),
 405                              6, get_wp_sym_small (wpt->symbol),
 406                              TRW_COL_NUM, vtl,
 407                              WPT_COL_NUM, wpt,
 408                              -1 );
 409 }
 410
 411 /*
 412  * Instead of comparing the pixbufs,
 413  *  look at the waypoint data and compare the symbol (as text).
 414  */
 415 gint sort_pixbuf_compare_func ( GtkTreeModel *model,
 416                                 GtkTreeIter  *a,
 417                                 GtkTreeIter  *b,
 418                                 gpointer      userdata )
 419 {
 420         VikWaypoint *wpt1, *wpt2;
 421         gtk_tree_model_get ( model, a, WPT_COL_NUM, &wpt1, -1 );
 422         if ( !wpt1 ) return 0;
 423         gtk_tree_model_get ( model, b, WPT_COL_NUM, &wpt2, -1 );
 424         if ( !wpt2 ) return 0;
 425
 426         return g_strcmp0 ( wpt1->symbol, wpt2->symbol );
 427 }
 428
 429 static GtkTreeViewColumn *my_new_column_text ( const gchar *title, GtkCellRenderer *renderer, GtkWidget *view, gint column_runner )
 430 {
 431         GtkTreeViewColumn *column = gtk_tree_view_column_new_with_attributes ( title, renderer, "text", column_runner, NULL );
 432         gtk_tree_view_column_set_sort_column_id ( column, column_runner );
 433         gtk_tree_view_append_column ( GTK_TREE_VIEW(view), column );
 434         gtk_tree_view_column_set_reorderable ( column, TRUE );
 435         gtk_tree_view_column_set_resizable ( column, TRUE );
 436         return column;
 437 }
 438
 439 /**
 440  * vik_trw_layer_waypoint_list_internal:
 441  * @dialog:            The dialog to create the widgets in
 442  * @waypoints_and_layers: The list of waypoints (and it's layer) to be shown
 443  * @show_layer_names:  Show the layer names that each waypoint belongs to
 444  *
 445  * Create a table of waypoints with corresponding waypoint information
 446  * This table does not support being actively updated
 447  */
 448 static void vik_trw_layer_waypoint_list_internal ( GtkWidget *dialog,
 449                                                    GList *waypoints_and_layers,
 450                                                    gboolean show_layer_names )
 451 {
 452         if ( !waypoints_and_layers )
 453                 return;
 454
 455         // It's simple storing the gdouble values in the tree store as the sort works automatically
 456         // Then apply specific cell data formatting (rather default double is to 6 decimal places!)
 457         // However not storing any doubles for waypoints ATM
 458         // TODO: Consider adding the waypoint icon into this store for display in the list
 459         GtkTreeStore *store = gtk_tree_store_new ( WPT_LIST_COLS,
 460                                                    G_TYPE_STRING,    // 0: Layer Name
 461                                                    G_TYPE_STRING,    // 1: Waypoint Name
 462                                                    G_TYPE_STRING,    // 2: Date
 463                                                    G_TYPE_BOOLEAN,   // 3: Visible
 464                                                    G_TYPE_STRING,    // 4: Comment
 465                                                    G_TYPE_INT,       // 5: Height
 466                                                    GDK_TYPE_PIXBUF,  // 6: Symbol Icon
 467                                                    G_TYPE_POINTER,   // 7: TrackWaypoint Layer pointer
 468                                                    G_TYPE_POINTER ); // 8: Waypoint pointer
 469
 470         //gtk_tree_selection_set_select_function ( gtk_tree_view_get_selection (GTK_TREE_VIEW(vt)), vik_treeview_selection_filter, vt, NULL );
 471
 472         vik_units_distance_t dist_units = a_vik_get_units_distance ();
 473         vik_units_height_t height_units = a_vik_get_units_height ();
 474
 475         //GList *gl = get_waypoints_and_layers_cb ( vl, user_data );
 476         //g_list_foreach ( waypoints_and_layers, (GFunc) trw_layer_waypoint_list_add, store );
 477         GList *gl = waypoints_and_layers;
 478         while ( gl ) {
 479                 trw_layer_waypoint_list_add ( (vik_trw_waypoint_list_t*)gl->data, store, dist_units, height_units );
 480                 gl = g_list_next ( gl );
 481         }
 482
 483         GtkWidget *view = gtk_tree_view_new();
 484         GtkCellRenderer *renderer = gtk_cell_renderer_text_new();
 485         g_object_set (G_OBJECT (renderer), "xalign", 0.0, "ellipsize", PANGO_ELLIPSIZE_END, NULL);
 486         GtkTreeViewColumn *column;
 487         GtkTreeViewColumn *sort_by_column;
 488
 489         gint column_runner = 0;
 490         if ( show_layer_names ) {
 491                 // Insert column for the layer name when viewing multi layers
 492                 column = my_new_column_text ( _("Layer"), renderer, view, column_runner++ );
 493                 g_object_set (G_OBJECT (renderer), "xalign", 0.0, "ellipsize", PANGO_ELLIPSIZE_END, NULL);
 494                 gtk_tree_view_column_set_expand ( column, TRUE );
 495                 // remember the layer column so we can sort by it later
 496                 sort_by_column = column;
 497         }
 498         else
 499                 column_runner++;
 500
 501         column = my_new_column_text ( _("Name"), renderer, view, column_runner++ );
 502         gtk_tree_view_column_set_expand ( column, TRUE );
 503         if ( !show_layer_names )
 504                 // remember the name column so we can sort by it later
 505                 sort_by_column = column;
 506
 507         column = my_new_column_text ( _("Date"), renderer, view, column_runner++ );
 508         gtk_tree_view_column_set_resizable ( column, TRUE );
 509
 510         GtkCellRenderer *renderer_toggle = gtk_cell_renderer_toggle_new ();
 511         column = gtk_tree_view_column_new_with_attributes ( _("Visible"), renderer_toggle, "active", column_runner, NULL );
 512         gtk_tree_view_column_set_sort_column_id ( column, column_runner );
 513         gtk_tree_view_append_column ( GTK_TREE_VIEW(view), column );
 514         column_runner++;
 515
 516         column = my_new_column_text ( _("Comment"), renderer, view, column_runner++ );
 517         gtk_tree_view_column_set_expand ( column, TRUE );
 518
 519         if ( height_units == VIK_UNITS_HEIGHT_FEET )
 520                 column = my_new_column_text ( _("Max Height\n(Feet)"), renderer, view, column_runner++ );
 521         else
 522                 column = my_new_column_text ( _("Max Height\n(Metres)"), renderer, view, column_runner++ );
 523
 524         GtkCellRenderer *renderer_pixbuf = gtk_cell_renderer_pixbuf_new ();
 525         g_object_set (G_OBJECT (renderer_pixbuf), "xalign", 0.5, NULL);
 526         column = gtk_tree_view_column_new_with_attributes ( _("Symbol"), renderer_pixbuf, "pixbuf", column_runner++, NULL );
 527         // Special sort required for pixbufs
 528         gtk_tree_sortable_set_sort_func ( GTK_TREE_SORTABLE(store), column_runner, sort_pixbuf_compare_func, NULL, NULL );
 529         gtk_tree_view_column_set_sort_column_id ( column, column_runner );
 530         gtk_tree_view_append_column ( GTK_TREE_VIEW(view), column );
 531
 532         gtk_tree_view_set_model ( GTK_TREE_VIEW(view), GTK_TREE_MODEL(store) );
 533         gtk_tree_selection_set_mode ( gtk_tree_view_get_selection(GTK_TREE_VIEW(view)), GTK_SELECTION_BROWSE ); // GTK_SELECTION_MULTIPLE
 534         gtk_tree_view_set_rules_hint ( GTK_TREE_VIEW(view), TRUE );
 535
 536         g_object_unref(store);
 537
 538         GtkWidget *scrolledwindow = gtk_scrolled_window_new ( NULL, NULL );
 539         gtk_scrolled_window_set_policy ( GTK_SCROLLED_WINDOW(scrolledwindow), GTK_POLICY_AUTOMATIC, GTK_POLICY_AUTOMATIC );
 540         gtk_container_add ( GTK_CONTAINER(scrolledwindow), view );
 541
 542         g_object_set ( view, "has-tooltip", TRUE, NULL);
 543
 544         g_signal_connect ( view, "query-tooltip", G_CALLBACK (trw_layer_waypoint_tooltip_cb), NULL );
 545         //g_signal_connect ( gtk_tree_view_get_selection (GTK_TREE_VIEW(view)), "changed", G_CALLBACK(trw_layer_waypoint_select_cb), view );
 546
 547         g_signal_connect ( view, "popup-menu", G_CALLBACK(trw_layer_waypoint_menu_popup), waypoints_and_layers );
 548         g_signal_connect ( view, "button-press-event", G_CALLBACK(trw_layer_waypoint_button_pressed), waypoints_and_layers );
 549
 550         gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), scrolledwindow, TRUE, TRUE, 0);
 551
 552         // Set ordering of the initial view by one of the name columns
 553         gtk_tree_view_column_clicked ( sort_by_column );
 554
 555         // Ensure a reasonable number of items are shown
 556         //  TODO: may be save window size, column order, sorted by between invocations.
 557         gtk_window_set_default_size ( GTK_WINDOW(dialog), show_layer_names ? 700 : 500, 400 );
 558 }
 559
 560
 561 /**
 562  * vik_trw_layer_waypoint_list_show_dialog:
 563  * @title:                    The title for the dialog
 564  * @vl:                       The #VikLayer passed on into get_waypoints_and_layers_cb()
 565  * @user_data:                Data passed on into get_waypoints_and_layers_cb()
 566  * @get_waypoints_and_layers_cb: The function to call to construct items to be analysed
 567  * @show_layer_names:         Normally only set when called from an aggregate level
 568  *
 569  * Common method for showing a list of waypoints with extended information
 570  *
 571  */
 572 void vik_trw_layer_waypoint_list_show_dialog ( gchar *title,
 573                                                VikLayer *vl,
 574                                                gpointer user_data,
 575                                                VikTrwlayerGetWaypointsAndLayersFunc get_waypoints_and_layers_cb,
 576                                                gboolean show_layer_names )
 577 {
 578         GtkWidget *dialog = gtk_dialog_new_with_buttons ( title,
 579                                                           VIK_GTK_WINDOW_FROM_LAYER(vl),
 580                                                           GTK_DIALOG_DESTROY_WITH_PARENT,
 581                                                           GTK_STOCK_CLOSE,
 582                                                           GTK_RESPONSE_CLOSE,
 583                                                           NULL );
 584
 585         GList *gl = get_waypoints_and_layers_cb ( vl, user_data );
 586
 587         vik_trw_layer_waypoint_list_internal ( dialog, gl, show_layer_names );
 588
 589         // Use response to close the dialog with tidy up
 590         g_signal_connect ( G_OBJECT(dialog), "response", G_CALLBACK(waypoint_close_cb), gl );
 591
 592         gtk_widget_show_all ( dialog );
 593         // Yes - set the size *AGAIN* - this time widgets are expanded nicely
 594         gtk_window_resize ( GTK_WINDOW(dialog), show_layer_names ? 800 : 600, 400 );
 595
 596         // ATM lock out on dialog run - to prevent list contents being manipulated in other parts of the GUI whilst shown here.
 597         gtk_dialog_run (GTK_DIALOG (dialog));
 598         // Unfortunately seems subsequently opening the Waypoint Properties we can't interact with it until this dialog is closed
 599         // Thus this dialog is then forcibly closed when opening the properties.
 600
 601         // Occassionally the 'View' doesn't update the viewport properly
 602         //  viewport center + zoom is changed but the viewport isn't updated
 603         // not sure why yet..
 604 }