chiark / gitweb /
Doxygen-clean
[disorder] / disobedience / choose.c
index 565902d6539f0923381aa5e4448d069f7fb82529..d1e951571ff04c5888ea64d9c11151d766a9981f 100644 (file)
 /*
  * This file is part of DisOrder
 /*
  * This file is part of DisOrder
- * Copyright (C) 2006, 2007 Richard Kettlewell
+ * Copyright (C) 2008 Richard Kettlewell
  *
  *
- * This program is free software; you can redistribute it and/or modify
+ * 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
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation, either version 3 of the License, or
  * (at your option) any later version.
  *
  * (at your option) any later version.
  *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ * 
  * You should have received a copy of the GNU General Public License
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
- * USA
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+/** @file disobedience/choose.c
+ * @brief Hierarchical track selection and search
+ *
+ * We now use an ordinary GtkTreeStore/GtkTreeView.
+ *
+ * We don't want to pull the entire tree in memory, but we want directories to
+ * show up as having children.  Therefore we give directories a placeholder
+ * child and replace their children when they are opened.  Placeholders have
+ * TRACK_COLUMN="" and ISFILE_COLUMN=FALSE (so that they don't get check boxes,
+ * lengths, etc).
+ *
+ * TODO:
+ * - sweep up contracted nodes, replacing their content with a placeholder
  */
 
 #include "disobedience.h"
  */
 
 #include "disobedience.h"
+#include "choose.h"
+#include <gdk/gdkkeysyms.h>
 
 
-/* Choose track ------------------------------------------------------------ */
-
-WT(label);
-WT(event_box);
-WT(menu);
-WT(menu_item);
-WT(layout);
-WT(vbox);
-WT(arrow);
-WT(hbox);
-WT(button);
-WT(image);
-WT(entry);
-
-/* We don't use the built-in tree widgets because they require that you know
- * the children of a node on demand, and we have to wait for the server to tell
- * us. */
-
-/* Types */
-
-struct choosenode;
-
-struct displaydata {
-  guint width;                          /* total width required */
-  guint height;                         /* total height required */
-};
-
-/* instantiate the node vector type */
-VECTOR_TYPE(nodevector, struct choosenode *, xrealloc)
-
-struct choosenode {
-  struct choosenode *parent;            /* parent node */
-  const char *path;                     /* full path or 0  */
-  const char *sort;                     /* sort key */
-  const char *display;                  /* display name */
-  int pending;                          /* pending resolve queries */
-  unsigned flags;
-#define CN_EXPANDABLE 0x0001            /* node is expandable */
-#define CN_EXPANDED 0x0002              /* node is expanded */
-/* Expandable items are directories; non-expandable ones are files */
-#define CN_DISPLAYED 0x0004             /* widget is displayed in layout */
-#define CN_SELECTED 0x0008              /* node is selected */
-  struct nodevector children;           /* vector of children */
-  void (*fill)(struct choosenode *);    /* request child fill or 0 for leaf */
-  GtkWidget *container;                 /* the container for this row */
-  GtkWidget *hbox;                      /* the hbox for this row */
-  GtkWidget *arrow;                     /* arrow widget or 0 */
-  GtkWidget *label;                     /* text label for this node */
-  GtkWidget *marker;                    /* queued marker */
-};
-
-struct menuitem {
-  /* Parameters */
-  const char *name;                     /* name */
-
-  /* Callbacks */
-  void (*activate)(GtkMenuItem *menuitem, gpointer user_data);
-  /* Called to activate the menu item.  The user data is the choosenode the
-   * pointer is over. */
-
-  gboolean (*sensitive)(struct choosenode *cn);
-  /* Called to determine whether the menu item should be sensitive.  TODO */
-
-  /* State */
-  gulong handlerid;                     /* signal handler ID */
-  GtkWidget *w;                         /* menu item widget */
-};
-
-/* Variables */
-
-static GtkWidget *chooselayout;
-static GtkWidget *searchentry;          /* search terms */
-static struct choosenode *root;
-static struct choosenode *realroot;
-static GtkWidget *menu;                 /* our popup menu */
-static struct choosenode *last_click;   /* last clicked node for selection */
-static int files_visible;               /* total files visible */
-static int files_selected;              /* total files selected */
-static int search_in_flight;            /* a search is underway */
-static int search_obsolete;             /* the current search is void */
-static char **searchresults;            /* search results */
-static int nsearchresults;              /* number of results */
-
-/* Forward Declarations */
-
-static void clear_children(struct choosenode *cn);
-static struct choosenode *newnode(struct choosenode *parent,
-                                  const char *path,
-                                  const char *display,
-                                  const char *sort,
-                                  unsigned flags,
-                                  void (*fill)(struct choosenode *));
-static void fill_root_node(struct choosenode *cn);
-static void fill_letter_node(struct choosenode *cn);
-static void fill_directory_node(struct choosenode *cn);
-static void got_files(void *v, int nvec, char **vec);
-static void got_resolved_file(void *v, const char *track);
-static void got_dirs(void *v, int nvec, char **vec);
-
-static void expand_node(struct choosenode *cn);
-static void contract_node(struct choosenode *cn);
-static void updated_node(struct choosenode *cn, int redisplay);
-
-static void display_selection(struct choosenode *cn);
-static void clear_selection(struct choosenode *cn);
-
-static void redisplay_tree(void);
-static struct displaydata display_tree(struct choosenode *cn, int x, int y);
-static void undisplay_tree(struct choosenode *cn);
-static void initiate_search(void);
-static void delete_widgets(struct choosenode *cn);
-
-static void clicked_choosenode(GtkWidget attribute((unused)) *widget,
-                               GdkEventButton *event,
-                               gpointer user_data);
-
-static void activate_play(GtkMenuItem *menuitem, gpointer user_data);
-#if 0
-static void activate_remove(GtkMenuItem *menuitem, gpointer user_data);
-#endif
-static void activate_properties(GtkMenuItem *menuitem, gpointer user_data);
-
-static gboolean sensitive_play(struct choosenode *cn);
-#if 0
-static gboolean sensitive_remove(struct choosenode *cn);
-#endif
-static gboolean sensitive_properties(struct choosenode *cn);
-
-static struct menuitem menuitems[] = {
-  { "Play track", activate_play, sensitive_play, 0, 0 },
-#if 0
-  /* Not implemented yet */
-  { "Remove", activate_remove, sensitive_remove, 0, 0 },
-#endif
-  { "Track properties", activate_properties, sensitive_properties, 0, 0 },
-};
-
-#define NMENUITEMS (int)(sizeof menuitems / sizeof *menuitems)
-
-/* Maintaining the data structure ------------------------------------------ */
-
-/* Create a new node */
-static struct choosenode *newnode(struct choosenode *parent,
-                                  const char *path,
-                                  const char *display,
-                                  const char *sort,
-                                  unsigned flags,
-                                  void (*fill)(struct choosenode *)) {
-  struct choosenode *const n = xmalloc(sizeof *n);
-
-  D(("newnode %s %s", path, display));
-  if(flags & CN_EXPANDABLE)
-    assert(fill);
-  else
-    assert(!fill);
-  n->parent = parent;
-  n->path = path;
-  n->display = display;
-  n->sort = sort;
-  n->flags = flags;
-  nodevector_init(&n->children);
-  n->fill = fill;
-  if(parent)
-    nodevector_append(&parent->children, n);
-  return n;
-}
-
-/* Fill the root */
-static void fill_root_node(struct choosenode *cn) {
-  int ch;
-  char *name;
-  struct callbackdata *cbd;
-
-  D(("fill_root_node"));
-  clear_children(cn);
-  if(choosealpha) {
-    if(!cn->children.nvec) {              /* Only need to do this once */
-      for(ch = 'A'; ch <= 'Z'; ++ch) {
-        byte_xasprintf(&name, "%c", ch);
-        newnode(cn, "<letter>", name, name, CN_EXPANDABLE, fill_letter_node);
-      }
-      newnode(cn, "<letter>", "*", "~", CN_EXPANDABLE, fill_letter_node);
-    }
-    updated_node(cn, 1);
-  } else {
-    /* More de-duping possible here */
-    gtk_label_set_text(GTK_LABEL(report_label), "getting files");
-    cbd = xmalloc(sizeof *cbd);
-    cbd->u.choosenode = cn;
-    disorder_eclient_dirs(client, got_dirs, "", 0, cbd);
-    cbd = xmalloc(sizeof *cbd);
-    cbd->u.choosenode = cn;
-    disorder_eclient_files(client, got_files, "", 0, cbd);
-  }
-}
-
-static void delete_cn_widgets(struct choosenode *cn) {
-  if(cn->arrow) {
-    DW(arrow);
-    gtk_widget_destroy(cn->arrow);
-    cn->arrow = 0;
-  }
-  if(cn->label) {
-    DW(label);
-    gtk_widget_destroy(cn->label);
-    cn->label = 0;
-  }
-  if(cn->marker) {
-    DW(image);
-    gtk_widget_destroy(cn->marker);
-    cn->marker = 0;
-  }
-  if(cn->hbox) {
-    DW(hbox);
-    gtk_widget_destroy(cn->hbox);
-    cn->hbox = 0;
-  }
-  if(cn->container) {
-    DW(event_box);
-    gtk_widget_destroy(cn->container);
-    cn->container = 0;
-  }
-}
-
-/* Clear all the children of CN */
-static void clear_children(struct choosenode *cn) {
-  int n;
-
-  D(("clear_children %s", cn->path));
-  /* Recursively clear subtrees */
-  for(n = 0; n < cn->children.nvec; ++n) {
-    clear_children(cn->children.vec[n]);
-    delete_cn_widgets(cn->children.vec[n]);
-  }
-  cn->children.nvec = 0;
-}
-
-/* Fill a child node */
-static void fill_letter_node(struct choosenode *cn) {
-  const char *regexp;
-  struct callbackdata *cbd;
-
-  D(("fill_letter_node %s", cn->display));
-  switch(cn->display[0]) {
-  default:
-    byte_xasprintf((char **)&regexp, "^(the )?%c", tolower(cn->display[0]));
-    break;
-  case 'T':
-    regexp = "^(?!the [^t])t";
-    break;
-  case '*':
-    regexp = "^[^a-z]";
-    break;
-  }
-  /* TODO: caching */
-  /* TODO: de-dupe against fill_directory_node */
-  gtk_label_set_text(GTK_LABEL(report_label), "getting files");
-  clear_children(cn);
-  cbd = xmalloc(sizeof *cbd);
-  cbd->u.choosenode = cn;
-  disorder_eclient_dirs(client, got_dirs, "", regexp, cbd);
-  cbd = xmalloc(sizeof *cbd);
-  cbd->u.choosenode = cn;
-  disorder_eclient_files(client, got_files, "", regexp, cbd);
-}
-
-/* Called with a list of files just below some node */
-static void got_files(void *v, int nvec, char **vec) {
-  struct callbackdata *cbd = v;
-  struct choosenode *cn = cbd->u.choosenode;
-  int n;
+/** @brief The current selection tree */
+GtkTreeStore *choose_store;
 
 
-  D(("got_files %d files for %s", nvec, cn->path));
-  /* Complicated by the need to resolve aliases.  We can save a bit of effort
-   * by re-using cbd though. */
-  cn->pending = nvec;
-  for(n = 0; n < nvec; ++n)
-    disorder_eclient_resolve(client, got_resolved_file, vec[n], cbd);
-}
+/** @brief The view onto the selection tree */
+GtkWidget *choose_view;
 
 
-static void got_resolved_file(void *v, const char *track) {
-  struct callbackdata *cbd = v;
-  struct choosenode *cn = cbd->u.choosenode, *file_cn;
+/** @brief The selection tree's selection */
+GtkTreeSelection *choose_selection;
 
 
-  file_cn = newnode(cn, track,
-                    trackname_transform("track", track, "display"),
-                    trackname_transform("track", track, "sort"),
-                    0/*flags*/, 0/*fill*/);
-  /* Only bother updating when we've got the lot */
-  if(--cn->pending == 0)
-    updated_node(cn, 1);
-}
+/** @brief Count of file listing operations in flight */
+static int choose_list_in_flight;
 
 
-/* Called with a list of directories just below some node */
-static void got_dirs(void *v, int nvec, char **vec) {
-  struct callbackdata *cbd = v;
-  struct choosenode *cn = cbd->u.choosenode;
-  int n;
+/** @brief If nonzero autocollapse column won't be set */
+static int choose_suppress_set_autocollapse;
 
 
-  D(("got_dirs %d dirs for %s", nvec, cn->path));
-  for(n = 0; n < nvec; ++n)
-    newnode(cn, vec[n],
-            trackname_transform("dir", vec[n], "display"),
-            trackname_transform("dir", vec[n], "sort"),
-            CN_EXPANDABLE, fill_directory_node);
-  updated_node(cn, 1);
+static char *choose_get_string(GtkTreeIter *iter, int column) {
+  gchar *gs;
+  gtk_tree_model_get(GTK_TREE_MODEL(choose_store), iter,
+                     column, &gs,
+                     -1);
+  char *s = xstrdup(gs);
+  g_free(gs);
+  return s;
 }
 }
-  
-/* Fill a child node */
-static void fill_directory_node(struct choosenode *cn) {
-  struct callbackdata *cbd;
 
 
-  D(("fill_directory_node %s", cn->path));
-  /* TODO: caching */
-  /* TODO: de-dupe against fill_letter_node */
-  assert(report_label != 0);
-  gtk_label_set_text(GTK_LABEL(report_label), "getting files");
-  clear_children(cn);
-  cbd = xmalloc(sizeof *cbd);
-  cbd->u.choosenode = cn;
-  disorder_eclient_dirs(client, got_dirs, cn->path, 0, cbd);
-  cbd = xmalloc(sizeof *cbd);
-  cbd->u.choosenode = cn;
-  disorder_eclient_files(client, got_files, cn->path, 0, cbd);
+char *choose_get_track(GtkTreeIter *iter) {
+  char *s = choose_get_string(iter, TRACK_COLUMN);
+  return *s ? s : 0;                    /* Placeholder -> NULL */
 }
 
 }
 
-/* Expand a node */
-static void expand_node(struct choosenode *cn) {
-  D(("expand_node %s", cn->path));
-  assert(cn->flags & CN_EXPANDABLE);
-  /* If node is already expanded do nothing. */
-  if(cn->flags & CN_EXPANDED) return;
-  /* We mark the node as expanded and request that it fill itself.  When it has
-   * completed it will called updated_node() and we can redraw at that
-   * point. */
-  cn->flags |= CN_EXPANDED;
-  /* TODO: visual feedback */
-  cn->fill(cn);
+char *choose_get_sort(GtkTreeIter *iter) {
+  return choose_get_string(iter, SORT_COLUMN);
 }
 
 }
 
-/* Contract a node */
-static void contract_node(struct choosenode *cn) {
-  D(("contract_node %s", cn->path));
-  assert(cn->flags & CN_EXPANDABLE);
-  /* If node is already contracted do nothing. */
-  if(!(cn->flags & CN_EXPANDED)) return;
-  cn->flags &= ~CN_EXPANDED;
-  /* Clear selection below this node */
-  clear_selection(cn);
-  /* Zot children.  We never used to do this but the result would be that over
-   * time you'd end up with the entire tree pulled into memory.  If the server
-   * is over a slow network it will make interactivity slightly worse; if
-   * anyone complains we can make it an option. */
-  clear_children(cn);
-  /* We can contract a node immediately. */
-  redisplay_tree();
+char *choose_get_display(GtkTreeIter *iter) {
+  return choose_get_string(iter, NAME_COLUMN);
 }
 
 }
 
-/* qsort callback for ordering choosenodes */
-static int compare_choosenode(const void *av, const void *bv) {
-  const struct choosenode *const *aa = av, *const *bb = bv;
-  const struct choosenode *a = *aa, *b = *bb;
-
-  return compare_tracks(a->sort, b->sort,
-                       a->display, b->display,
-                       a->path, b->path);
+int choose_is_file(GtkTreeIter *iter) {
+  gboolean isfile;
+  gtk_tree_model_get(GTK_TREE_MODEL(choose_store), iter,
+                     ISFILE_COLUMN, &isfile,
+                     -1);
+  return isfile;
 }
 
 }
 
-/* Called when an expandable node is updated.   */
-static void updated_node(struct choosenode *cn, int redisplay) {
-  D(("updated_node %s", cn->path));
-  assert(cn->flags & CN_EXPANDABLE);
-  /* It might be that the node has been de-expanded since we requested the
-   * update.  In that case we ignore this notification. */
-  if(!(cn->flags & CN_EXPANDED)) return;
-  /* Sort children */
-  qsort(cn->children.vec, cn->children.nvec, sizeof (struct choosenode *),
-        compare_choosenode);
-  if(redisplay)
-    redisplay_tree();
+int choose_is_dir(GtkTreeIter *iter) {
+  gboolean isfile;
+  gtk_tree_model_get(GTK_TREE_MODEL(choose_store), iter,
+                     ISFILE_COLUMN, &isfile,
+                     -1);
+  if(isfile)
+    return FALSE;
+  return !choose_is_placeholder(iter);
 }
 
 }
 
-/* Searching --------------------------------------------------------------- */
-
-static int compare_track_for_qsort(const void *a, const void *b) {
-  return compare_path(*(char **)a, *(char **)b);
-}
-
-/* Return true iff FILE is a child of DIR */
-static int is_child(const char *dir, const char *file) {
-  const size_t dlen = strlen(dir);
-
-  return (!strncmp(file, dir, dlen)
-          && file[dlen] == '/'
-          && strchr(file + dlen + 1, '/') == 0);
+int choose_is_placeholder(GtkTreeIter *iter) {
+  return choose_get_string(iter, TRACK_COLUMN)[0] == 0;
 }
 
 }
 
-/* Return true iff FILE is a descendant of DIR */
-static int is_descendant(const char *dir, const char *file) {
-  const size_t dlen = strlen(dir);
-
-  return !strncmp(file, dir, dlen) && file[dlen] == '/';
+int choose_can_autocollapse(GtkTreeIter *iter) {
+  gboolean autocollapse;
+  gtk_tree_model_get(GTK_TREE_MODEL(choose_store), iter,
+                     AUTOCOLLAPSE_COLUMN, &autocollapse,
+                     -1);
+  return autocollapse;
 }
 
 }
 
-/* Called to fill a node in the search results tree */
-static void fill_search_node(struct choosenode *cn) {
-  int n;
-  const size_t plen = strlen(cn->path);
-  const char *s;
-  char *dir, *last = 0;
-
-  D(("fill_search_node %s", cn->path));
-  /* We depend on the search results being sorted as by compare_path(). */
-  clear_children(cn);
-  for(n = 0; n < nsearchresults; ++n) {
-    /* We only care about descendants of CN */
-    if(!is_descendant(cn->path, searchresults[n]))
-       continue;
-    s = strchr(searchresults[n] + plen + 1, '/');
-    if(s) {
-      /* We've identified a subdirectory of CN. */
-      dir = xstrndup(searchresults[n], s - searchresults[n]);
-      if(!last || strcmp(dir, last)) {
-        /* Not a duplicate */
-        last = dir;
-        newnode(cn, dir,
-                trackname_transform("dir", dir, "display"),
-                trackname_transform("dir", dir, "sort"),
-                CN_EXPANDABLE, fill_search_node);
-      }
-    } else {
-      /* We've identified a file in CN */
-      newnode(cn, searchresults[n],
-              trackname_transform("track", searchresults[n], "display"),
-              trackname_transform("track", searchresults[n], "sort"),
-              0/*flags*/, 0/*fill*/);
-    }
+/** @brief Remove node @p it and all its children
+ * @param it Iterator, updated to point to next
+ * @return True if iterator remains valid
+ *
+ * TODO is this necessary?  gtk_tree_store_remove() does not document what
+ * happens to children.
+ */
+static gboolean choose_remove_node(GtkTreeIter *it) {
+  GtkTreeIter child[1];
+  gboolean childv = gtk_tree_model_iter_children(GTK_TREE_MODEL(choose_store),
+                                                 child,
+                                                 it);
+  while(childv)
+    childv = choose_remove_node(child);
+  return gtk_tree_store_remove(choose_store, it);
+}
+
+/** @brief Update length and state fields */
+static gboolean choose_set_state_callback(GtkTreeModel attribute((unused)) *model,
+                                          GtkTreePath attribute((unused)) *path,
+                                          GtkTreeIter *it,
+                                          gpointer attribute((unused)) data) {
+  if(choose_is_file(it)) {
+    const char *track = choose_get_track(it);
+    const long l = namepart_length(track);
+    char length[64];
+    if(l > 0)
+      byte_snprintf(length, sizeof length, "%ld:%02ld", l / 60, l % 60);
+    else
+      length[0] = 0;
+    gtk_tree_store_set(choose_store, it,
+                       LENGTH_COLUMN, length,
+                       STATE_COLUMN, queued(track),
+                       -1);
+    if(choose_is_search_result(track))
+      gtk_tree_store_set(choose_store, it,
+                         BG_COLUMN, SEARCH_RESULT_BG,
+                         FG_COLUMN, SEARCH_RESULT_FG,
+                         -1);
+    else
+      gtk_tree_store_set(choose_store, it,
+                         BG_COLUMN, (char *)0,
+                         FG_COLUMN, (char *)0,
+                         -1);
   }
   }
-  updated_node(cn, 1);
+  return FALSE;                         /* continue walking */
 }
 
 }
 
-/* This is called from eclient with a (possibly empty) list of search results,
- * and also from initiate_seatch with an always empty list to indicate that
- * we're not searching for anything in particular. */
-static void search_completed(void attribute((unused)) *v,
-                             int nvec, char **vec) {
-  struct choosenode *cn;
-  int n;
-  const char *dir;
-
-  search_in_flight = 0;
-  if(search_obsolete) {
-    /* This search has been obsoleted by user input since it started.
-     * Therefore we throw away the result and search again. */
-    search_obsolete = 0;
-    initiate_search();
-  } else {
-    if(nvec) {
-      /* We will replace the choose tree with a tree structured view of search
-       * results.  First we must disabled the choose tree's widgets. */
-      delete_widgets(root);
-      /* Put the tracks into order, grouped by directory.  They'll probably
-       * come back this way anyway in current versions of the server, but it's
-       * cheap not to rely on it (compared with the massive effort we expend
-       * later on) */
-      qsort(vec, nvec, sizeof(char *), compare_track_for_qsort);
-      searchresults = vec;
-      nsearchresults = nvec;
-      cn = root = newnode(0/*parent*/, "", "Search results", "",
-                  CN_EXPANDABLE|CN_EXPANDED, fill_search_node);
-      /* Construct the initial tree.  We do this in a single pass and expand
-       * everything, so you can actually see your search results. */
-      for(n = 0; n < nsearchresults; ++n) {
-        /* Firstly we might need to go up a few directories to each an ancestor
-         * of this track */
-        while(!is_descendant(cn->path, searchresults[n])) {
-          /* We report the update on each node the last time we see it (With
-           * display=0, the main purpose of this is to get the order of the
-           * children right.) */
-          updated_node(cn, 0);
-          cn = cn->parent;
-        }
-        /* Secondly we might need to insert some new directories */
-        while(!is_child(cn->path, searchresults[n])) {
-          /* Figure out the subdirectory */
-          dir = xstrndup(searchresults[n],
-                         strchr(searchresults[n] + strlen(cn->path) + 1,
-                                '/') - searchresults[n]);
-          cn = newnode(cn, dir,
-                       trackname_transform("dir", dir, "display"),
-                       trackname_transform("dir", dir, "sort"),
-                       CN_EXPANDABLE|CN_EXPANDED, fill_search_node);
-        }
-        /* Finally we can insert the track as a child of the current
-         * directory */
-        newnode(cn, searchresults[n],
-                trackname_transform("track", searchresults[n], "display"),
-                trackname_transform("track", searchresults[n], "sort"),
-                0/*flags*/, 0/*fill*/);
-      }
-      while(cn) {
-        /* Update all the nodes back up to the root */
-        updated_node(cn, 0);
-        cn = cn->parent;
-      }
-      /* Now it's worth displaying the tree */
-      redisplay_tree();
-    } else if(root != realroot) {
-      delete_widgets(root);
-      root = realroot;
-      redisplay_tree();
-    }
-  }
+/** @brief Called when the queue or playing track change */
+static void choose_set_state(const char attribute((unused)) *event,
+                             void attribute((unused)) *eventdata,
+                             void attribute((unused)) *callbackdata) {
+  gtk_tree_model_foreach(GTK_TREE_MODEL(choose_store),
+                         choose_set_state_callback,
+                         NULL);
 }
 
 }
 
-static void initiate_search(void) {
-  char *terms, *e;
-
-  /* Find out what the user is after */
-  terms = xstrdup(gtk_entry_get_text(GTK_ENTRY(searchentry)));
-  /* Strip leading and trailing space */
-  while(*terms == ' ') ++terms;
-  e = terms + strlen(terms);
-  while(e > terms && e[-1] == ' ') --e;
-  *e = 0;
-  /* If a search is already underway then mark it as obsolete.  We'll revisit
-   * when it returns. */
-  if(search_in_flight) {
-    search_obsolete = 1;
-    return;
-  }
-  if(*terms) {
-    /* There's still something left.  Initiate the search. */
-    if(disorder_eclient_search(client, search_completed, terms, 0)) {
-      /* The search terms are bad!  We treat this as if there were no search
-       * terms at all.  Some kind of feedback would be handy. */
-      fprintf(stderr, "bad terms [%s]\n", terms); /* TODO */
-      search_completed(0, 0, 0);
-    } else {
-      search_in_flight = 1;
-    }
+/** @brief (Re-)populate a node
+ * @param parent_ref Node to populate or NULL to fill root
+ * @param nvec Number of children to add
+ * @param vec Children
+ * @param isfile 1 if children are files, 0 if directories
+ *
+ * Adjusts the set of files (or directories) below @p parent_ref to match those
+ * listed in @p nvec and @p vec.
+ *
+ * @p parent_ref will be destroyed.
+ */
+static void choose_populate(GtkTreeRowReference *parent_ref,
+                            int nvec, char **vec,
+                            int isfile) {
+  const char *type = isfile ? "track" : "dir";
+  //fprintf(stderr, "%d new children of type %s\n", nvec, type);
+  if(!nvec)
+    goto skip;
+  /* Compute parent_* */
+  GtkTreeIter pit[1], *parent_it;
+  GtkTreePath *parent_path;
+  if(parent_ref) {
+    parent_path = gtk_tree_row_reference_get_path(parent_ref);
+    parent_it = pit;
+    gboolean pitv = gtk_tree_model_get_iter(GTK_TREE_MODEL(choose_store),
+                                            pit, parent_path);
+    assert(pitv);
+    /*fprintf(stderr, "choose_populate %s: parent path is [%s]\n",
+            type,
+            gtk_tree_path_to_string(parent_path));*/
   } else {
   } else {
-    /* No search terms - we want to see all tracks */
-    search_completed(0, 0, 0);
+    parent_path = 0;
+    parent_it = 0;
+    /*fprintf(stderr, "choose_populate %s: populating the root\n",
+            type);*/
   }
   }
-}
-
-/* Called when the cancel search button is clicked */
-static void clearsearch_clicked(GtkButton attribute((unused)) *button,
-                                gpointer attribute((unused)) userdata) {
-  gtk_entry_set_text(GTK_ENTRY(searchentry), "");
-}
-
-/* Display functions ------------------------------------------------------- */
-
-/* Delete all the widgets in the tree */
-static void delete_widgets(struct choosenode *cn) {
-  int n;
-
-  delete_cn_widgets(cn);
-  for(n = 0; n < cn->children.nvec; ++n)
-    delete_widgets(cn->children.vec[n]);
-  cn->flags &= ~(CN_DISPLAYED|CN_SELECTED);
-  files_selected = 0;
-}
-
-/* Update the display */
-static void redisplay_tree(void) {
-  struct displaydata d;
-  guint oldwidth, oldheight;
-
-  D(("redisplay_tree"));
-  /* We'll count these up empirically each time */
-  files_selected = 0;
-  files_visible = 0;
-  /* Correct the layout and find out how much space it uses */
-  d = display_tree(root, 0, 0);
-  /* We must set the total size or scrolling will not work (it wouldn't be hard
-   * for GtkLayout to figure it out for itself but presumably you're supposed
-   * to be able to have widgets off the edge of the layuot.)
-   *
-   * There is a problem: if we shrink the size then the part of the screen that
-   * is outside the new size but inside the old one is not updated.  I think
-   * this is arguably bug in GTK+ but it's easy to force a redraw if this
-   * region is nonempty.
-   */
-  gtk_layout_get_size(GTK_LAYOUT(chooselayout), &oldwidth, &oldheight);
-  if(oldwidth > d.width || oldheight > d.height)
-    gtk_widget_queue_draw(chooselayout);
-  gtk_layout_set_size(GTK_LAYOUT(chooselayout), d.width, d.height);
-  /* Notify the main menu of any recent changes */
-  menu_update(-1);
-}
-
-/* Make sure all displayed widgets from CN down exist and are in their proper
- * place and return the vertical space used. */
-static struct displaydata display_tree(struct choosenode *cn, int x, int y) {
-  int n, aw;
-  GtkRequisition req;
-  struct displaydata d, cd;
-  GdkPixbuf *pb;
-  
-  D(("display_tree %s %d,%d", cn->path, x, y));
-
-  /* An expandable item contains an arrow and a text label.  When you press the
-   * button it flips its expand state.
-   *
-   * A non-expandable item has just a text label and no arrow.
-   */
-  if(!cn->container) {
-    /* Widgets need to be created */
-    NW(hbox);
-    cn->hbox = gtk_hbox_new(FALSE, 1);
-    if(cn->flags & CN_EXPANDABLE) {
-      NW(arrow);
-      cn->arrow = gtk_arrow_new(cn->flags & CN_EXPANDED ? GTK_ARROW_DOWN
-                                                        : GTK_ARROW_RIGHT,
-                                GTK_SHADOW_NONE);
-      cn->marker = 0;
+  /* Both td[] and the current node set are sorted so we can do a single linear
+   * pass to insert new nodes and remove unwanted ones.  The total performance
+   * may be worse than linear depending on the performance of GTK+'s insert and
+   * delete operations. */
+  //fprintf(stderr, "sorting tracks\n");
+  struct tracksort_data *td = tracksort_init(nvec, vec, type);
+  GtkTreeIter it[1];
+  gboolean itv = gtk_tree_model_iter_children(GTK_TREE_MODEL(choose_store),
+                                              it,
+                                              parent_it);
+  int inserted = 0, deleted_placeholder = 0;
+  //fprintf(stderr, "inserting tracks type=%s\n", type);
+  while(nvec > 0 || itv) {
+    /*fprintf(stderr, "td[] = %s, it=%s [%s]\n",
+            nvec > 0 ? td->track : "(none)",
+            itv ? choose_get_track(it) : "(!itv)",
+            itv ? (choose_is_file(it) ? "file" : "dir") : "");*/
+    enum { INSERT, DELETE, SKIP_TREE, SKIP_BOTH } action;
+    const char *track = itv ? choose_get_track(it) : 0;
+    if(itv && !track) {
+      //fprintf(stderr, " placeholder\n");
+      action = DELETE;
+      ++deleted_placeholder;
+    } else if(nvec > 0 && itv) {
+      /* There's both a tree row and a td[] entry */
+      const int cmp = compare_tracks(td->sort, choose_get_sort(it),
+                                     td->display, choose_get_display(it),
+                                     td->track, track);
+      //fprintf(stderr, " cmp=%d\n", cmp);
+      if(cmp < 0)
+        /* td < it, so we insert td before it */
+        action = INSERT;
+      else if(cmp > 0) {
+        /* td > it, so we must either delete it (if the same type) or skip it */
+        if(choose_is_file(it) == isfile)
+          action = DELETE;
+        else
+          action = SKIP_TREE;
+      } else
+        /* td = it, so we step past both */
+        action = SKIP_BOTH;
+    } else if(nvec > 0) {
+      /* We've reached the end of the tree rows, but new have tracks left in
+       * td[] */
+      //fprintf(stderr, " inserting\n");
+      action = INSERT;
     } else {
     } else {
-      cn->arrow = 0;
-      if((pb = find_image("notes.png"))) {
-        NW(image);
-        cn->marker = gtk_image_new_from_pixbuf(pb);
+      /* We've reached the end of the new tracks from td[], but there are
+       * further tracks in the tree */
+      //fprintf(stderr, " deleting\n");
+      if(choose_is_file(it) == isfile)
+        action = DELETE;
+      else
+        action = SKIP_TREE;
+    }
+    
+    switch(action) {
+    case INSERT: {
+      //fprintf(stderr, " INSERT %s\n", td->track);
+      /* Insert a new row from td[] before it, or at the end if it is no longer
+       * valid */
+      GtkTreeIter child[1];
+      gtk_tree_store_insert_before(choose_store,
+                                   child, /* new row */
+                                   parent_it, /* parent */
+                                   itv ? it : NULL); /* successor */
+      gtk_tree_store_set(choose_store, child,
+                         NAME_COLUMN, td->display,
+                         ISFILE_COLUMN, isfile,
+                         TRACK_COLUMN, td->track,
+                         SORT_COLUMN, td->sort,
+                         AUTOCOLLAPSE_COLUMN, FALSE,
+                         -1);
+      /* Update length and state; we expect this to kick off length lookups
+       * rather than necessarily get the right value the first time round. */
+      choose_set_state_callback(0, 0, child, 0);
+      /* If we inserted a directory, insert a placeholder too, so it appears to
+       * have children; it will be deleted when we expand the directory. */
+      if(!isfile) {
+        //fprintf(stderr, "  inserting a placeholder\n");
+        GtkTreeIter placeholder[1];
+
+        gtk_tree_store_append(choose_store, placeholder, child);
+        gtk_tree_store_set(choose_store, placeholder,
+                           NAME_COLUMN, "Waddling...",
+                           TRACK_COLUMN, "",
+                           ISFILE_COLUMN, FALSE,
+                           -1);
       }
       }
+      ++inserted;
+      ++td;
+      --nvec;
+      break;
+    }
+    case SKIP_BOTH:
+      //fprintf(stderr, " SKIP_BOTH\n");
+      ++td;
+      --nvec;
+      /* fall thru */
+    case SKIP_TREE:
+      //fprintf(stderr, " SKIP_TREE\n");
+      itv = gtk_tree_model_iter_next(GTK_TREE_MODEL(choose_store), it);
+      break;
+    case DELETE:
+      //fprintf(stderr, " DELETE\n");
+      itv = choose_remove_node(it);
+      break;
     }
     }
-    NW(label);
-    cn->label = gtk_label_new(cn->display);
-    if(cn->arrow)
-      gtk_container_add(GTK_CONTAINER(cn->hbox), cn->arrow);
-    gtk_container_add(GTK_CONTAINER(cn->hbox), cn->label);
-    if(cn->marker)
-      gtk_container_add(GTK_CONTAINER(cn->hbox), cn->marker);
-    NW(event_box);
-    cn->container = gtk_event_box_new();
-    gtk_container_add(GTK_CONTAINER(cn->container), cn->hbox);
-    g_signal_connect(cn->container, "button-release-event", 
-                     G_CALLBACK(clicked_choosenode), cn);
-    g_signal_connect(cn->container, "button-press-event", 
-                     G_CALLBACK(clicked_choosenode), cn);
-    g_object_ref(cn->container);
-    gtk_widget_set_name(cn->label, "choose");
-    gtk_widget_set_name(cn->container, "choose");
-    /* Show everything by default */
-    gtk_widget_show_all(cn->container);
-  }
-  assert(cn->container);
-  /* Make sure the icon is right */
-  if(cn->flags & CN_EXPANDABLE)
-    gtk_arrow_set(GTK_ARROW(cn->arrow),
-                  cn->flags & CN_EXPANDED ? GTK_ARROW_DOWN : GTK_ARROW_RIGHT,
-                  GTK_SHADOW_NONE);
-  else if(cn->marker)
-    /* Make sure the queued marker is right */
-    /* TODO: doesn't always work */
-    (queued(cn->path) ? gtk_widget_show : gtk_widget_hide)(cn->marker);
-  /* Put the widget in the right place */
-  if(cn->flags & CN_DISPLAYED)
-    gtk_layout_move(GTK_LAYOUT(chooselayout), cn->container, x, y);
-  else {
-    gtk_layout_put(GTK_LAYOUT(chooselayout), cn->container, x, y);
-    cn->flags |= CN_DISPLAYED;
   }
   }
-  /* Set the widget's selection status */
-  if(!(cn->flags & CN_EXPANDABLE))
-    display_selection(cn);
-  /* Find the size used so we can get vertical positioning right. */
-  gtk_widget_size_request(cn->container, &req);
-  d.width = x + req.width;
-  d.height = y + req.height;
-  if(cn->flags & CN_EXPANDED) {
-    /* We'll offset children by the size of the arrow whatever it might be. */
-    assert(cn->arrow);
-    gtk_widget_size_request(cn->arrow, &req);
-    aw = req.width;
-    for(n = 0; n < cn->children.nvec; ++n) {
-      cd = display_tree(cn->children.vec[n], x + aw, d.height);
-      if(cd.width > d.width)
-        d.width = cd.width;
-      d.height = cd.height;
+  /*fprintf(stderr, "inserted=%d deleted_placeholder=%d\n\n",
+          inserted, deleted_placeholder);*/
+  if(parent_ref) {
+    /* If we deleted a placeholder then we must re-expand the row */
+    if(deleted_placeholder) {
+      ++choose_suppress_set_autocollapse;
+      gtk_tree_view_expand_row(GTK_TREE_VIEW(choose_view), parent_path, FALSE);
+      --choose_suppress_set_autocollapse;
     }
     }
-  } else {
-    for(n = 0; n < cn->children.nvec; ++n)
-      undisplay_tree(cn->children.vec[n]);
+    gtk_tree_row_reference_free(parent_ref);
+    gtk_tree_path_free(parent_path);
   }
   }
-  if(!(cn->flags & CN_EXPANDABLE)) {
-    ++files_visible;
-    if(cn->flags & CN_SELECTED)
-      ++files_selected;
+skip:
+  /* We only notify others that we've inserted tracks when there are no more
+   * insertions pending, so that they don't have to keep track of how many
+   * requests they've made.  */
+  if(--choose_list_in_flight == 0) {
+    /* Notify interested parties that we inserted some tracks, AFTER making
+     * sure that the row is properly expanded */
+    //fprintf(stderr, "raising choose-more-tracks\n");
+    event_raise("choose-more-tracks", 0);
   }
   }
-  /* report back how much space we used */
-  D(("display_tree %s %d,%d total size %dx%d", cn->path, x, y,
-     d.width, d.height));
-  return d;
+  //fprintf(stderr, "choose_list_in_flight -> %d-\n", choose_list_in_flight);
 }
 
 }
 
-/* Remove widgets for newly hidden nodes */
-static void undisplay_tree(struct choosenode *cn) {
-  int n;
-
-  D(("undisplay_tree %s", cn->path));
-  /* Remove this widget from the display */
-  if(cn->flags & CN_DISPLAYED) {
-    gtk_container_remove(GTK_CONTAINER(chooselayout), cn->container);
-    cn->flags ^= CN_DISPLAYED;
+static void choose_dirs_completed(void *v,
+                                  const char *err,
+                                  int nvec, char **vec) {
+  if(err) {
+    popup_protocol_error(0, err);
+    return;
   }
   }
-  /* Remove children too */
-  for(n = 0; n < cn->children.nvec; ++n)
-    undisplay_tree(cn->children.vec[n]);
-}
-
-/* Selection --------------------------------------------------------------- */
-
-static void display_selection(struct choosenode *cn) {
-  /* Need foreground and background colors */
-  gtk_widget_set_state(cn->label, (cn->flags & CN_SELECTED
-                                   ? GTK_STATE_SELECTED : GTK_STATE_NORMAL));
-  gtk_widget_set_state(cn->container, (cn->flags & CN_SELECTED
-                                       ? GTK_STATE_SELECTED : GTK_STATE_NORMAL));
+  choose_populate(v, nvec, vec, 0/*!isfile*/);
 }
 
 }
 
-/* Set the selection state of a widget.  Directories can never be selected, we
- * just ignore attempts to do so. */
-static void set_selection(struct choosenode *cn, int selected) {
-  unsigned f = selected ? CN_SELECTED : 0;
-
-  D(("set_selection %d %s", selected, cn->path));
-  if(!(cn->flags & CN_EXPANDABLE) && (cn->flags & CN_SELECTED) != f) {
-    cn->flags ^= CN_SELECTED;
-    /* Maintain selection count */
-    if(selected)
-      ++files_selected;
-    else
-      --files_selected;
-    display_selection(cn);
-    /* Update main menu sensitivity */
-    menu_update(-1);
+static void choose_files_completed(void *v,
+                                   const char *err,
+                                   int nvec, char **vec) {
+  if(err) {
+    popup_protocol_error(0, err);
+    return;
   }
   }
+  choose_populate(v, nvec, vec, 1/*isfile*/);
+}
+
+void choose_play_completed(void attribute((unused)) *v,
+                           const char *err) {
+  if(err)
+    popup_protocol_error(0, err);
+}
+
+static void choose_state_toggled
+    (GtkCellRendererToggle attribute((unused)) *cell_renderer,
+     gchar *path_str,
+     gpointer attribute((unused)) user_data) {
+  GtkTreeIter it[1];
+  /* Identify the track */
+  gboolean itv =
+    gtk_tree_model_get_iter_from_string(GTK_TREE_MODEL(choose_store),
+                                        it,
+                                        path_str);
+  if(!itv)
+    return;
+  if(!choose_is_file(it))
+    return;
+  const char *track = choose_get_track(it);
+  if(queued(track))
+    return;
+  disorder_eclient_play(client, track, choose_play_completed, 0);
+  
 }
 
 }
 
-/* Recursively clear all selection bits from CN down */
-static void clear_selection(struct choosenode *cn) {
-  int n;
-
-  set_selection(cn, 0);
-  for(n = 0; n < cn->children.nvec; ++n)
-    clear_selection(cn->children.vec[n]);
-}
-
-/* User actions ------------------------------------------------------------ */
-
-/* Clicked on something */
-static void clicked_choosenode(GtkWidget attribute((unused)) *widget,
-                               GdkEventButton *event,
-                               gpointer user_data) {
-  struct choosenode *cn = user_data;
-  int ind, last_ind, n;
-
-  D(("clicked_choosenode %s", cn->path));
-  if(event->type == GDK_BUTTON_RELEASE
-     && event->button == 1) {
-    /* Left click */
-    if(cn->flags & CN_EXPANDABLE) {
-      /* This is a directory.  Flip its expansion status. */
-      if(cn->flags & CN_EXPANDED)
-        contract_node(cn);
-      else
-        expand_node(cn);
-      last_click = 0;
+/** @brief (Re-)get the children of @p path
+ * @param path Path to target row
+ * @param iter Iterator pointing at target row
+ *
+ * Called from choose_row_expanded() to make sure that the contents are present
+ * and from choose_refill_callback() to (re-)synchronize.
+ */
+static void choose_refill_row(GtkTreePath *path,
+                              GtkTreeIter *iter) {
+  const char *track = choose_get_track(iter);
+  disorder_eclient_files(client, choose_files_completed,
+                         track,
+                         NULL,
+                         gtk_tree_row_reference_new(GTK_TREE_MODEL(choose_store),
+                                                    path));
+  disorder_eclient_dirs(client, choose_dirs_completed,
+                        track,
+                        NULL,
+                        gtk_tree_row_reference_new(GTK_TREE_MODEL(choose_store),
+                                                   path));
+  /* The row references are destroyed in the _completed handlers. */
+  choose_list_in_flight += 2;
+}
+
+static void choose_row_expanded(GtkTreeView attribute((unused)) *treeview,
+                                GtkTreeIter *iter,
+                                GtkTreePath *path,
+                                gpointer attribute((unused)) user_data) {
+  /*fprintf(stderr, "row-expanded path=[%s]\n\n",
+          gtk_tree_path_to_string(path));*/
+  /* We update a node's contents whenever it is expanded, even if it was
+   * already populated; the effect is that contracting and expanding a node
+   * suffices to update it to the latest state on the server. */
+  choose_refill_row(path, iter);
+  if(!choose_suppress_set_autocollapse) {
+    if(choose_auto_expanding) {
+      /* This was an automatic expansion; mark it the row for auto-collapse. */
+      gtk_tree_store_set(choose_store, iter,
+                         AUTOCOLLAPSE_COLUMN, TRUE,
+                         -1);
+      /*fprintf(stderr, "enable auto-collapse for %s\n",
+              gtk_tree_path_to_string(path));*/
     } else {
     } else {
-      /* This is a file.  Adjust selection status */
-      /* TODO the basic logic here is essentially the same as that in queue.c.
-       * Can we share code at all? */
-      switch(event->state & (GDK_SHIFT_MASK|GDK_CONTROL_MASK)) {
-      case 0:
-        clear_selection(root);
-        set_selection(cn, 1);
-        last_click = cn;
-        break;
-      case GDK_CONTROL_MASK:
-        set_selection(cn, !(cn->flags & CN_SELECTED));
-        last_click = cn;
-        break;
-      case GDK_SHIFT_MASK:
-      case GDK_SHIFT_MASK|GDK_CONTROL_MASK:
-        if(last_click && last_click->parent == cn->parent) {
-          /* Figure out where the current and last clicks are in the list */
-          ind = last_ind = -1;
-          for(n = 0; n < cn->parent->children.nvec; ++n) {
-            if(cn->parent->children.vec[n] == cn)
-              ind = n;
-            if(cn->parent->children.vec[n] == last_click)
-              last_ind = n;
-          }
-          /* Test shouldn't ever fail, but still */
-          if(ind >= 0 && last_ind >= 0) {
-            if(!(event->state & GDK_CONTROL_MASK)) {
-              for(n = 0; n < cn->parent->children.nvec; ++n)
-                set_selection(cn->parent->children.vec[n], 0);
-            }
-            if(ind > last_ind)
-              for(n = last_ind; n <= ind; ++n)
-                set_selection(cn->parent->children.vec[n], 1);
-            else
-              for(n = ind; n <= last_ind; ++n)
-                set_selection(cn->parent->children.vec[n], 1);
-            if(event->state & GDK_CONTROL_MASK)
-              last_click = cn;
-          }
-        }
-        break;
-      }
-    }
-  } else if(event->type == GDK_BUTTON_RELEASE
-     && event->button == 2) {
-    /* Middle click - play the pointed track */
-    if(!(cn->flags & CN_EXPANDABLE)) {
-      clear_selection(root);
-      set_selection(cn, 1);
-      gtk_label_set_text(GTK_LABEL(report_label), "adding track to queue");
-      disorder_eclient_play(client, cn->path, 0, 0);
-      last_click = 0;
-    }
-  } else if(event->type == GDK_BUTTON_PRESS
-     && event->button == 3) {
-    /* Right click.  Pop up a menu. */
-    /* If the current file isn't selected, switch the selection to just that.
-     * (If we're looking at a directory then leave the selection alone.) */
-    if(!(cn->flags & CN_EXPANDABLE) && !(cn->flags & CN_SELECTED)) {
-      clear_selection(root);
-      set_selection(cn, 1);
-      last_click = cn;
-    }
-    /* Set the item sensitivity and callbacks */
-    for(n = 0; n < NMENUITEMS; ++n) {
-      if(menuitems[n].handlerid)
-        g_signal_handler_disconnect(menuitems[n].w,
-                                    menuitems[n].handlerid);
-      gtk_widget_set_sensitive(menuitems[n].w,
-                               menuitems[n].sensitive(cn));
-      menuitems[n].handlerid = g_signal_connect
-        (menuitems[n].w, "activate", G_CALLBACK(menuitems[n].activate), cn);
+      /* This was a manual expansion.  Inhibit automatic collapse on this row
+       * and all its ancestors.  */
+      gboolean itv;
+      do {
+        gtk_tree_store_set(choose_store, iter,
+                           AUTOCOLLAPSE_COLUMN, FALSE,
+                           -1);
+        /*fprintf(stderr, "suppress auto-collapse for %s\n",
+                gtk_tree_model_get_string_from_iter(GTK_TREE_MODEL(choose_store),
+                                                    iter));*/
+        GtkTreeIter child = *iter;
+        itv = gtk_tree_model_iter_parent(GTK_TREE_MODEL(choose_store),
+                                         iter,
+                                         &child);
+      } while(itv);
+      /* The effect of this is that if you expand a row that's actually a
+       * sibling of the real target of the auto-expansion, it stays expanded
+       * when you clear a search.  That's find and good, but it _still_ stays
+       * expanded if you expand it and then collapse it.
+       *
+       * An alternative policy would be to only auto-collapse rows that don't
+       * have any expanded children (apart from ones also subject to
+       * auto-collapse).  I'm not sure what the most usable policy is.
+       */
     }
     }
-    /* Pop up the menu */
-    gtk_widget_show_all(menu);
-    gtk_menu_popup(GTK_MENU(menu), 0, 0, 0, 0,
-                   event->button, event->time);
   }
 }
 
   }
 }
 
-static void searchentry_changed(GtkEditable attribute((unused)) *editable,
-                                gpointer attribute((unused)) user_data) {
-  initiate_search();
-}
-
-/* Menu items -------------------------------------------------------------- */
+static void choose_auto_collapse_callback(GtkTreeView *tree_view,
+                                          GtkTreePath *path,
+                                          gpointer attribute((unused)) user_data) {
+  GtkTreeIter it[1];
 
 
-static void recurse_selected(struct choosenode *cn, struct vector *v) {
-  int n;
-
-  if(cn->flags & CN_EXPANDABLE) {
-    if(cn->flags & CN_EXPANDED)
-      for(n = 0; n < cn->children.nvec; ++n)
-        recurse_selected(cn->children.vec[n], v);
-  } else {
-    if((cn->flags & CN_SELECTED) && cn->path)
-      vector_append(v, (char *)cn->path);
+  gtk_tree_model_get_iter(GTK_TREE_MODEL(choose_store), it, path);
+  if(choose_can_autocollapse(it)) {
+    /*fprintf(stderr, "collapse %s\n",
+            gtk_tree_path_to_string(path));*/
+    gtk_tree_store_set(choose_store, it,
+                       AUTOCOLLAPSE_COLUMN, FALSE,
+                       -1);
+    gtk_tree_view_collapse_row(tree_view, path);
   }
 }
 
   }
 }
 
-static char **gather_selected(int *ntracks) {
-  struct vector v;
-
-  vector_init(&v);
-  recurse_selected(root, &v);
-  vector_terminate(&v);
-  if(ntracks) *ntracks = v.nvec;
-  return v.vec;
-}
-
-static void activate_play(GtkMenuItem attribute((unused)) *menuitem,
-                          gpointer attribute((unused)) user_data) {
-  char **tracks = gather_selected(0);
-  int n;
-  
-  gtk_label_set_text(GTK_LABEL(report_label), "adding track to queue");
-  for(n = 0; tracks[n]; ++n)
-    disorder_eclient_play(client, tracks[n], 0, 0);
-}
-
-#if 0
-static void activate_remove(GtkMenuItem attribute((unused)) *menuitem,
-                            gpointer attribute((unused)) user_data) {
-  /* TODO remove all selected tracks */
-}
-#endif
-
-static void activate_properties(GtkMenuItem attribute((unused)) *menuitem,
-                                gpointer attribute((unused)) user_data) {
-  int ntracks;
-  char **tracks = gather_selected(&ntracks);
-
-  properties(ntracks, tracks);
-}
-
-static gboolean sensitive_play(struct choosenode attribute((unused)) *cn) {
-  return !!files_selected;
-}
-
-#if 0
-static gboolean sensitive_remove(struct choosenode attribute((unused)) *cn) {
-  return FALSE;                         /* not implemented yet */
-}
-#endif
-
-static gboolean sensitive_properties(struct choosenode attribute((unused)) *cn) {
-  return !!files_selected;
-}
-
-/* Main menu plumbing ------------------------------------------------------ */
-
-static int choose_properties_sensitive(GtkWidget attribute((unused)) *w) {
-  return !!files_selected;
+/** @brief Perform automatic collapse after a search is cleared */
+void choose_auto_collapse(void) {
+  gtk_tree_view_map_expanded_rows(GTK_TREE_VIEW(choose_view),
+                                  choose_auto_collapse_callback,
+                                  0);
 }
 
 }
 
-static int choose_selectall_sensitive(GtkWidget attribute((unused)) *w) {
-  return FALSE;                         /* TODO */
-}
+/** @brief Called from choose_refill() with each expanded row */
+static void choose_refill_callback(GtkTreeView attribute((unused)) *tree_view,
+                                   GtkTreePath *path,
+                                   gpointer attribute((unused)) user_data) {
+  GtkTreeIter it[1];
 
 
-static void choose_properties_activate(GtkWidget attribute((unused)) *w) {
-  activate_properties(0, 0);
+  gtk_tree_model_get_iter(GTK_TREE_MODEL(choose_store), it, path);
+  choose_refill_row(path, it);
 }
 
 }
 
-static void choose_selectall_activate(GtkWidget attribute((unused)) *w) {
-  /* TODO */
+/** @brief Synchronize all visible data with the server
+ *
+ * Called at startup, when a rescan completes, and via periodic_slow().
+ */
+static void choose_refill(const char attribute((unused)) *event,
+                          void attribute((unused)) *eventdata,
+                          void attribute((unused)) *callbackdata) {
+  //fprintf(stderr, "choose_refill\n");
+  /* Update the root */
+  disorder_eclient_files(client, choose_files_completed, "", NULL, NULL); 
+  disorder_eclient_dirs(client, choose_dirs_completed, "", NULL, NULL); 
+  choose_list_in_flight += 2;
+  /* Update all expanded rows */
+  gtk_tree_view_map_expanded_rows(GTK_TREE_VIEW(choose_view),
+                                  choose_refill_callback,
+                                  0);
+  //fprintf(stderr, "choose_list_in_flight -> %d+\n", choose_list_in_flight);
+}
+
+/** @brief Called for key-*-event on the main view
+ */
+static gboolean choose_key_event(GtkWidget attribute((unused)) *widget,
+                                 GdkEventKey *event,
+                                 gpointer user_data) {
+  /*fprintf(stderr, "choose_key_event type=%d state=%#x keyval=%#x\n",
+          event->type, event->state, event->keyval);*/
+  switch(event->keyval) {
+  case GDK_Page_Up:
+  case GDK_Page_Down:
+  case GDK_Up:
+  case GDK_Down:
+  case GDK_Home:
+  case GDK_End:
+    return FALSE;                       /* We'll take these */
+  case 'f': case 'F':
+    /* ^F is expected to start a search.  We implement this by focusing the
+     * search entry box. */
+    if((event->state & ~(GDK_LOCK_MASK|GDK_SHIFT_MASK)) == GDK_CONTROL_MASK
+       && event->type == GDK_KEY_PRESS) {
+      choose_search_new();
+      return TRUE;                      /* Handled it */
+    }
+    break;
+  case 'g': case 'G':
+    /* ^G is expected to go the next match.  We simulate a click on the 'next'
+     * button. */
+    if((event->state & ~(GDK_LOCK_MASK|GDK_SHIFT_MASK)) == GDK_CONTROL_MASK
+       && event->type == GDK_KEY_PRESS) {
+      choose_next_clicked(0, 0);
+      return TRUE;                      /* Handled it */
+    }
+    break;
+  }
+  /* Anything not handled we redirected to the search entry field */
+  gtk_widget_event(user_data, (GdkEvent *)event);
+  return TRUE;                          /* Handled it */
 }
 
 }
 
-static const struct tabtype tabtype_choose = {
-  choose_properties_sensitive,
-  choose_selectall_sensitive,
-  choose_properties_activate,
-  choose_selectall_activate,
-};
-
-/* Public entry points ----------------------------------------------------- */
-
-/* Create a track choice widget */
+/** @brief Create the choose tab */
 GtkWidget *choose_widget(void) {
 GtkWidget *choose_widget(void) {
-  int n;
-  GtkWidget *scrolled;
-  GtkWidget *vbox, *hbox, *clearsearch;
-
-  /*
-   *   +--vbox-------------------------------------------------------+
-   *   | +-hbox----------------------------------------------------+ |
-   *   | | searchentry                               | clearsearch | |
-   *   | +---------------------------------------------------------+ |
-   *   | +-scrolled------------------------------------------------+ |
-   *   | | +-chooselayout------------------------------------++--+ | |
-   *   | | | Tree structure is manually layed out in here    ||^^| | |
-   *   | | |                                                 ||  | | |
-   *   | | |                                                 ||  | | |
-   *   | | |                                                 ||  | | |
-   *   | | |                                                 ||vv| | |
-   *   | | +-------------------------------------------------++--+ | |
-   *   | | +-------------------------------------------------+     | |
-   *   | | |<                                               >|     | |
-   *   | | +-------------------------------------------------+     | |
-   *   | +---------------------------------------------------------+ |
-   *   +-------------------------------------------------------------+
-   */
+  /* Create the tree store. */
+  choose_store = gtk_tree_store_new(CHOOSE_COLUMNS,
+                                    G_TYPE_BOOLEAN,
+                                    G_TYPE_STRING,
+                                    G_TYPE_STRING,
+                                    G_TYPE_BOOLEAN,
+                                    G_TYPE_STRING,
+                                    G_TYPE_STRING,
+                                    G_TYPE_STRING,
+                                    G_TYPE_STRING,
+                                    G_TYPE_BOOLEAN);
+
+  /* Create the view */
+  choose_view = gtk_tree_view_new_with_model(GTK_TREE_MODEL(choose_store));
+  gtk_tree_view_set_rules_hint(GTK_TREE_VIEW(choose_view), TRUE);
+  /* Suppress built-in typeahead find, we do our own search support. */
+  gtk_tree_view_set_enable_search(GTK_TREE_VIEW(choose_view), FALSE);
+
+  /* Create cell renderers and columns */
+  /* TODO use a table */
+  {
+    GtkCellRenderer *r = gtk_cell_renderer_toggle_new();
+    GtkTreeViewColumn *c = gtk_tree_view_column_new_with_attributes
+      ("Queued",
+       r,
+       "active", STATE_COLUMN,
+       "visible", ISFILE_COLUMN,
+       (char *)0);
+    gtk_tree_view_column_set_resizable(c, TRUE);
+    gtk_tree_view_column_set_reorderable(c, TRUE);
+    gtk_tree_view_append_column(GTK_TREE_VIEW(choose_view), c);
+    g_signal_connect(r, "toggled",
+                     G_CALLBACK(choose_state_toggled), 0);
+  }
+  {
+    GtkCellRenderer *r = gtk_cell_renderer_text_new();
+    GtkTreeViewColumn *c = gtk_tree_view_column_new_with_attributes
+      ("Length",
+       r,
+       "text", LENGTH_COLUMN,
+       (char *)0);
+    gtk_tree_view_column_set_resizable(c, TRUE);
+    gtk_tree_view_column_set_reorderable(c, TRUE);
+    g_object_set(r, "xalign", (gfloat)1.0, (char *)0);
+    gtk_tree_view_append_column(GTK_TREE_VIEW(choose_view), c);
+  }
+  {
+    GtkCellRenderer *r = gtk_cell_renderer_text_new();
+    GtkTreeViewColumn *c = gtk_tree_view_column_new_with_attributes
+      ("Track",
+       r,
+       "text", NAME_COLUMN,
+       "background", BG_COLUMN,
+       "foreground", FG_COLUMN,
+       (char *)0);
+    gtk_tree_view_column_set_resizable(c, TRUE);
+    gtk_tree_view_column_set_reorderable(c, TRUE);
+    g_object_set(c, "expand", TRUE, (char *)0);
+    gtk_tree_view_append_column(GTK_TREE_VIEW(choose_view), c);
+    gtk_tree_view_set_expander_column(GTK_TREE_VIEW(choose_view), c);
+  }
   
   
-  /* Text entry box for search terms */
-  NW(entry);
-  searchentry = gtk_entry_new();
-  g_signal_connect(searchentry, "changed", G_CALLBACK(searchentry_changed), 0);
-
-  /* Cancel button to clear the search */
-  NW(button);
-  clearsearch = gtk_button_new_from_stock(GTK_STOCK_CANCEL);
-  g_signal_connect(G_OBJECT(clearsearch), "clicked",
-                   G_CALLBACK(clearsearch_clicked), 0);
-
-  /* hbox packs the search box and the cancel button together on a line */
-  NW(hbox);
-  hbox = gtk_hbox_new(FALSE/*homogeneous*/, 1/*spacing*/);
-  gtk_box_pack_start(GTK_BOX(hbox), searchentry,
+  /* The selection should support multiple things being selected */
+  choose_selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(choose_view));
+  gtk_tree_selection_set_mode(choose_selection, GTK_SELECTION_MULTIPLE);
+
+  /* Catch button presses */
+  g_signal_connect(choose_view, "button-press-event",
+                   G_CALLBACK(choose_button_event), 0);
+  g_signal_connect(choose_view, "button-release-event",
+                   G_CALLBACK(choose_button_event), 0);
+  /* Catch row expansions so we can fill in placeholders */
+  g_signal_connect(choose_view, "row-expanded",
+                   G_CALLBACK(choose_row_expanded), 0);
+
+  event_register("queue-list-changed", choose_set_state, 0);
+  event_register("playing-track-changed", choose_set_state, 0);
+  event_register("search-results-changed", choose_set_state, 0);
+  event_register("lookups-completed", choose_set_state, 0);
+
+  /* After a rescan we update the choose tree.  We get a rescan-complete
+   * automatically at startup and upon connection too. */
+  event_register("rescan-complete", choose_refill, 0);
+
+  /* Make the widget scrollable */
+  GtkWidget *scrolled = scroll_widget(choose_view);
+
+  /* Pack vertically with the search widget */
+  GtkWidget *vbox = gtk_vbox_new(FALSE/*homogenous*/, 1/*spacing*/);
+  gtk_box_pack_start(GTK_BOX(vbox), scrolled,
                      TRUE/*expand*/, TRUE/*fill*/, 0/*padding*/);
                      TRUE/*expand*/, TRUE/*fill*/, 0/*padding*/);
-  gtk_box_pack_end(GTK_BOX(hbox), clearsearch,
+  gtk_box_pack_end(GTK_BOX(vbox), choose_search_widget(),
                    FALSE/*expand*/, FALSE/*fill*/, 0/*padding*/);
   
                    FALSE/*expand*/, FALSE/*fill*/, 0/*padding*/);
   
-  /* chooselayout contains the currently visible subset of the track
-   * namespace */
-  NW(layout);
-  chooselayout = gtk_layout_new(0, 0);
-  root = newnode(0/*parent*/, "<root>", "All files", "",
-                 CN_EXPANDABLE, fill_root_node);
-  realroot = root;
-  expand_node(root);                    /* will call redisplay_tree */
-  /* Create the popup menu */
-  NW(menu);
-  menu = gtk_menu_new();
-  g_signal_connect(menu, "destroy", G_CALLBACK(gtk_widget_destroyed), &menu);
-  for(n = 0; n < NMENUITEMS; ++n) {
-    NW(menu_item);
-    menuitems[n].w = gtk_menu_item_new_with_label(menuitems[n].name);
-    gtk_menu_attach(GTK_MENU(menu), menuitems[n].w, 0, 1, n, n + 1);
-  }
-  /* The layout is scrollable */
-  scrolled = scroll_widget(GTK_WIDGET(chooselayout), "choose");
+  g_object_set_data(G_OBJECT(vbox), "type", (void *)&choose_tabtype);
 
 
-  /* The scrollable layout and the search hbox go together in a vbox */
-  NW(vbox);
-  vbox = gtk_vbox_new(FALSE/*homogenous*/, 1/*spacing*/);
-  gtk_box_pack_start(GTK_BOX(vbox), hbox,
-                     FALSE/*expand*/, FALSE/*fill*/, 0/*padding*/);
-  gtk_box_pack_end(GTK_BOX(vbox), scrolled,
-                   TRUE/*expand*/, TRUE/*fill*/, 0/*padding*/);
+  /* Redirect keyboard activity to the search widget */
+  g_signal_connect(choose_view, "key-press-event",
+                   G_CALLBACK(choose_key_event), choose_search_entry);
+  g_signal_connect(choose_view, "key-release-event",
+                   G_CALLBACK(choose_key_event), choose_search_entry);
 
 
-  g_object_set_data(G_OBJECT(vbox), "type", (void *)&tabtype_choose);
   return vbox;
 }
 
   return vbox;
 }
 
-/* Called when something we care about here might have changed */
-void choose_update(void) {
-  redisplay_tree();
-}
-
 /*
 Local Variables:
 c-basic-offset:2
 /*
 Local Variables:
 c-basic-offset:2