X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~mdw/git/disorder/blobdiff_plain/a3602333d0e12150d84396b1e5a1841ea3689e96..34fe753171d9b0ae90090565e4043ffaf2b02a52:/disobedience/choose-menu.c diff --git a/disobedience/choose-menu.c b/disobedience/choose-menu.c index a4e5430..44ad04f 100644 --- a/disobedience/choose-menu.c +++ b/disobedience/choose-menu.c @@ -2,164 +2,112 @@ * This file is part of DisOrder * 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 - * 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. * - * 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 - * 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 . + */ +/** @file disobedience/choose-menu.c + * @brief Popup menu for choose screen */ #include "disobedience.h" #include "popup.h" #include "choose.h" -VECTOR_TYPE(cdvector, struct choosedata *, xrealloc); - /** @brief Popup menu */ static GtkWidget *choose_menu; -/** @brief Callback for choose_get_selected() */ -static void choose_gather_selected_callback(GtkTreeModel attribute((unused)) *model, - GtkTreePath attribute((unused)) *path, - GtkTreeIter *iter, - gpointer data) { - struct cdvector *v = data; - struct choosedata *cd = choose_iter_to_data(iter); +/** @brief Path to directory pending a "select children" operation */ +static GtkTreePath *choose_eventually_select_children; - if(cd) - cdvector_append(v, cd); +/** @brief Should edit->select all be sensitive? No, for the choose tab. */ +static int choose_selectall_sensitive(void attribute((unused)) *extra) { + return FALSE; } -/** @brief Get a list of all selected tracks and directories */ -static struct choosedata **choose_get_selected(int *nselected) { - struct cdvector v[1]; - - cdvector_init(v); - gtk_tree_selection_selected_foreach(choose_selection, - choose_gather_selected_callback, - v); - cdvector_terminate(v); - if(nselected) - *nselected = v->nvec; - return v->vec; +/** @brief Activate edit->select all (which should do nothing) */ +static void choose_selectall_activate(GtkMenuItem attribute((unused)) *item, + gpointer attribute((unused)) userdata) { } -/** @brief Recursion step for choose_get_visible() - * @param parent A visible node, or NULL for the root - * @param cdv Visible nodes accumulated here +/** @brief Should 'select none' be sensitive + * + * Yes if anything is selected. */ -static void choose_visible_recurse(GtkTreeIter *parent, - void (*callback)(GtkTreeIter *it, - struct choosedata *cd, - void *userdata), - void *userdata) { - struct choosedata *cd; - int expanded; - if(parent) { - cd = choose_iter_to_data(parent); - callback(parent, cd, userdata); - if(cd->type != CHOOSE_DIRECTORY) - /* Only directories can be expanded so we can avoid the more - * expensive test below */ - return; - GtkTreePath *parent_path - = gtk_tree_model_get_path(GTK_TREE_MODEL(choose_store), - parent); - expanded = gtk_tree_view_row_expanded(GTK_TREE_VIEW(choose_view), - parent_path); - gtk_tree_path_free(parent_path); - } else - expanded = 1; - /* See if parent is expanded */ - if(expanded) { - /* Parent is expanded, visit all its children */ - GtkTreeIter it[1]; - gboolean itv = gtk_tree_model_iter_children(GTK_TREE_MODEL(choose_store), - it, - parent); - while(itv) { - choose_visible_recurse(it, callback, userdata); - itv = gtk_tree_model_iter_next(GTK_TREE_MODEL(choose_store), it); - } - } -} - -static void choose_visible_visit(void (*callback)(GtkTreeIter *it, - struct choosedata *cd, - void *userdata), - void *userdata) { - choose_visible_recurse(NULL, callback, userdata); +static int choose_selectnone_sensitive(void attribute((unused)) *extra) { + return gtk_tree_selection_count_selected_rows(choose_selection) > 0; } -static void count_choosedatas(struct choosedata **cds, - int counts[2]) { - struct choosedata *cd; - counts[CHOOSE_FILE] = counts[CHOOSE_DIRECTORY] = 0; - while((cd = *cds++)) - ++counts[cd->type]; +/** @brief Activate select none */ +static void choose_selectnone_activate(GtkMenuItem attribute((unused)) *item, + gpointer attribute((unused)) userdata) { + gtk_tree_selection_unselect_all(choose_selection); } +static void choose_play_sensitive_callback(GtkTreeModel attribute((unused)) *model, + GtkTreePath attribute((unused)) *path, + GtkTreeIter *iter, + gpointer data) { + int *filesp = data; -static void choose_selectall_sensitive_callback - (GtkTreeIter attribute((unused)) *it, - struct choosedata *cd, - void *userdata) { - if(cd->type == CHOOSE_FILE) - ++*(int *)userdata; + if(*filesp == -1) + return; + if(choose_is_dir(iter)) + *filesp = -1; + else if(choose_is_file(iter)) + ++*filesp; } -static int choose_selectall_sensitive(void attribute((unused)) *extra) { +/** @brief Should 'play' be sensitive? + * + * Yes if tracks are selected and no directories are */ +static int choose_play_sensitive(void attribute((unused)) *extra) { int files = 0; - choose_visible_visit(choose_selectall_sensitive_callback, &files); + + gtk_tree_selection_selected_foreach(choose_selection, + choose_play_sensitive_callback, + &files); return files > 0; } -static void choose_selectall_activate_callback - (GtkTreeIter *it, - struct choosedata *cd, - void attribute((unused)) *userdata) { - if(cd->type == CHOOSE_FILE) - gtk_tree_selection_select_iter(choose_selection, it); - else - gtk_tree_selection_unselect_iter(choose_selection, it); -} +static void choose_gather_selected_files_callback(GtkTreeModel attribute((unused)) *model, + GtkTreePath attribute((unused)) *path, + GtkTreeIter *iter, + gpointer data) { + struct vector *v = data; -static void choose_selectall_activate(GtkMenuItem attribute((unused)) *item, - gpointer attribute((unused)) userdata) { - choose_visible_visit(choose_selectall_activate_callback, 0); + if(choose_is_file(iter)) + vector_append(v, choose_get_track(iter)); } - -static int choose_selectnone_sensitive(void attribute((unused)) *extra) { - return gtk_tree_selection_count_selected_rows(choose_selection) > 0; -} - -static void choose_selectnone_activate(GtkMenuItem attribute((unused)) *item, - gpointer attribute((unused)) userdata) { - gtk_tree_selection_unselect_all(choose_selection); -} - -static int choose_play_sensitive(void attribute((unused)) *extra) { - int counts[2]; - count_choosedatas(choose_get_selected(NULL), counts); - return !counts[CHOOSE_DIRECTORY] && counts[CHOOSE_FILE]; + +static void choose_gather_selected_dirs_callback(GtkTreeModel attribute((unused)) *model, + GtkTreePath attribute((unused)) *path, + GtkTreeIter *iter, + gpointer data) { + struct vector *v = data; + + if(choose_is_dir(iter)) + vector_append(v, choose_get_track(iter)); } + static void choose_play_activate(GtkMenuItem attribute((unused)) *item, gpointer attribute((unused)) userdata) { - struct choosedata *cd, **cdp = choose_get_selected(NULL); - while((cd = *cdp++)) { - if(cd->type == CHOOSE_FILE) - disorder_eclient_play(client, xstrdup(cd->track), - choose_play_completed, 0); - } + struct vector v[1]; + vector_init(v); + gtk_tree_selection_selected_foreach(choose_selection, + choose_gather_selected_files_callback, + v); + for(int n = 0; n < v->nvec; ++n) + disorder_eclient_play(client, v->vec[n], choose_play_completed, 0); } static int choose_properties_sensitive(void *extra) { @@ -168,12 +116,98 @@ static int choose_properties_sensitive(void *extra) { static void choose_properties_activate(GtkMenuItem attribute((unused)) *item, gpointer attribute((unused)) userdata) { - struct choosedata *cd, **cdp = choose_get_selected(NULL); struct vector v[1]; vector_init(v); - while((cd = *cdp++)) - vector_append(v, xstrdup(cd->track)); - properties(v->nvec, (const char **)v->vec); + gtk_tree_selection_selected_foreach(choose_selection, + choose_gather_selected_files_callback, + v); + properties(v->nvec, (const char **)v->vec, toplevel); +} + +/** @brief Set sensitivity for select children + * + * Sensitive if we've selected exactly one directory. + */ +static int choose_selectchildren_sensitive(void attribute((unused)) *extra) { + struct vector v[1]; + /* Only one thing should be selected */ + if(gtk_tree_selection_count_selected_rows(choose_selection) != 1) + return FALSE; + /* The selected thing should be a directory */ + vector_init(v); + gtk_tree_selection_selected_foreach(choose_selection, + choose_gather_selected_dirs_callback, + v); + return v->nvec == 1; +} + +/** @brief Actually select the children of path + * + * We deselect everything else, too. + */ +static void choose_select_children(GtkTreePath *path) { + GtkTreeIter iter[1], child[1]; + + if(gtk_tree_model_get_iter(GTK_TREE_MODEL(choose_store), iter, path)) { + gtk_tree_selection_unselect_all(choose_selection); + for(int n = 0; + gtk_tree_model_iter_nth_child(GTK_TREE_MODEL(choose_store), child, + iter, n); + ++n) { + if(choose_is_file(child)) + gtk_tree_selection_select_iter(choose_selection, child); + } + } +} + +/** @brief Called to expand the children of path/iter */ +static void choose_selectchildren_callback(GtkTreeModel attribute((unused)) *model, + GtkTreePath *path, + GtkTreeIter attribute((unused)) *iter, + gpointer attribute((unused)) data) { + if(gtk_tree_view_row_expanded(GTK_TREE_VIEW(choose_view), path)) { + /* Directory is already expanded */ + choose_select_children(path); + } else { + /* Directory is not expanded, so expand it */ + gtk_tree_view_expand_row(GTK_TREE_VIEW(choose_view), path, FALSE/*!expand_all*/); + /* Select its children when it's done */ + if(choose_eventually_select_children) + gtk_tree_path_free(choose_eventually_select_children); + choose_eventually_select_children = gtk_tree_path_copy(path); + } +} + +/** @brief Called when all pending track fetches are finished + * + * If there's a pending select-children operation, it can now be actioned + * (or might have gone stale). + */ +void choose_menu_moretracks(const char attribute((unused)) *event, + void attribute((unused)) *eventdata, + void attribute((unused)) *callbackdata) { + if(choose_eventually_select_children) { + choose_select_children(choose_eventually_select_children); + gtk_tree_path_free(choose_eventually_select_children); + choose_eventually_select_children = 0; + } +} + +/** @brief Select all children + * + * Easy enough if the directory is already expanded, we can just select its + * children. However if it is not then we must expand it and _when this has + * completed_ select its children. + * + * The way this is implented could cope with multiple directories but + * choose_selectchildren_sensitive() should stop this. + */ +static void choose_selectchildren_activate + (GtkMenuItem attribute((unused)) *item, + gpointer attribute((unused)) userdata) { + gtk_tree_selection_selected_foreach(choose_selection, + choose_selectchildren_callback, + 0); } /** @brief Pop-up menu for choose */ @@ -193,9 +227,9 @@ static struct menuitem choose_menuitems[] = { 0 }, { - "Select all tracks", - choose_selectall_activate, - choose_selectall_sensitive, + "Select children", + choose_selectchildren_activate, + choose_selectchildren_sensitive, 0, 0 },