OSDN Git Service

LinGui: nested presets and drag n drop re-ordering of presets
authorjstebbins <jstebbins@b64f7644-9d1e-0410-96f1-a4d463321fa5>
Sun, 5 Oct 2008 22:03:09 +0000 (22:03 +0000)
committerjstebbins <jstebbins@b64f7644-9d1e-0410-96f1-a4d463321fa5>
Sun, 5 Oct 2008 22:03:09 +0000 (22:03 +0000)
adding a preset always places it at the top level. drag it to where
you want it.

git-svn-id: svn://localhost/HandBrake/trunk@1816 b64f7644-9d1e-0410-96f1-a4d463321fa5

gtk/src/callbacks.c
gtk/src/ghb.ui
gtk/src/internal_defaults.xml
gtk/src/main.c
gtk/src/presets.c
gtk/src/presets.h
gtk/src/resource_data.h
gtk/src/resources.plist
gtk/src/settings.h
gtk/src/values.c
gtk/src/values.h

index 985c1e1..e634e0a 100644 (file)
@@ -1039,6 +1039,7 @@ title_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
        ghb_title_info_t tinfo;
        gint titleindex;
        gchar *preset;
+       gchar *folder;
        
        g_debug("title_changed_cb ()");
        ghb_widget_to_setting(ud->settings, widget);
@@ -1049,8 +1050,10 @@ title_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
        ghb_update_ui_combo_box (ud->builder, "subtitle_lang", titleindex, FALSE);
 
        preset = ghb_settings_get_string (ud->settings, "preset");
-       ghb_update_from_preset(ud, preset, "subtitle_lang");
+       folder = ghb_settings_get_string (ud->settings, "folder");
+       ghb_update_from_preset(ud, folder, preset, "subtitle_lang");
        g_free(preset);
+       g_free(folder);
        if (ghb_get_title_info (&tinfo, titleindex))
        {
                show_title_info(ud, &tinfo);
index 1a5c4af..6e53e8e 100644 (file)
@@ -3703,6 +3703,7 @@ this setting.</property>
                     <property name="activates_default">True</property>
                     <property name="width_chars">30</property>
                     <property name="truncate_multiline">True</property>
+                    <signal handler="preset_name_changed_cb" name="changed"/>
                   </object>
                   <packing>
                     <property name="position">1</property>
@@ -3729,7 +3730,7 @@ this setting.</property>
                     <property name="right_padding">4</property>
                     <child>
                       <object class="GtkTextView" id="preset_description">
-                        <property name="height_request">50</property>
+                        <property name="height_request">60</property>
                         <property name="visible">True</property>
                         <property name="can_focus">True</property>
                         <property name="events">GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK</property>
@@ -3753,6 +3754,43 @@ this setting.</property>
                 <property name="position">1</property>
               </packing>
             </child>
+            <child>
+              <object class="GtkHBox" id="hbox43">
+                <property name="visible">True</property>
+                <child>
+                  <object class="GtkRadioButton" id="preset_type_folder">
+                    <property name="visible">True</property>
+                    <property name="can_focus">True</property>
+                    <property name="label" translatable="yes">Folder</property>
+                    <property name="active">True</property>
+                    <property name="draw_indicator">True</property>
+                    <signal name="toggled" handler="preset_type_changed_cb"/>
+                  </object>
+                  <packing>
+                    <property name="expand">False</property>
+                  </packing>
+                </child>
+                <child>
+                  <object class="GtkRadioButton" id="preset_type_normal">
+                    <property name="visible">True</property>
+                    <property name="can_focus">True</property>
+                    <property name="label" translatable="yes">Preset</property>
+                    <property name="active">True</property>
+                    <property name="draw_indicator">True</property>
+                    <property name="group">preset_type_folder</property>
+                    <signal name="toggled" handler="preset_type_changed_cb"/>
+                  </object>
+                  <packing>
+                    <property name="expand">False</property>
+                    <property name="position">1</property>
+                  </packing>
+                </child>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="position">2</property>
+              </packing>
+            </child>
           </object>
           <packing>
             <property name="position">1</property>
index ed66ec1..822ed41 100644 (file)
                <integer>0</integer>
                <key>end_chapter</key>
                <integer>100</integer>
+               <key>folder</key>
+               <string></string>
                <key>preset</key>
                <string>Normal</string>
+               <key>preset_type_folder</key>
+               <false />
+               <key>preset_type_normal</key>
+               <true />
                <key>scale_height</key>
                <integer>480</integer>
                <key>scale_width</key>
@@ -74,6 +80,8 @@
                <false />
                <key>chapters_in_destination</key>
                <false />
+               <key>default_folder</key>
+               <string></string>
                <key>default_preset</key>
                <string>Normal</string>
                <key>default_source</key>
index f54e8c7..705bc22 100644 (file)
@@ -357,6 +357,8 @@ bind_audio_tree_model (signal_user_data_t *ud)
 }
 
 extern void presets_list_selection_changed_cb(void);
+extern void presets_drag_cb(void);
+extern void presets_drag_motion_cb(void);
 
 // Create and bind the tree model to the tree view for the preset list
 // Also, connect up the signal that lets us know the selection has changed
@@ -369,6 +371,9 @@ bind_presets_tree_model (signal_user_data_t *ud)
        GtkTreeView  *treeview;
        GtkTreeSelection *selection;
        GtkWidget *widget;
+       GtkTargetEntry SrcEntry;
+       SrcEntry.target = "DATA";
+       SrcEntry.flags = GTK_TARGET_SAME_WIDGET;
 
        g_debug("bind_presets_tree_model ()\n");
        treeview = GTK_TREE_VIEW(GHB_WIDGET (ud->builder, "presets_list"));
@@ -383,6 +388,14 @@ bind_presets_tree_model (signal_user_data_t *ud)
     gtk_tree_view_append_column(treeview, GTK_TREE_VIEW_COLUMN(column));
        gtk_tree_view_column_set_expand (column, TRUE);
        gtk_tree_view_set_tooltip_column (treeview, 4);
+
+       gtk_tree_view_enable_model_drag_dest (treeview, &SrcEntry, 1, 
+                                                                                       GDK_ACTION_MOVE);
+       gtk_tree_view_enable_model_drag_source (treeview, GDK_BUTTON1_MASK, 
+                                                                                       &SrcEntry, 1, GDK_ACTION_MOVE);
+
+       g_signal_connect(treeview, "drag_data_received", presets_drag_cb, ud);
+       g_signal_connect(treeview, "drag_motion", presets_drag_motion_cb, ud);
        g_signal_connect(selection, "changed", presets_list_selection_changed_cb, ud);
        widget = GHB_WIDGET (ud->builder, "presets_remove");
        gtk_widget_set_sensitive(widget, FALSE);
@@ -474,6 +487,7 @@ main (int argc, char *argv[])
        GtkWidget *window;
        signal_user_data_t *ud;
        gchar *preset;
+       gchar *folder;
        GError *error = NULL;
        GOptionContext *context;
 
@@ -575,17 +589,19 @@ main (int argc, char *argv[])
        ghb_x264_parse_options(ud, "");
 
        // Populate the presets tree view
-       ghb_presets_list_update(ud);
+       ghb_presets_list_init(ud, NULL, NULL, NULL);
        // Get the first preset name
        if (arg_preset != NULL)
        {
-               ghb_select_preset(ud->builder, arg_preset);
+               ghb_select_preset(ud->builder, NULL, arg_preset);
        }
        else
        {
                preset = ghb_settings_get_string (ud->settings, "default_preset");
-               ghb_select_preset(ud->builder, preset);
+               folder = ghb_settings_get_string (ud->settings, "default_folder");
+               ghb_select_preset(ud->builder, folder, preset);
                g_free(preset);
+               g_free(folder);
        }
 
        // Grey out widgets that are dependent on a disabled feature
index 064b016..d6dcb78 100644 (file)
@@ -17,6 +17,7 @@
 #include <string.h>
 #include <gtk/gtk.h>
 #include "settings.h"
+#include "callbacks.h"
 #include "audiohandler.h"
 #include "hb-backend.h"
 #include "plist.h"
 #include "presets.h"
 #include "values.h"
 
+// These are flags.  One bit for each feature
+enum
+{
+       PRESETS_CUST = 0x01,
+       PRESETS_FOLDER = 0x02,
+};
+
 static GValue *presetsPlist = NULL;
 static GValue *internalPlist = NULL;
 static GValue *prefsPlist = NULL;
@@ -43,48 +51,150 @@ preset_get_name(GValue *dict)
        return g_value_get_string(ghb_dict_lookup(dict, "preset_name"));
 }
 
+gint
+ghb_preset_flags(GValue *dict)
+{
+       const GValue *gval;
+       gint ptype = 0;
+
+       gval = preset_dict_get_value(dict, "preset_type");
+       if (gval)
+       {
+               ptype = ghb_value_int(gval);
+       }
+       return ptype;
+}
+
 static GValue*
 presets_get_first_dict(GValue *presets)
 {
-       GValue *dict;
-       gint count, ii, ptype;
+       gint count;
        
+       g_debug("presets_get_first_dict ()");
        if (presets == NULL) return NULL;
        count = ghb_array_len(presets);
-       for (ii = 0; ii < count; ii++)
+       if (count <= 0) return NULL;
+       return ghb_array_get_nth(presets, 0);
+}
+
+static void
+presets_remove_nth(GValue *presets, gint pos)
+{
+       GValue *dict;
+       gint count;
+       
+       if (presets == NULL || pos < 0) return;
+       count = ghb_array_len(presets);
+       if (pos >= count) return;
+       dict = ghb_array_get_nth(presets, pos);
+       ghb_array_remove(presets, pos);
+       ghb_value_free(dict);
+}
+
+gboolean
+ghb_presets_remove(
+       GValue *presets, 
+       gint folder_pos,
+       gint pos)
+{
+       GValue *nested;
+       gint ii;
+
+       if (folder_pos >= 0)
        {
-               dict = ghb_array_get_nth(presets, ii);
-               ptype = ghb_value_int(preset_dict_get_value(dict, "preset_type"));
-               if (ptype != 2)
-                       return dict;
+               if (pos >= 0)
+               {
+                       ii = pos;
+                       nested = ghb_array_get_nth(presets, folder_pos);
+                       nested = ghb_dict_lookup(nested, "preset_folder");
+               }
+               else
+               {
+                       ii = folder_pos;
+                       nested = presets;
+               }
+       }
+       else
+       {
+               ii = pos;
+               nested = presets;
+       }
+       if (ii >= 0)
+               presets_remove_nth(nested, ii);
+       else
+       {
+               g_warning("internal preset lookup error (%d/%d)", folder_pos, pos);
+               return FALSE;
        }
-       return NULL;
+       return TRUE;
 }
 
-static GValue*
-presets_get_dict(GValue *presets, const gchar *name)
+static void
+ghb_presets_replace(
+       GValue *presets, 
+       GValue *dict,
+       gint folder_pos,
+       gint pos)
+{
+       GValue *nested;
+       gint ii;
+
+       if (folder_pos >= 0)
+       {
+               if (pos >= 0)
+               {
+                       ii = pos;
+                       nested = ghb_array_get_nth(presets, folder_pos);
+                       nested = ghb_dict_lookup(nested, "preset_folder");
+               }
+               else
+               {
+                       ii = folder_pos;
+                       nested = presets;
+               }
+       }
+       else
+       {
+               ii = pos;
+               nested = presets;
+       }
+       if (ii >= 0)
+               ghb_array_replace(nested, ii, dict);
+       else
+       {
+               g_warning("internal preset lookup error (%d/%d)", folder_pos, pos);
+       }
+}
+
+static gint
+presets_find_pos(GValue *presets, const gchar *name, gint type)
 {
        GValue *dict;
-       gint count, ii;
+       gint count, ii, ptype, last;
        
-       if (presets == NULL || name == NULL) return NULL;
-       count = ghb_array_len(presets);
+       if (presets == NULL || name == NULL) return -1;
+       last = count = ghb_array_len(presets);
        for (ii = 0; ii < count; ii++)
        {
                const gchar *str;
                dict = ghb_array_get_nth(presets, ii);
                str = preset_get_name(dict);
-               if (strcmp(str, name) == 0)
-                       return dict;
+               ptype = ghb_value_int(preset_dict_get_value(dict, "preset_type"));
+               if (strcasecmp(name, str) <= 0 && ptype == type)
+               {
+                       return ii;
+               }
+               if (ptype == type)
+                       last = ii+1;
        }
-       return NULL;
+       return last;
 }
 
 static gint
-presets_remove(GValue *presets, const gchar *name)
+presets_find_folder(GValue *presets, const gchar *name)
 {
        GValue *dict;
-       gint count, ii;
+       gint count, ii, ptype;
        
        if (presets == NULL || name == NULL) return -1;
        count = ghb_array_len(presets);
@@ -93,9 +203,9 @@ presets_remove(GValue *presets, const gchar *name)
                const gchar *str;
                dict = ghb_array_get_nth(presets, ii);
                str = preset_get_name(dict);
-               if (strcmp(str, name) == 0)
+               ptype = ghb_value_int(preset_dict_get_value(dict, "preset_type"));
+               if ((ptype & PRESETS_FOLDER) && strcasecmp(name, str) == 0)
                {
-                       ghb_array_remove(presets, ii);
                        return ii;
                }
        }
@@ -103,38 +213,168 @@ presets_remove(GValue *presets, const gchar *name)
 }
 
 static gint
-presets_find_pos(GValue *presets, const gchar *name, gint type)
+presets_find_preset(GValue *presets, const gchar *name)
 {
        GValue *dict;
-       gint count, ii, ptype, last;
+       gint count, ii, ptype;
        
+       g_debug("presets_find_preset () (%s)", name);
        if (presets == NULL || name == NULL) return -1;
-       last = count = ghb_array_len(presets);
+       count = ghb_array_len(presets);
        for (ii = 0; ii < count; ii++)
        {
                const gchar *str;
                dict = ghb_array_get_nth(presets, ii);
                str = preset_get_name(dict);
                ptype = ghb_value_int(preset_dict_get_value(dict, "preset_type"));
-               if (strcasecmp(name, str) < 0 && ptype == type)
+               if (!(ptype & PRESETS_FOLDER) && strcasecmp(name, str) == 0)
                {
                        return ii;
                }
-               if (ptype == type)
-                       last = ii+1;
        }
-       return last;
+       return -1;
+}
+
+gboolean
+ghb_presets_find(
+       GValue *presets, 
+       const gchar *folder, 
+       const gchar *name, 
+       gint *folder_pos,
+       gint *pos)
+{
+       GValue *nested;
+
+       *pos = -1;
+       *folder_pos = -1;
+       g_debug("ghb_presets_find () (%s) (%s)", folder, name);
+       if (folder == NULL || folder[0] == 0)
+       {
+               // name could be a folder or a regular preset
+               *folder_pos = presets_find_folder(presets, name);
+               if (*folder_pos < 0)
+               {
+                       *pos = presets_find_preset(presets, name);
+                       if (*pos < 0)
+                       {
+                               g_debug("No presets/folder (%s)", name);
+                               return FALSE;
+                       }
+               }
+       }
+       else
+       {
+               *folder_pos = presets_find_folder(presets, folder);
+               if (*folder_pos < 0)
+               {
+                       g_debug("No presets folder (%s)", folder);
+                       return FALSE;
+               }
+               nested = ghb_array_get_nth(presets, *folder_pos);
+               nested = ghb_dict_lookup(nested, "preset_folder");
+               if (name != NULL)
+               {
+                       *pos = presets_find_preset(nested, name);
+                       if (*pos < 0)
+                       {
+                               g_debug("No preset (%s/%s)", folder, name);
+                               return FALSE;
+                       }
+               }
+       }
+       return TRUE;
+}
+
+static GValue*
+presets_get_dict(GValue *presets, gint folder_pos, gint pos)
+{
+       g_debug("presets_get_dict () (%d) (%d)", folder_pos, pos);
+       if (presets == NULL) return NULL;
+       GValue *nested;
+       gint ii;
+
+       if (folder_pos >= 0)
+       {
+               if (pos >= 0)
+               {
+                       ii = pos;
+                       nested = ghb_array_get_nth(presets, folder_pos);
+                       nested = ghb_dict_lookup(nested, "preset_folder");
+               }
+               else
+               {
+                       ii = folder_pos;
+                       nested = presets;
+               }
+       }
+       else
+       {
+               ii = pos;
+               nested = presets;
+       }
+       if (ii >= 0)
+               return ghb_array_get_nth(nested, ii);
+       else
+       {
+               g_warning("internal preset lookup error (%d/%d)", folder_pos, pos);
+               return NULL;
+       }
+}
+
+static gint
+ghb_presets_get_type(
+       GValue *presets, 
+       gint folder_pos,
+       gint pos)
+{
+       GValue *nested;
+       GValue *dict;
+       gint ii;
+       gint flags = 0;
+
+       if (folder_pos >= 0)
+       {
+               if (pos >= 0)
+               {
+                       ii = pos;
+                       nested = ghb_array_get_nth(presets, folder_pos);
+                       nested = ghb_dict_lookup(nested, "preset_folder");
+               }
+               else
+               {
+                       ii = folder_pos;
+                       nested = presets;
+               }
+       }
+       else
+       {
+               ii = pos;
+               nested = presets;
+       }
+       if (ii >= 0)
+       {
+               dict = ghb_array_get_nth(nested, ii);
+               flags = ghb_preset_flags(dict);
+       }
+       else
+       {
+               g_warning("internal preset lookup error (%d/%d)", folder_pos, pos);
+       }
+       return flags;
 }
 
 void
 ghb_set_preset_default(GValue *settings)
 {
-       gchar *preset;
+       gchar *preset, *folder;
        
        preset = ghb_settings_get_string (settings, "preset");
+       folder = ghb_settings_get_string (settings, "folder");
        ghb_settings_set_string(settings, "default_preset", preset);
+       ghb_settings_set_string(settings, "default_folder", folder);
        ghb_prefs_save(settings);
        g_free(preset);
+       g_free(folder);
 }
 
 // Used for sorting dictionaries.
@@ -147,13 +387,6 @@ key_cmp(gconstpointer a, gconstpointer b)
        return strcmp(stra, strb);
 }
 
-const gchar*
-ghb_presets_get_description(GValue *pdict)
-{
-       if (pdict == NULL) return g_strdup("");
-       return g_value_get_string(ghb_dict_lookup(pdict, "preset_description"));
-}
-
 static const GValue*
 preset_dict_get_value(GValue *dict, const gchar *key)
 {
@@ -173,29 +406,26 @@ preset_dict_get_value(GValue *dict, const gchar *key)
        return gval;
 }
 
-static const GValue*
-preset_get_value(const gchar *name, const gchar *key)
+const gchar*
+ghb_presets_get_description(GValue *pdict)
 {
-       GValue *dict;
-
-       dict = presets_get_dict(presetsPlist, name);
-       return preset_dict_get_value(dict, key);
+       if (pdict == NULL) return g_strdup("");
+       return g_value_get_string(
+               preset_dict_get_value(pdict, "preset_description"));
 }
 
-gint
-ghb_preset_flags(GValue *dict)
+
+static const GValue*
+preset_get_value(const gchar *folder, const gchar *name, const gchar *key)
 {
-       const GValue *gval;
-       gint ptype;
-       gint ret = 0;
+       GValue *dict = NULL;
+       gint folder_pos, pos;
 
-       gval = preset_dict_get_value(dict, "preset_type");
-       if (gval)
+       if (ghb_presets_find(presetsPlist, folder, name, &folder_pos, &pos))
        {
-               ptype = ghb_value_int(gval);
-               ret = (ptype != 0 ? PRESET_CUSTOM : 0);
+               dict = presets_get_dict(presetsPlist, folder_pos, pos);
        }
-       return ret;
+       return preset_dict_get_value(dict, key);
 }
 
 static void init_settings_from_dict(
@@ -363,19 +593,24 @@ ghb_settings_to_ui(signal_user_data_t *ud, GValue *dict)
 }
 
 void
-ghb_set_preset(signal_user_data_t *ud, const gchar *name)
+ghb_set_preset(signal_user_data_t *ud, const gchar *folder, const gchar *name)
 {
-       GValue *dict;
+       GValue *dict = NULL;
+       gint folder_pos, pos, ptype;
        
-       g_debug("ghb_set_preset() %s\n", name);
-       if (name == NULL)
+       g_debug("ghb_set_preset() %s %s\n", folder, name);
+       if (ghb_presets_find(presetsPlist, folder, name, &folder_pos, &pos))
+               dict = presets_get_dict(presetsPlist, folder_pos, pos);
+
+       if (dict == NULL)
        {
                dict = presets_get_first_dict(presetsPlist);
-               name = preset_get_name(dict);
-       }
-       else
-       {
-               dict = presets_get_dict(presetsPlist, name);
+               folder = NULL;
+               if (dict)
+                       name = preset_get_name(dict);
+               else
+                       name = NULL;
+               folder = "";
        }
        if (dict == NULL || name == NULL)
        {
@@ -383,14 +618,20 @@ ghb_set_preset(signal_user_data_t *ud, const gchar *name)
        }
        else
        {
-               preset_to_ui(ud, dict);
+               ptype = ghb_value_int(preset_dict_get_value(dict, "preset_type"));
+               if (ptype & PRESETS_FOLDER)
+                       preset_to_ui(ud, NULL);
+               else
+                       preset_to_ui(ud, dict);
                ghb_settings_set_string(ud->settings, "preset", name);
+               ghb_settings_set_string(ud->settings, "folder", folder);
        }
 }
 
 void
 ghb_update_from_preset(
        signal_user_data_t *ud, 
+       const gchar *folder, 
        const gchar *name, 
        const gchar *key)
 {
@@ -398,7 +639,7 @@ ghb_update_from_preset(
        
        g_debug("ghb_update_from_preset() %s %s", name, key);
        if (name == NULL) return;
-       gval = preset_get_value(name, key);
+       gval = preset_get_value(folder, name, key);
        if (gval != NULL)
        {
                ghb_ui_update(ud, key, gval);
@@ -707,6 +948,241 @@ ghb_prefs_load(signal_user_data_t *ud)
 }
 
 void
+ghb_presets_list_init(
+       signal_user_data_t *ud, 
+       GValue *presets, 
+       const gchar *parent_name,
+       GtkTreeIter *parent)
+{
+       GtkTreeView *treeview;
+       GtkTreeIter iter;
+       GtkTreeStore *store;
+       const gchar *preset;
+       gchar *def_preset, *def_folder;
+       const gchar *description;
+       gint flags, custom;
+       gboolean def;
+       gint count, ii, ptype;
+       GValue *dict;
+       
+       g_debug("ghb_presets_list_init ()");
+       if (presets == NULL)
+               presets = presetsPlist;
+       def_folder = ghb_settings_get_string(ud->settings, "default_folder");
+       def_preset = ghb_settings_get_string(ud->settings, "default_preset");
+       count = ghb_array_len(presets);
+       treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "presets_list"));
+       store = GTK_TREE_STORE(gtk_tree_view_get_model(treeview));
+       ii = 0;
+       while (ii < count)
+       {
+               // Additional settings, add row
+               g_debug("Adding rows");
+               dict = ghb_array_get_nth(presets, ii);
+               preset = preset_get_name(dict);
+               def = FALSE;
+               if (strcmp(preset, def_preset) == 0)
+               {
+                       if (parent_name && strcmp(parent_name, def_folder) == 0)
+                               def = TRUE;
+                       else if (parent_name == NULL && def_folder[0] == 0)
+                               def = TRUE;
+               }
+
+               description = ghb_presets_get_description(dict);
+               gtk_tree_store_append(store, &iter, parent);
+               flags = ghb_preset_flags(dict);
+               custom = flags & PRESETS_CUST;
+               gtk_tree_store_set(store, &iter, 0, preset, 
+                                                       1, def ? 800 : 400, 
+                                                       2, def ? 2 : 0,
+                                                       3, custom ? "black" : "blue", 
+                                                       4, description,
+                                                       -1);
+               if (def)
+               {
+                       GtkTreePath *path;
+
+                       path = gtk_tree_model_get_path(GTK_TREE_MODEL(store), parent);
+                       gtk_tree_view_expand_row(treeview, path, FALSE);
+                       gtk_tree_path_free(path);
+               }
+               ptype = ghb_value_int(preset_dict_get_value(dict, "preset_type"));
+               if (ptype & PRESETS_FOLDER)
+               {
+                       GValue *nested;
+                       nested = ghb_dict_lookup(dict, "preset_folder");
+                       if (nested != NULL)
+                               ghb_presets_list_init(ud, nested, preset, &iter);
+               }
+               ii++;
+       }
+       g_free(def_preset);
+       g_free(def_folder);
+}
+
+static void
+presets_list_update_item(
+       signal_user_data_t *ud, 
+       GValue *presets,
+       GtkTreeIter *iter,
+       gint folder_pos,
+       gint pos)
+{
+       GtkTreeView *treeview;
+       GtkTreeStore *store;
+       const gchar *preset;
+       gchar *def_preset, *def_folder;
+       const gchar *description;
+       gint flags, custom;
+       gboolean def;
+       GValue *dict;
+       const gchar *parent_name;
+       
+       g_debug("presets_list_update_item ()");
+       dict = presets_get_dict(presets, folder_pos, pos);
+       if (dict == NULL)
+               return;
+       def_folder = ghb_settings_get_string(ud->settings, "default_folder");
+       def_preset = ghb_settings_get_string(ud->settings, "default_preset");
+       treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "presets_list"));
+       store = GTK_TREE_STORE(gtk_tree_view_get_model(treeview));
+       // Additional settings, add row
+       preset = preset_get_name(dict);
+       if (pos >= 0)
+       {
+               GValue *parent_dict;
+               parent_dict = presets_get_dict(presets, folder_pos, -1);
+               parent_name = preset_get_name(parent_dict);
+       }
+       else
+               parent_name = NULL;
+       def = FALSE;
+       if (strcmp(preset, def_preset) == 0)
+       {
+               if (parent_name && strcmp(parent_name, def_folder) == 0)
+                       def = TRUE;
+               else if (parent_name == NULL && def_folder[0] == 0)
+                       def = TRUE;
+       }
+
+       description = ghb_presets_get_description(dict);
+       flags = ghb_preset_flags(dict);
+       custom = flags & PRESETS_CUST;
+       gtk_tree_store_set(store, iter, 0, preset, 
+                                               1, def ? 800 : 400, 
+                                               2, def ? 2 : 0,
+                                               3, custom ? "black" : "blue", 
+                                               4, description,
+                                               -1);
+       if (flags & PRESETS_FOLDER)
+       {
+               presets = ghb_dict_lookup(dict, "preset_folder");
+               ghb_presets_list_init(ud, presets, preset, iter);
+       }
+       g_free(def_preset);
+}
+
+static void
+presets_list_insert(
+       signal_user_data_t *ud, 
+       GValue *presets,
+       const gchar *parent_name,
+       GtkTreeIter *parent,
+       gint pos)
+{
+       GtkTreeView *treeview;
+       GtkTreeIter iter;
+       GtkTreeStore *store;
+       const gchar *preset;
+       gchar *def_preset, *def_folder;
+       const gchar *description;
+       gint flags, custom;
+       gboolean def;
+       gint count;
+       GValue *dict;
+       
+       g_debug("presets_list_insert ()");
+       count = ghb_array_len(presets);
+       if (pos >= count)
+               return;
+       def_folder = ghb_settings_get_string(ud->settings, "default_folder");
+       def_preset = ghb_settings_get_string(ud->settings, "default_preset");
+       treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "presets_list"));
+       store = GTK_TREE_STORE(gtk_tree_view_get_model(treeview));
+       // Additional settings, add row
+       dict = ghb_array_get_nth(presets, pos);
+       preset = preset_get_name(dict);
+       def = FALSE;
+       if (strcmp(preset, def_preset) == 0)
+       {
+               if (parent_name && strcmp(parent_name, def_folder) == 0)
+                       def = TRUE;
+               else if (parent_name == NULL && def_folder[0] == 0)
+                       def = TRUE;
+       }
+
+       description = ghb_presets_get_description(dict);
+       gtk_tree_store_insert(store, &iter, parent, pos);
+       flags = ghb_preset_flags(dict);
+       custom = flags & PRESETS_CUST;
+       gtk_tree_store_set(store, &iter, 0, preset, 
+                                               1, def ? 800 : 400, 
+                                               2, def ? 2 : 0,
+                                               3, custom ? "black" : "blue", 
+                                               4, description,
+                                               -1);
+       if (flags & PRESETS_FOLDER)
+       {
+               presets = ghb_dict_lookup(dict, "preset_folder");
+               ghb_presets_list_init(ud, presets, preset, &iter);
+       }
+       g_free(def_preset);
+}
+
+static void
+presets_list_remove(
+       signal_user_data_t *ud, 
+       gint folder_pos,
+       gint pos)
+{
+       GtkTreeView *treeview;
+       GtkTreeIter iter, piter;
+       GtkTreeStore *store;
+       
+       g_debug("presets_list_remove ()");
+       treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "presets_list"));
+       store = GTK_TREE_STORE(gtk_tree_view_get_model(treeview));
+       if (folder_pos >= 0)
+       {
+               if (gtk_tree_model_iter_nth_child(GTK_TREE_MODEL(store), &piter, 
+                       NULL, folder_pos))
+               {
+                       if (pos >= 0)
+                       {
+                               if (gtk_tree_model_iter_nth_child(GTK_TREE_MODEL(store), &iter, 
+                                       &piter, pos))
+                               {
+                                       gtk_tree_store_remove(store, &iter);
+                               }
+                       }
+                       else
+                       {
+                               gtk_tree_store_remove(store, &piter);
+                       }
+               }
+       }
+       else if (pos >= 0)
+       {
+               if (gtk_tree_model_iter_nth_child(GTK_TREE_MODEL(store), &iter, 
+                       NULL, pos))
+               {
+                       gtk_tree_store_remove(store, &iter);
+               }
+       }
+}
+
+void
 ghb_presets_reload(signal_user_data_t *ud)
 {
        GValue *std_presets;
@@ -726,12 +1202,17 @@ ghb_presets_reload(signal_user_data_t *ud)
                GHashTableIter iter;
                gchar *key;
                GValue *value;
-               gint pos;
+               gint folder_pos, pos;
 
                std_dict = ghb_array_get_nth(std_presets, ii);
                name = preset_get_name(std_dict);
-               presets_remove(presetsPlist, name);
-
+               if (ghb_presets_find(presetsPlist, NULL, name, &folder_pos, &pos))
+               {
+                       if (ghb_presets_remove(presetsPlist, folder_pos, pos))
+                       {
+                               presets_list_remove(ud, folder_pos, pos);
+                       }
+               }
                copy_dict = ghb_dict_value_new();
                pos = presets_find_pos(presetsPlist, name, 0);
                ghb_array_insert(presetsPlist, pos, copy_dict);
@@ -743,17 +1224,11 @@ ghb_presets_reload(signal_user_data_t *ud)
                {
                        ghb_dict_insert(copy_dict, g_strdup(key), ghb_value_dup(value));
                }
+               presets_list_insert(ud, presetsPlist, NULL, NULL, pos);
        }
        store_plist(presetsPlist, "presets");
 }
 
-static void
-presets_store()
-{
-       g_debug("presets_store ()\n");
-       store_plist(presetsPlist, "presets");
-}
-
 void
 ghb_save_queue(GValue *queue)
 {
@@ -779,7 +1254,7 @@ ghb_presets_load()
        if (presetsPlist == NULL)
        {
                presetsPlist = ghb_value_dup(ghb_resource_get("standard-presets"));
-               presets_store();
+               store_plist(presetsPlist, "presets");
        }
        if (G_VALUE_TYPE(presetsPlist) == ghb_dict_get_type())
        { // Presets is older dictionary format. Convert to array
@@ -814,18 +1289,19 @@ ghb_presets_load()
                }
                ghb_value_free(presetsPlist);
                presetsPlist = presets;
-               presets_store();
+               store_plist(presetsPlist, "presets");
        }
 }
 
-void
-ghb_settings_save(signal_user_data_t *ud, const gchar *name)
+static void
+settings_save(signal_user_data_t *ud, const gchar *folder, const gchar *name)
 {
        GValue *dict, *internal;
        GHashTableIter iter;
        gchar *key;
        GValue *value;
        gboolean autoscale;
+       gint folder_pos, pos;
 
        if (internalPlist == NULL) return;
        if (ghb_settings_get_boolean(ud->settings, "allow_tweaks"))
@@ -845,14 +1321,34 @@ ghb_settings_save(signal_user_data_t *ud, const gchar *name)
                }
        }
        autoscale = ghb_settings_get_boolean(ud->settings, "autoscale");
-       ghb_settings_set_int64(ud->settings, "preset_type", 1);
+       ghb_settings_set_int64(ud->settings, "preset_type", PRESETS_CUST);
 
        dict = ghb_dict_value_new();
+       if (ghb_presets_find(presetsPlist, folder, name, &folder_pos, &pos))
+       {
+               if (ghb_presets_get_type(presetsPlist, folder_pos, pos) & 
+                       PRESETS_FOLDER)
+               {
+                       gchar *message;
+                       message = g_strdup_printf(
+                                               "%s: Folder already exists.\n"
+                                               "You can not replace it with a preset.",
+                                               name);
+                       ghb_message_dialog(GTK_MESSAGE_ERROR, message, "Cancel", NULL);
+                       g_free(message);
+                       return;
+               }
+               ghb_presets_replace(presetsPlist, dict, folder_pos, pos);
+               pos = -1;
+       }
+       else
+       {
+               pos = presets_find_pos(presetsPlist, name, 1);
+               ghb_array_insert(presetsPlist, pos, dict);
+       }
        ghb_dict_insert(dict, g_strdup("preset_name"), ghb_string_value_new(name));
-       gint pos = presets_find_pos(presetsPlist, name, 1);
-       ghb_array_insert(presetsPlist, pos, dict);
-       internal = plist_get_dict(internalPlist, "Presets");
 
+       internal = plist_get_dict(internalPlist, "Presets");
        ghb_dict_iter_init(&iter, internal);
        // middle (void*) cast prevents gcc warning "defreferencing type-punned
        // pointer will break strict-aliasing rules"
@@ -886,139 +1382,232 @@ ghb_settings_save(signal_user_data_t *ud, const gchar *name)
                        ghb_dict_insert(dict, g_strdup(key), ghb_value_dup(gval));
                }
        }
-       presets_store();
+       if (pos >= 0)
+               presets_list_insert(ud, presetsPlist, NULL, NULL, pos);
+       store_plist(presetsPlist, "presets");
        ud->dont_clear_presets = TRUE;
-       ghb_set_preset (ud, name);
+       ghb_set_preset (ud, NULL, name);
        ud->dont_clear_presets = FALSE;
+       return;
 }
 
-void
-ghb_presets_remove(const gchar *name)
+static void
+folder_save(signal_user_data_t *ud, const gchar *name)
 {
-       if (presets_get_dict(presetsPlist, name))
+       GValue *dict, *folder;
+       gint folder_pos, pos;
+
+
+       if (ghb_presets_find(presetsPlist, name, NULL, &folder_pos, &pos))
+       {
+               if (!(ghb_presets_get_type(presetsPlist, folder_pos, pos) & 
+                       PRESETS_FOLDER))
+               {
+                       gchar *message;
+                       message = g_strdup_printf(
+                                               "%s: Preset already exists.\n"
+                                               "You can not replace it with a folder.",
+                                               name);
+                       ghb_message_dialog(GTK_MESSAGE_ERROR, message, "Cancel", NULL);
+                       g_free(message);
+                       return;
+               }
+               // Already exists, update its description
+               dict = presets_get_dict(presetsPlist, folder_pos, pos);
+               ghb_dict_insert(dict, g_strdup("preset_description"), 
+                       ghb_value_dup(preset_dict_get_value(
+                               ud->settings, "preset_description")));
+               return;
+       }
+       else
        {
-               presets_remove(presetsPlist, name);
-               presets_store();
+               dict = ghb_dict_value_new();
+               pos = presets_find_pos(presetsPlist, name, 1);
+               ghb_array_insert(presetsPlist, pos, dict);
        }
+       ghb_dict_insert(dict, g_strdup("preset_description"), 
+               ghb_value_dup(preset_dict_get_value(
+                       ud->settings, "preset_description")));
+       ghb_dict_insert(dict, g_strdup("preset_name"), ghb_string_value_new(name));
+       folder = ghb_array_value_new(8);
+       ghb_dict_insert(dict, g_strdup("preset_folder"), folder);
+       ghb_dict_insert(dict, g_strdup("preset_type"),
+                                                       ghb_int64_value_new(PRESETS_FOLDER|PRESETS_CUST));
+
+       presets_list_insert(ud, presetsPlist, NULL, NULL, pos);
+       store_plist(presetsPlist, "presets");
+       ud->dont_clear_presets = TRUE;
+       ghb_set_preset (ud, NULL, name);
+       ud->dont_clear_presets = FALSE;
+       return;
 }
 
 void
-ghb_presets_list_update(signal_user_data_t *ud)
+ghb_presets_list_default(signal_user_data_t *ud)
 {
        GtkTreeView *treeview;
-       GtkTreeIter iter;
+       GtkTreeIter iter, citer;
        GtkTreeStore *store;
        gboolean done;
-       const gchar *preset;
-       gchar *def_preset;
-       const gchar *description;
-       gint flags, custom, def;
-       gint count, ii;
-       GValue *dict;
+       gchar *preset;
+       gchar *def_preset, *def_folder;
+       gint def, weight;
        
-       g_debug("ghb_presets_list_update ()");
+       g_debug("ghb_presets_list_default ()");
+       def_folder = ghb_settings_get_string(ud->settings, "default_folder");
        def_preset = ghb_settings_get_string(ud->settings, "default_preset");
-       count = ghb_array_len(presetsPlist);
        treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "presets_list"));
        store = GTK_TREE_STORE(gtk_tree_view_get_model(treeview));
-       ii = 0;
        if (gtk_tree_model_get_iter_first(GTK_TREE_MODEL(store), &iter))
        {
+               if (def_folder[0] != 0) 
+               {
+                       gboolean found = FALSE;
+                       do
+                       {
+                               gtk_tree_model_get(GTK_TREE_MODEL(store), &iter, 
+                                                                       0, &preset, 1, &weight, -1);
+                               if (strcmp(preset, def_folder) == 0)
+                               {
+                                       if (gtk_tree_model_iter_children(
+                                                                       GTK_TREE_MODEL(store), &citer, &iter))
+                                       {
+                                               iter = citer;
+                                               found = TRUE;
+                                       }
+                                       g_free(preset);
+                                       break;
+                               }
+                               g_free(preset);
+                               done = !gtk_tree_model_iter_next(GTK_TREE_MODEL(store), &iter);
+                       } while (!done);
+                       if (!found) return;
+               }
                do
                {
-                       if (ii < count)
+                       def = FALSE;
+                       gtk_tree_model_get(GTK_TREE_MODEL(store), &iter, 
+                                                               0, &preset, 1, &weight, -1);
+                       if (strcmp(preset, def_preset) == 0)
+                               def = TRUE;
+                       if ((!def && weight == 800) || def)
                        {
-                               // Update row with settings data
-                               g_debug("Updating row");
-                               dict = ghb_array_get_nth(presetsPlist, ii);
-                               preset = preset_get_name(dict);
-                               def = 0;
-                               if (strcmp(preset, def_preset) == 0)
-                                       def = PRESET_DEFAULT;
-                               
-                               description = ghb_presets_get_description(dict);
-                               flags = ghb_preset_flags(dict);
-                               custom = flags & PRESET_CUSTOM;
                                gtk_tree_store_set(store, &iter, 
-                                                       0, preset, 
                                                        1, def ? 800 : 400, 
                                                        2, def ? 2 : 0,
-                                                       3, custom ? "black" : "blue", 
-                                                       4, description,
                                                        -1);
-                               ii++;
-                               done = !gtk_tree_model_iter_next(GTK_TREE_MODEL(store), &iter);
+                       }
+                       g_free(preset);
+                       done = !gtk_tree_model_iter_next(GTK_TREE_MODEL(store), &iter);
+               } while (!done);
+       }
+       g_free(def_folder);
+       g_free(def_preset);
+}
+
+void
+ghb_presets_list_clear_default(signal_user_data_t *ud)
+{
+       GtkTreeView *treeview;
+       GtkTreeIter iter, piter;
+       GtkTreeStore *store;
+       gchar *def_preset, *def_folder;
+       gint folder_pos, pos;
+       gboolean found = FALSE;
+       
+       g_debug("ghb_presets_list_default ()");
+       def_folder = ghb_settings_get_string(ud->settings, "default_folder");
+       def_preset = ghb_settings_get_string(ud->settings, "default_preset");
+       treeview = GTK_TREE_VIEW(GHB_WIDGET(ud->builder, "presets_list"));
+       store = GTK_TREE_STORE(gtk_tree_view_get_model(treeview));
+       if (!ghb_presets_find(presetsPlist, def_folder, def_preset, 
+               &folder_pos, &pos))
+       {
+               return;
+       }
+       // de-emphasize the current default
+       if (folder_pos >= 0)
+       {
+               if (gtk_tree_model_iter_nth_child(GTK_TREE_MODEL(store), &piter, 
+                       NULL, folder_pos))
+               {
+                       if (pos >= 0)
+                       {
+                               if (gtk_tree_model_iter_nth_child(GTK_TREE_MODEL(store), &iter, 
+                                       &piter, pos))
+                               {
+                                       found = TRUE;
+                               }
                        }
                        else
                        {
-                               // No more settings data, remove row
-                               g_debug("Removing row");
-                               done = !gtk_tree_store_remove(store, &iter);
+                               found = TRUE;
                        }
-               } while (!done);
+               }
        }
-       while (ii < count)
+       else if (pos >= 0)
        {
-               // Additional settings, add row
-               g_debug("Adding rows");
-               dict = ghb_array_get_nth(presetsPlist, ii);
-               preset = preset_get_name(dict);
-               def = 0;
-               if (strcmp(preset, def_preset) == 0)
-                       def = PRESET_DEFAULT;
-
-               description = ghb_presets_get_description(dict);
-               gtk_tree_store_append(store, &iter, NULL);
-               flags = ghb_preset_flags(dict);
-               custom = flags & PRESET_CUSTOM;
-               gtk_tree_store_set(store, &iter, 0, preset, 
-                                                       1, def ? 800 : 400, 
-                                                       2, def ? 2 : 0,
-                                                       3, custom ? "black" : "blue", 
-                                                       4, description,
-                                                       -1);
-               ii++;
+               if (gtk_tree_model_iter_nth_child(GTK_TREE_MODEL(store), &iter, 
+                       NULL, pos))
+               {
+                       found = TRUE;
+               }
+       }
+       if (found)
+       {
+               gtk_tree_store_set(store, &iter, 
+                                       1, 400, 
+                                       2, 0,
+                                       -1);
        }
+       g_free(def_folder);
        g_free(def_preset);
 }
 
-void
-ghb_select_preset(GtkBuilder *builder, const gchar *preset)
+static void
+ghb_select_preset2(
+       GtkBuilder *builder, 
+       gint folder_pos, 
+       gint pos)
 {
        GtkTreeView *treeview;
        GtkTreeSelection *selection;
        GtkTreeModel *store;
        GtkTreeIter iter;
-       gchar *tpreset;
-       gboolean done;
-       gboolean foundit = FALSE;
+       GtkTreePath *path;
        
-       g_debug("select_preset()");
-       if (preset == NULL) return;
+       g_debug("ghb_select_preset2() (%d) (%d)", folder_pos, pos);
        treeview = GTK_TREE_VIEW(GHB_WIDGET(builder, "presets_list"));
        selection = gtk_tree_view_get_selection (treeview);
        store = gtk_tree_view_get_model (treeview);
-       if (gtk_tree_model_get_iter_first(store, &iter))
+       if (folder_pos == -1)
        {
-               do
-               {
-                       gtk_tree_model_get(store, &iter, 0, &tpreset, -1);
-                       if (strcmp(preset, tpreset) == 0)
-                       {
-                               gtk_tree_selection_select_iter (selection, &iter);
-                               foundit = TRUE;
-                               g_free(tpreset);
-                               break;
-                       }
-                       g_free(tpreset);
-                       done = !gtk_tree_model_iter_next(store, &iter);
-               } while (!done);
+               folder_pos = pos;
+               pos = -1;
        }
-       if (!foundit)
+       path = gtk_tree_path_new_from_indices(folder_pos, pos, -1);
+       if (gtk_tree_model_get_iter(store, &iter, path))
+       {
+               gtk_tree_selection_select_iter (selection, &iter);
+       }
+       else
        {
                gtk_tree_model_get_iter_first(store, &iter);
                gtk_tree_selection_select_iter (selection, &iter);
        }
+       gtk_tree_path_free(path);
+}
+
+void
+ghb_select_preset(GtkBuilder *builder, const gchar *folder, const gchar *preset)
+{
+       gint folder_pos, pos;
+
+       g_debug("ghb_select_preset() (%s) (%s)", folder, preset);
+       if (ghb_presets_find(presetsPlist, folder, preset, &folder_pos, &pos))
+       {
+               ghb_select_preset2(builder, folder_pos, pos);
+       }
 }
 
 static void
@@ -1032,6 +1621,34 @@ update_audio_presets(signal_user_data_t *ud)
 }
 
 void
+enforce_preset_type(const gchar *name, signal_user_data_t *ud)
+{
+       gint folder_pos, pos;
+       GtkWidget *normal, *folder;
+       gint ptype;
+
+       normal = GHB_WIDGET(ud->builder, "preset_type_normal");
+       folder = GHB_WIDGET(ud->builder, "preset_type_folder");
+       if (ghb_presets_find(presetsPlist, NULL, name, &folder_pos, &pos))
+       {
+               ptype = ghb_presets_get_type(presetsPlist, folder_pos, pos);
+               if (ptype & PRESETS_FOLDER)
+                       gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(folder), 
+                                                                       TRUE);
+               else
+                       gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(normal), 
+                                                                       TRUE);
+               gtk_widget_set_sensitive(folder,  ptype & PRESETS_FOLDER);
+               gtk_widget_set_sensitive(normal,  !(ptype & PRESETS_FOLDER));
+       }
+       else
+       {
+               gtk_widget_set_sensitive(folder, TRUE);
+               gtk_widget_set_sensitive(normal, TRUE);
+       }
+}
+
+void
 presets_save_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
 {
        GtkWidget *dialog;
@@ -1047,6 +1664,7 @@ presets_save_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
        dialog = GHB_WIDGET(ud->builder, "preset_save_dialog");
        entry = GTK_ENTRY(GHB_WIDGET(ud->builder, "preset_name"));
        gtk_entry_set_text(entry, preset);
+       enforce_preset_type(preset, ud);
        g_free(preset);
        response = gtk_dialog_run(GTK_DIALOG(dialog));
        gtk_widget_hide(dialog);
@@ -1056,26 +1674,47 @@ presets_save_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
                const gchar *name = gtk_entry_get_text(entry);
                g_debug("description to settings");
                ghb_widget_to_setting(ud->settings, GTK_WIDGET(desc));
-               // Construct the audio settings presets from the current audio list
-               update_audio_presets(ud);
-               ghb_settings_save(ud, name);
-               ghb_presets_list_update(ud);
+               if (ghb_settings_get_boolean(ud->settings, "preset_type_folder"))
+               {
+                       folder_save(ud, name);
+               }
+               else
+               {
+                       // Construct the audio settings presets from the current audio list
+                       update_audio_presets(ud);
+                       settings_save(ud, NULL, name);
+               }
                // Make the new preset the selected item
-               ghb_select_preset(ud->builder, name);
+               ghb_select_preset(ud->builder, NULL, name);
        }
 }
 
 void
+preset_type_changed_cb(GtkWidget *widget, signal_user_data_t *ud)
+{
+       ghb_widget_to_setting(ud->settings, widget);
+}
+
+void
+preset_name_changed_cb(GtkWidget *entry, signal_user_data_t *ud)
+{
+       gchar *name;
+
+       name = ghb_widget_string(entry);
+       enforce_preset_type(name, ud);
+}
+
+void
 presets_restore_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
 {
        g_debug("presets_restore_clicked_cb ()");
        // Reload only the standard presets
        ghb_presets_reload(ud);
-       ghb_presets_list_update(ud);
        // Updating the presets list shuffles things around
        // need to make sure the proper preset is selected
+       gchar *folder = ghb_settings_get_string (ud->settings, "folder");
        gchar *preset = ghb_settings_get_string (ud->settings, "preset");
-       ghb_select_preset(ud->builder, preset);
+       ghb_select_preset(ud->builder, folder, preset);
        g_free(preset);
 }
 
@@ -1095,35 +1734,298 @@ presets_remove_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
        if (gtk_tree_selection_get_selected(selection, &store, &iter))
        {
                GtkWidget *dialog;
+               GtkTreePath *path;
+               gint *indices;
+               gint folder_pos, pos, ptype;
 
                gtk_tree_model_get(store, &iter, 0, &preset, -1);
+               path = gtk_tree_model_get_path(store, &iter);
+               indices = gtk_tree_path_get_indices(path);
+               folder_pos = indices[0];
+               pos = -1;
+               if (gtk_tree_path_get_depth(path) > 1)
+                       pos = indices[1];
+               gtk_tree_path_free(path);
+               ptype = ghb_presets_get_type(presetsPlist, folder_pos, pos);
                dialog = gtk_message_dialog_new(NULL, GTK_DIALOG_MODAL,
-                                                               GTK_MESSAGE_QUESTION, GTK_BUTTONS_YES_NO,
-                                                               "Confirm deletion of preset %s.", preset);
+                                                       GTK_MESSAGE_QUESTION, GTK_BUTTONS_YES_NO,
+                                                       "Confirm deletion of %s:\n\n%s", 
+                                                       (ptype & PRESETS_FOLDER) ? "folder" : "preset",
+                                                       preset);
                response = gtk_dialog_run(GTK_DIALOG(dialog));
                gtk_widget_destroy (dialog);
                if (response == GTK_RESPONSE_YES)
                {
                        GtkTreeIter nextIter = iter;
-                       gchar *nextPreset = NULL;
+                       gboolean valid = TRUE;
                        if (!gtk_tree_model_iter_next(store, &nextIter))
                        {
-                               if (gtk_tree_model_get_iter_first(store, &nextIter))
+                               if (!gtk_tree_model_iter_parent(store, &nextIter, &iter))
                                {
-                                       gtk_tree_model_get(store, &nextIter, 0, &nextPreset, -1);
+                                       valid = FALSE;
                                }
                        }
-                       else
-                       {
-                               gtk_tree_model_get(store, &nextIter, 0, &nextPreset, -1);
-                       }
                        // Remove the selected item
                        // First unselect it so that selecting the new item works properly
                        gtk_tree_selection_unselect_iter (selection, &iter);
-                       ghb_presets_remove(preset);
-                       ghb_presets_list_update(ud);
-                       ghb_select_preset(ud->builder, nextPreset);
+                       if (ghb_presets_remove(presetsPlist, folder_pos, pos))
+                       {
+                               store_plist(presetsPlist, "presets");
+                               presets_list_remove(ud, folder_pos, pos);
+                       }
+                       if (!valid)
+                               valid = gtk_tree_model_get_iter_first(store, &nextIter);
+                       if (valid)
+                       {
+                               path = gtk_tree_model_get_path(store, &nextIter);
+                               indices = gtk_tree_path_get_indices(path);
+                               folder_pos = indices[0];
+                               pos = -1;
+                               if (gtk_tree_path_get_depth(path) > 1)
+                                       pos = indices[1];
+                               gtk_tree_path_free(path);
+                               ghb_select_preset2(ud->builder, folder_pos, pos);
+                       }
+               }
+               g_free(preset);
+       }
+}
+
+// controls where valid drop locations are
+gboolean
+presets_drag_motion_cb(
+       GtkTreeView *tv,
+       GdkDragContext *ctx,
+       gint x,
+       gint y,
+       guint time,
+       signal_user_data_t *ud)
+{
+       GtkTreePath *path = NULL;
+       GtkTreeViewDropPosition drop_pos;
+       gint *indices, folder_pos, pos;
+       GtkTreeIter iter;
+       GtkTreeView *srctv;
+       GtkTreeModel *model;
+       GtkTreeSelection *select;
+       gint src_ptype, dst_ptype;
+
+       // Get the type of the object being dragged
+       srctv = GTK_TREE_VIEW(gtk_drag_get_source_widget(ctx));
+       select = gtk_tree_view_get_selection (srctv);
+       gtk_tree_selection_get_selected (select, &model, &iter);
+       path = gtk_tree_model_get_path (model, &iter);
+       indices = gtk_tree_path_get_indices(path);
+       folder_pos = indices[0];
+       pos = -1;
+       if (gtk_tree_path_get_depth(path) > 1)
+               pos = indices[1];
+       gtk_tree_path_free(path);
+       src_ptype = ghb_presets_get_type(presetsPlist, folder_pos, pos);
+
+       // The rest checks that the destination is a valid position
+       // in the list.
+       gtk_tree_view_get_dest_row_at_pos (tv, x, y, &path, &drop_pos);
+       if (path == NULL)
+       {
+               gdk_drag_status(ctx, GDK_ACTION_MOVE, time);
+               return TRUE;
+       }
+       indices = gtk_tree_path_get_indices(path);
+       folder_pos = indices[0];
+       pos = -1;
+       if (gtk_tree_path_get_depth(path) > 1)
+               pos = indices[1];
+       dst_ptype = ghb_presets_get_type(presetsPlist, folder_pos, pos);
+
+       // Don't allow *drop into* if the source is a folder
+       if (((src_ptype & PRESETS_FOLDER) || (!(dst_ptype & PRESETS_FOLDER))) && 
+               drop_pos == GTK_TREE_VIEW_DROP_INTO_OR_BEFORE)
+               drop_pos = GTK_TREE_VIEW_DROP_BEFORE;
+       if (((src_ptype & PRESETS_FOLDER) || (!(dst_ptype & PRESETS_FOLDER))) && 
+               drop_pos == GTK_TREE_VIEW_DROP_INTO_OR_AFTER)
+               drop_pos = GTK_TREE_VIEW_DROP_AFTER;
+       // Don't allow droping folders into child items
+       if ((src_ptype & PRESETS_FOLDER) && gtk_tree_path_get_depth(path) > 1)
+       {
+               gtk_tree_path_up(path);
+               drop_pos = GTK_TREE_VIEW_DROP_AFTER;
+       }
+       gtk_tree_view_set_drag_dest_row(tv, path, drop_pos);
+       gtk_tree_path_free(path);
+       gdk_drag_status(ctx, GDK_ACTION_MOVE, time);
+       return TRUE;
+}
+
+void 
+presets_drag_cb(
+       GtkTreeView *dstwidget, 
+       GdkDragContext *dc, 
+       gint x, gint y, 
+       GtkSelectionData *selection_data, 
+       guint info, guint t, 
+       signal_user_data_t *ud)
+{
+       GtkTreePath *path = NULL;
+       GtkTreeViewDropPosition drop_pos;
+       GtkTreeIter dstiter, srciter;
+       gint *indices;
+       gint dst_folder_pos, dst_pos, src_folder_pos, src_pos;
+       gint src_ptype, dst_ptype;
+       
+       GtkTreeModel *dstmodel = gtk_tree_view_get_model(dstwidget);
+                       
+       g_debug("preset_drag_cb ()");
+       // This doesn't work here for some reason...
+       // gtk_tree_view_get_drag_dest_row(dstwidget, &path, &drop_pos);
+       gtk_tree_view_get_dest_row_at_pos (dstwidget, x, y, &path, &drop_pos);
+       // This little hack is needed because attempting to drop after
+       // the last item gives us no path or drop_pos.
+       if (path == NULL)
+       {
+               gint n_children;
+
+               n_children = gtk_tree_model_iter_n_children(dstmodel, NULL);
+               if (n_children)
+               {
+                       drop_pos = GTK_TREE_VIEW_DROP_AFTER;
+                       path = gtk_tree_path_new_from_indices(n_children-1, -1);
+               }
+               else
+               {
+                       drop_pos = GTK_TREE_VIEW_DROP_BEFORE;
+                       path = gtk_tree_path_new_from_indices(0, -1);
+               }
+       }
+       if (path)
+       {
+               GtkTreeView *srcwidget;
+               GtkTreeModel *srcmodel;
+               GtkTreeSelection *select;
+               GtkTreePath *srcpath = NULL;
+               GValue *dict, *presets;
+               GValue *preset;
+
+               srcwidget = GTK_TREE_VIEW(gtk_drag_get_source_widget(dc));
+               select = gtk_tree_view_get_selection (srcwidget);
+               gtk_tree_selection_get_selected (select, &srcmodel, &srciter);
+
+               srcpath = gtk_tree_model_get_path (srcmodel, &srciter);
+               indices = gtk_tree_path_get_indices(srcpath);
+               src_folder_pos = indices[0];
+               src_pos = -1;
+               if (gtk_tree_path_get_depth(srcpath) > 1)
+                       src_pos = indices[1];
+               src_ptype = ghb_presets_get_type(presetsPlist, src_folder_pos, src_pos);
+               preset = presets_get_dict(presetsPlist, src_folder_pos, src_pos);
+               gtk_tree_path_free(srcpath);
+
+               indices = gtk_tree_path_get_indices(path);
+               dst_folder_pos = indices[0];
+               dst_pos = -1;
+               if (gtk_tree_path_get_depth(path) > 1)
+                       dst_pos = indices[1];
+               dst_ptype = ghb_presets_get_type(presetsPlist, dst_folder_pos, dst_pos);
+
+               if ((src_ptype & PRESETS_FOLDER) && gtk_tree_path_get_depth(path) > 1)
+                       gtk_tree_path_up(path);
+
+               if (gtk_tree_model_get_iter (dstmodel, &dstiter, path))
+               {
+                       GtkTreeIter iter;
+                       GtkTreePath *dstpath = NULL;
+
+                       if ((src_ptype & PRESETS_FOLDER) || 
+                               gtk_tree_path_get_depth(path) > 1)
+                       {
+                               switch (drop_pos)
+                               {
+                                       case GTK_TREE_VIEW_DROP_BEFORE:
+                                       case GTK_TREE_VIEW_DROP_INTO_OR_BEFORE:
+                                               gtk_tree_store_insert_before(GTK_TREE_STORE (dstmodel), 
+                                                                                                       &iter, NULL, &dstiter);
+                                               break;
+
+                                       case GTK_TREE_VIEW_DROP_AFTER:
+                                       case GTK_TREE_VIEW_DROP_INTO_OR_AFTER:
+                                               gtk_tree_store_insert_after(GTK_TREE_STORE (dstmodel), 
+                                                                                                       &iter, NULL, &dstiter);
+                                               break;
+
+                                       default:
+                                               break;
+                               }
+                       }
+                       else
+                       {
+                               switch (drop_pos)
+                               {
+                                       case GTK_TREE_VIEW_DROP_BEFORE:
+                                               gtk_tree_store_insert_before(GTK_TREE_STORE (dstmodel), 
+                                                                                                       &iter, NULL, &dstiter);
+                                               break;
+
+                                       case GTK_TREE_VIEW_DROP_INTO_OR_BEFORE:
+                                               gtk_tree_store_insert(GTK_TREE_STORE (dstmodel), 
+                                                                                                       &iter, &dstiter, 0);
+                                               break;
+
+                                       case GTK_TREE_VIEW_DROP_AFTER:
+                                               gtk_tree_store_insert_after(GTK_TREE_STORE (dstmodel), 
+                                                                                                       &iter, NULL, &dstiter);
+                                               break;
+
+                                       case GTK_TREE_VIEW_DROP_INTO_OR_AFTER:
+                                               gtk_tree_store_insert_after(GTK_TREE_STORE (dstmodel), 
+                                                                                                       &iter, &dstiter, 0);
+                                               break;
+
+                                       default:
+                                               break;
+                               }
+                       }
+                       presets_list_update_item(ud, presetsPlist, &iter, 
+                               src_folder_pos, src_pos);
+
+                       dstpath = gtk_tree_model_get_path (dstmodel, &iter);
+                       indices = gtk_tree_path_get_indices(dstpath);
+                       dst_folder_pos = indices[0];
+                       dst_pos = -1;
+                       if (gtk_tree_path_get_depth(dstpath) > 1)
+                               dst_pos = indices[1];
+                       gtk_tree_path_free(dstpath);
+                       if (dst_pos != -1)
+                       {
+                               dict = presets_get_dict(presetsPlist, dst_folder_pos, -1);
+                               presets = ghb_dict_lookup(dict, "preset_folder");
+                               ghb_array_insert(presets, dst_pos, preset);
+                       }
+                       else
+                       {
+                               ghb_array_insert(presetsPlist, dst_folder_pos, preset);
+                       }
+
+                       srcpath = gtk_tree_model_get_path (srcmodel, &srciter);
+                       indices = gtk_tree_path_get_indices(srcpath);
+                       src_folder_pos = indices[0];
+                       src_pos = -1;
+                       if (gtk_tree_path_get_depth(srcpath) > 1)
+                               src_pos = indices[1];
+                       gtk_tree_path_free(srcpath);
+                       if (src_pos != -1)
+                       {
+                               dict = presets_get_dict(presetsPlist, src_folder_pos, -1);
+                               presets = ghb_dict_lookup(dict, "preset_folder");
+                               ghb_array_remove(presets, src_pos);
+                       }
+                       else
+                       {
+                               ghb_array_remove(presetsPlist, src_folder_pos);
+                       }
+                       gtk_tree_store_remove (GTK_TREE_STORE (srcmodel), &srciter);
+                       store_plist(presetsPlist, "presets");
                }
+               gtk_tree_path_free(path);
        }
 }
 
@@ -1170,8 +2072,9 @@ void
 presets_list_selection_changed_cb(GtkTreeSelection *selection, signal_user_data_t *ud)
 {
        GtkTreeModel *store;
-       GtkTreeIter iter;
+       GtkTreeIter iter, piter;
        gchar *preset;
+       gchar *folder = NULL;
        ghb_title_info_t tinfo;
        GtkWidget *widget;
        
@@ -1180,6 +2083,10 @@ presets_list_selection_changed_cb(GtkTreeSelection *selection, signal_user_data_
        if (gtk_tree_selection_get_selected(selection, &store, &iter))
        {
                gtk_tree_model_get(store, &iter, 0, &preset, -1);
+               if (gtk_tree_model_iter_parent(store, &piter, &iter))
+               {
+                       gtk_tree_model_get(store, &piter, 0, &folder, -1);
+               }
                ud->dont_clear_presets = TRUE;
                // Temporarily set the video_quality range to (0,100)
                // This is needed so the video_quality value does not get
@@ -1190,7 +2097,7 @@ presets_list_selection_changed_cb(GtkTreeSelection *selection, signal_user_data_
                // can cause the container extension to be automatically changed when
                // it shouldn't be
                ghb_clear_audio_list(ud);
-               ghb_set_preset(ud, preset);
+               ghb_set_preset(ud, folder, preset);
                gint titleindex;
                titleindex = ghb_settings_combo_int(ud->settings, "title");
                ghb_set_pref_audio(titleindex, ud);
@@ -1206,6 +2113,8 @@ presets_list_selection_changed_cb(GtkTreeSelection *selection, signal_user_data_
                ghb_vquality_range(ud, &vqmin, &vqmax);
                gtk_range_set_range (GTK_RANGE(qp), vqmin, vqmax);
                gtk_widget_set_sensitive(widget, TRUE);
+               g_free(preset);
+               if (folder) g_free(folder);
        }
        else
        {
@@ -1251,7 +2160,22 @@ presets_frame_size_allocate_cb(GtkWidget *widget, GtkAllocation *allocation, sig
 void
 presets_default_clicked_cb(GtkWidget *xwidget, signal_user_data_t *ud)
 {
-       ghb_set_preset_default(ud->settings);
-       ghb_presets_list_update(ud);
+       gchar *folder, *preset;
+       gint folder_pos, pos;
+
+       folder = ghb_settings_get_string(ud->settings, "folder");
+       preset = ghb_settings_get_string(ud->settings, "preset");
+       if (ghb_presets_find(presetsPlist, folder, preset, &folder_pos, &pos))
+       {
+               if (!(ghb_presets_get_type(presetsPlist, folder_pos, pos) & 
+                       PRESETS_FOLDER))
+               {
+                       ghb_presets_list_clear_default(ud);
+                       ghb_set_preset_default(ud->settings);
+                       ghb_presets_list_default(ud);
+               }
+       }
+       g_free(folder);
+       g_free(preset);
 }
 
index 37bc005..ff11e3f 100644 (file)
 
 void ghb_settings_save(signal_user_data_t *ud, const gchar *name);
 void ghb_presets_load(void);
-void ghb_presets_reload(signal_user_data_t *ud);
-void ghb_set_preset(signal_user_data_t *ud, const gchar *name);
-void ghb_update_from_preset( 
-       signal_user_data_t *ud, const gchar *name, const gchar *key);
-void ghb_presets_remove(const gchar *name);
+void ghb_update_from_preset(signal_user_data_t *ud, 
+               const gchar *folder, const gchar *name, const gchar *key);
 void ghb_prefs_load(signal_user_data_t *ud);
 void ghb_settings_init(signal_user_data_t *ud);
 void ghb_settings_close();
@@ -36,7 +33,9 @@ void ghb_remove_queue_file(void);;
 gchar* ghb_get_user_config_dir();
 void ghb_settings_to_ui(signal_user_data_t *ud, GValue *dict);
 void ghb_clear_presets_selection(signal_user_data_t *ud);
-void ghb_select_preset(GtkBuilder *builder, const gchar *preset);
-void ghb_presets_list_update(signal_user_data_t *ud);
+void ghb_select_preset(GtkBuilder *builder, 
+               const gchar *folder, const gchar *preset);
+void ghb_presets_list_init( signal_user_data_t *ud, 
+       GValue *presets, const gchar *parent_name, GtkTreeIter *parent);
 
 #endif // _GHB_PRESETS_H_
index 3c03dab..7a54317 100644 (file)
 "/property&gt;\n"
 "                    &lt;property name=&quot;truncate_multiline&quot;&gt"
 ";True&lt;/property&gt;\n"
+"                    &lt;signal handler=&quot;preset_name_changed_cb&quo"
+"t; name=&quot;changed&quot;/&gt;\n"
 "                  &lt;/object&gt;\n"
 "                  &lt;packing&gt;\n"
 "                    &lt;property name=&quot;position&quot;&gt;1&lt;/pro"
 "                      &lt;object class=&quot;GtkTextView&quot; id=&quot"
 ";preset_description&quot;&gt;\n"
 "                        &lt;property name=&quot;height_request&quot;&gt"
-";50&lt;/property&gt;\n"
+";60&lt;/property&gt;\n"
 "                        &lt;property name=&quot;visible&quot;&gt;True&l"
 "t;/property&gt;\n"
 "                        &lt;property name=&quot;can_focus&quot;&gt;True"
 "y&gt;\n"
 "              &lt;/packing&gt;\n"
 "            &lt;/child&gt;\n"
+"            &lt;child&gt;\n"
+"              &lt;object class=&quot;GtkHBox&quot; id=&quot;hbox43&quot"
+";&gt;\n"
+"                &lt;property name=&quot;visible&quot;&gt;True&lt;/prope"
+"rty&gt;\n"
+"                &lt;child&gt;\n"
+"                  &lt;object class=&quot;GtkRadioButton&quot; id=&quot;"
+"preset_type_folder&quot;&gt;\n"
+"                    &lt;property name=&quot;visible&quot;&gt;True&lt;/p"
+"roperty&gt;\n"
+"                    &lt;property name=&quot;can_focus&quot;&gt;True&lt;"
+"/property&gt;\n"
+"                    &lt;property name=&quot;label&quot; translatable=&q"
+"uot;yes&quot;&gt;Folder&lt;/property&gt;\n"
+"                    &lt;property name=&quot;active&quot;&gt;True&lt;/pr"
+"operty&gt;\n"
+"                    &lt;property name=&quot;draw_indicator&quot;&gt;Tru"
+"e&lt;/property&gt;\n"
+"                    &lt;signal name=&quot;toggled&quot; handler=&quot;p"
+"reset_type_changed_cb&quot;/&gt;\n"
+"                  &lt;/object&gt;\n"
+"                  &lt;packing&gt;\n"
+"                    &lt;property name=&quot;expand&quot;&gt;False&lt;/p"
+"roperty&gt;\n"
+"                  &lt;/packing&gt;\n"
+"                &lt;/child&gt;\n"
+"                &lt;child&gt;\n"
+"                  &lt;object class=&quot;GtkRadioButton&quot; id=&quot;"
+"preset_type_normal&quot;&gt;\n"
+"                    &lt;property name=&quot;visible&quot;&gt;True&lt;/p"
+"roperty&gt;\n"
+"                    &lt;property name=&quot;can_focus&quot;&gt;True&lt;"
+"/property&gt;\n"
+"                    &lt;property name=&quot;label&quot; translatable=&q"
+"uot;yes&quot;&gt;Preset&lt;/property&gt;\n"
+"                    &lt;property name=&quot;active&quot;&gt;True&lt;/pr"
+"operty&gt;\n"
+"                    &lt;property name=&quot;draw_indicator&quot;&gt;Tru"
+"e&lt;/property&gt;\n"
+"                    &lt;property name=&quot;group&quot;&gt;preset_type_"
+"folder&lt;/property&gt;\n"
+"                    &lt;signal name=&quot;toggled&quot; handler=&quot;p"
+"reset_type_changed_cb&quot;/&gt;\n"
+"                  &lt;/object&gt;\n"
+"                  &lt;packing&gt;\n"
+"                    &lt;property name=&quot;expand&quot;&gt;False&lt;/p"
+"roperty&gt;\n"
+"                    &lt;property name=&quot;position&quot;&gt;1&lt;/pro"
+"perty&gt;\n"
+"                  &lt;/packing&gt;\n"
+"                &lt;/child&gt;\n"
+"              &lt;/object&gt;\n"
+"              &lt;packing&gt;\n"
+"                &lt;property name=&quot;expand&quot;&gt;False&lt;/prope"
+"rty&gt;\n"
+"                &lt;property name=&quot;position&quot;&gt;2&lt;/propert"
+"y&gt;\n"
+"              &lt;/packing&gt;\n"
+"            &lt;/child&gt;\n"
 "          &lt;/object&gt;\n"
 "          &lt;packing&gt;\n"
 "            &lt;property name=&quot;position&quot;&gt;1&lt;/property&gt"
 "                      <integer>0</integer>\n"
 "                      <key>end_chapter</key>\n"
 "                      <integer>100</integer>\n"
+"                      <key>folder</key>\n"
+"                      <string></string>\n"
 "                      <key>preset</key>\n"
 "                      <string>Normal</string>\n"
+"                      <key>preset_type_folder</key>\n"
+"                      <false />\n"
+"                      <key>preset_type_normal</key>\n"
+"                      <true />\n"
 "                      <key>scale_height</key>\n"
 "                      <integer>480</integer>\n"
 "                      <key>scale_width</key>\n"
 "                      <false />\n"
 "                      <key>chapters_in_destination</key>\n"
 "                      <false />\n"
+"                      <key>default_folder</key>\n"
+"                      <string></string>\n"
 "                      <key>default_preset</key>\n"
 "                      <string>Normal</string>\n"
 "                      <key>default_source</key>\n"
index 8a15b20..a940771 100644 (file)
@@ -3708,6 +3708,7 @@ this setting.&lt;/property&gt;
                     &lt;property name=&quot;activates_default&quot;&gt;True&lt;/property&gt;
                     &lt;property name=&quot;width_chars&quot;&gt;30&lt;/property&gt;
                     &lt;property name=&quot;truncate_multiline&quot;&gt;True&lt;/property&gt;
+                    &lt;signal handler=&quot;preset_name_changed_cb&quot; name=&quot;changed&quot;/&gt;
                   &lt;/object&gt;
                   &lt;packing&gt;
                     &lt;property name=&quot;position&quot;&gt;1&lt;/property&gt;
@@ -3734,7 +3735,7 @@ this setting.&lt;/property&gt;
                     &lt;property name=&quot;right_padding&quot;&gt;4&lt;/property&gt;
                     &lt;child&gt;
                       &lt;object class=&quot;GtkTextView&quot; id=&quot;preset_description&quot;&gt;
-                        &lt;property name=&quot;height_request&quot;&gt;50&lt;/property&gt;
+                        &lt;property name=&quot;height_request&quot;&gt;60&lt;/property&gt;
                         &lt;property name=&quot;visible&quot;&gt;True&lt;/property&gt;
                         &lt;property name=&quot;can_focus&quot;&gt;True&lt;/property&gt;
                         &lt;property name=&quot;events&quot;&gt;GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK&lt;/property&gt;
@@ -3758,6 +3759,43 @@ this setting.&lt;/property&gt;
                 &lt;property name=&quot;position&quot;&gt;1&lt;/property&gt;
               &lt;/packing&gt;
             &lt;/child&gt;
+            &lt;child&gt;
+              &lt;object class=&quot;GtkHBox&quot; id=&quot;hbox43&quot;&gt;
+                &lt;property name=&quot;visible&quot;&gt;True&lt;/property&gt;
+                &lt;child&gt;
+                  &lt;object class=&quot;GtkRadioButton&quot; id=&quot;preset_type_folder&quot;&gt;
+                    &lt;property name=&quot;visible&quot;&gt;True&lt;/property&gt;
+                    &lt;property name=&quot;can_focus&quot;&gt;True&lt;/property&gt;
+                    &lt;property name=&quot;label&quot; translatable=&quot;yes&quot;&gt;Folder&lt;/property&gt;
+                    &lt;property name=&quot;active&quot;&gt;True&lt;/property&gt;
+                    &lt;property name=&quot;draw_indicator&quot;&gt;True&lt;/property&gt;
+                    &lt;signal name=&quot;toggled&quot; handler=&quot;preset_type_changed_cb&quot;/&gt;
+                  &lt;/object&gt;
+                  &lt;packing&gt;
+                    &lt;property name=&quot;expand&quot;&gt;False&lt;/property&gt;
+                  &lt;/packing&gt;
+                &lt;/child&gt;
+                &lt;child&gt;
+                  &lt;object class=&quot;GtkRadioButton&quot; id=&quot;preset_type_normal&quot;&gt;
+                    &lt;property name=&quot;visible&quot;&gt;True&lt;/property&gt;
+                    &lt;property name=&quot;can_focus&quot;&gt;True&lt;/property&gt;
+                    &lt;property name=&quot;label&quot; translatable=&quot;yes&quot;&gt;Preset&lt;/property&gt;
+                    &lt;property name=&quot;active&quot;&gt;True&lt;/property&gt;
+                    &lt;property name=&quot;draw_indicator&quot;&gt;True&lt;/property&gt;
+                    &lt;property name=&quot;group&quot;&gt;preset_type_folder&lt;/property&gt;
+                    &lt;signal name=&quot;toggled&quot; handler=&quot;preset_type_changed_cb&quot;/&gt;
+                  &lt;/object&gt;
+                  &lt;packing&gt;
+                    &lt;property name=&quot;expand&quot;&gt;False&lt;/property&gt;
+                    &lt;property name=&quot;position&quot;&gt;1&lt;/property&gt;
+                  &lt;/packing&gt;
+                &lt;/child&gt;
+              &lt;/object&gt;
+              &lt;packing&gt;
+                &lt;property name=&quot;expand&quot;&gt;False&lt;/property&gt;
+                &lt;property name=&quot;position&quot;&gt;2&lt;/property&gt;
+              &lt;/packing&gt;
+            &lt;/child&gt;
           &lt;/object&gt;
           &lt;packing&gt;
             &lt;property name=&quot;position&quot;&gt;1&lt;/property&gt;
@@ -4858,8 +4896,14 @@ R2RrUAAABBgBAQACAAAAQAAAABAAAAAQ////AP///wD///8A////AP///wD///8A////AP///wD///8A
                        <integer>0</integer>
                        <key>end_chapter</key>
                        <integer>100</integer>
+                       <key>folder</key>
+                       <string></string>
                        <key>preset</key>
                        <string>Normal</string>
+                       <key>preset_type_folder</key>
+                       <false />
+                       <key>preset_type_normal</key>
+                       <true />
                        <key>scale_height</key>
                        <integer>480</integer>
                        <key>scale_width</key>
@@ -4915,6 +4959,8 @@ R2RrUAAABBgBAQACAAAAQAAAABAAAAAQ////AP///wD///8A////AP///wD///8A////AP///wD///8A
                        <false />
                        <key>chapters_in_destination</key>
                        <false />
+                       <key>default_folder</key>
+                       <string></string>
                        <key>default_preset</key>
                        <string>Normal</string>
                        <key>default_source</key>
index 69a79c2..e328ba6 100644 (file)
@@ -18,9 +18,6 @@
 
 #include <gtk/gtk.h>
 
-#define PRESET_CUSTOM  1
-#define PRESET_DEFAULT 2
-
 #define GHB_WIDGET(b,n)        GTK_WIDGET(gtk_builder_get_object ((b), (n)))
 //#define GHB_WIDGET(b,n)      GTK_WIDGET(debug_get_object((b), (n)))
 #define GHB_ACTION(b,n)        GTK_ACTION(gtk_builder_get_object ((b), (n)))
index 6013db8..4f6c407 100644 (file)
@@ -640,6 +640,18 @@ ghb_array_remove(GValue *gval, guint ii)
        g_value_take_boxed(gval, arr);
 }
 
+void
+ghb_array_replace(GValue *gval, guint ii, GValue *val)
+{
+       GArray *arr = g_value_get_boxed(gval);
+       // A little nastyness here.  The array pointer
+       // can change when the array changes size.  So
+       // I must re-box it in the GValue each time.
+       if (ii >= arr->len) return;
+       ghb_value_free(((GValue**)arr->data)[ii]);
+       ((GValue**)arr->data)[ii] = val;
+}
+
 gint
 ghb_array_len(const GValue *gval)
 {
index 5e02900..feb995d 100644 (file)
@@ -31,6 +31,7 @@ GType ghb_array_get_type(void);
 GType ghb_dict_get_type(void);
 GValue* ghb_array_get_nth(const GValue *array, gint ii);
 void ghb_array_insert(GValue *gval, guint ii, GValue *val);
+void ghb_array_replace(GValue *gval, guint ii, GValue *val);
 void ghb_array_append(GValue *gval, GValue *val);
 void ghb_array_remove(GValue *gval, guint ii);
 gint ghb_array_len(const GValue *gval);