Remove redundant view <-> model value mapping stuff from AudioRegionGainLine.
[ardour.git] / gtk2_ardour / new_session_dialog.cc
index 462fb57711c4d037882a898909a98b33b47594bb..43f98a5271132dc0ce374e8d3ddeabca93bb21ad 100644 (file)
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include "i18n.h"
@@ -23,6 +22,7 @@
 
 #include <ardour/recent_sessions.h>
 #include <ardour/session.h>
+#include <ardour/template_utils.h>
 
 #include <gtkmm/entry.h>
 #include <gtkmm/filechooserbutton.h>
 #include <gtkmm/stock.h>
 #include <gdkmm/cursor.h>
 
+#include <gtkmm2ext/window_title.h>
+
+using namespace Gtkmm2ext;
+using namespace ARDOUR;
+using namespace PBD;
+
 #include "opts.h"
 
 NewSessionDialog::NewSessionDialog()
-       : ArdourDialog ("New Session Dialog")
+       : ArdourDialog ("session control")
 {
-        session_name_label = Gtk::manage(new class Gtk::Label(_("New Session Name :")));
+       session_name_label = Gtk::manage(new class Gtk::Label(_("Name :")));
        m_name = Gtk::manage(new class Gtk::Entry());
        m_name->set_text(GTK_ARDOUR::session_name);
 
-       session_location_label = Gtk::manage(new class Gtk::Label(_("Create Session Directory In :")));
+       chan_count_label_1 = Gtk::manage(new class Gtk::Label(_("channels")));
+       chan_count_label_2 = Gtk::manage(new class Gtk::Label(_("channels")));
+       chan_count_label_3 = Gtk::manage(new class Gtk::Label(_("channels")));
+       chan_count_label_4 = Gtk::manage(new class Gtk::Label(_("channels")));
+
+       chan_count_label_1->set_alignment(0,0.5);
+       chan_count_label_1->set_padding(0,0);
+       chan_count_label_1->set_line_wrap(false);
+
+       chan_count_label_2->set_alignment(0,0.5);
+       chan_count_label_2->set_padding(0,0);
+       chan_count_label_2->set_line_wrap(false);
+
+       chan_count_label_3->set_alignment(0,0.5);
+       chan_count_label_3->set_padding(0,0);
+       chan_count_label_3->set_line_wrap(false);
+
+       chan_count_label_4->set_alignment(0,0.5);
+       chan_count_label_4->set_padding(0,0);
+       chan_count_label_4->set_line_wrap(false);
+
+       bus_label = Gtk::manage(new class Gtk::Label(_("<b>Busses</b>")));
+       input_label = Gtk::manage(new class Gtk::Label(_("<b>Inputs</b>")));
+       output_label = Gtk::manage(new class Gtk::Label(_("<b>Outputs</b>")));
+
+       session_location_label = Gtk::manage(new class Gtk::Label(_("Create Folder In :")));
        m_folder = Gtk::manage(new class Gtk::FileChooserButton(Gtk::FILE_CHOOSER_ACTION_SELECT_FOLDER));
-       session_template_label = Gtk::manage(new class Gtk::Label(_("Use Session Template :")));
+       session_template_label = Gtk::manage(new class Gtk::Label(_("Template :")));
        m_template = Gtk::manage(new class Gtk::FileChooserButton());
-       chan_count_label = Gtk::manage(new class Gtk::Label(_("Channel Count")));
        m_create_control_bus = Gtk::manage(new class Gtk::CheckButton(_("Create Monitor Bus")));
        
        Gtk::Adjustment *m_control_bus_channel_count_adj = Gtk::manage(new class Gtk::Adjustment(2, 0, 100, 1, 10, 10));
@@ -56,8 +86,8 @@ NewSessionDialog::NewSessionDialog()
        m_master_bus_channel_count = Gtk::manage(new class Gtk::SpinButton(*m_master_bus_channel_count_adj, 1, 0));
        m_create_master_bus = Gtk::manage(new class Gtk::CheckButton(_("Create Master Bus")));
        advanced_table = Gtk::manage(new class Gtk::Table(2, 2, true));
-       m_connect_inputs = Gtk::manage(new class Gtk::CheckButton(_("Automatically Connect Inputs")));
-       m_limit_input_ports = Gtk::manage(new class Gtk::CheckButton(_("Port Limit")));
+       m_connect_inputs = Gtk::manage(new class Gtk::CheckButton(_("Automatically Connect to Physical Inputs")));
+       m_limit_input_ports = Gtk::manage(new class Gtk::CheckButton(_("Use only")));
        
        Gtk::Adjustment *m_input_limit_count_adj = Gtk::manage(new class Gtk::Adjustment(1, 0, 100, 1, 10, 10));
        m_input_limit_count = Gtk::manage(new class Gtk::SpinButton(*m_input_limit_count_adj, 1, 0));
@@ -65,10 +95,12 @@ NewSessionDialog::NewSessionDialog()
        input_port_vbox = Gtk::manage(new class Gtk::VBox(false, 0));
        input_table = Gtk::manage(new class Gtk::Table(2, 2, false));
 
-       input_label = Gtk::manage(new class Gtk::Label(_("<b>Track/Bus Inputs</b>")));
+       bus_frame = Gtk::manage(new class Gtk::Frame());
+       bus_table = Gtk::manage (new Gtk::Table (2, 3, false));
+
        input_frame = Gtk::manage(new class Gtk::Frame());
        m_connect_outputs = Gtk::manage(new class Gtk::CheckButton(_("Automatically Connect Outputs")));
-       m_limit_output_ports = Gtk::manage(new class Gtk::CheckButton(_("Port Limit")));
+       m_limit_output_ports = Gtk::manage(new class Gtk::CheckButton(_("Use only")));
        
        Gtk::Adjustment *m_output_limit_count_adj = Gtk::manage(new class Gtk::Adjustment(1, 0, 100, 1, 10, 10));
        m_output_limit_count = Gtk::manage(new class Gtk::SpinButton(*m_output_limit_count_adj, 1, 0));
@@ -76,12 +108,11 @@ NewSessionDialog::NewSessionDialog()
        output_port_vbox = Gtk::manage(new class Gtk::VBox(false, 0));
        
        Gtk::RadioButton::Group _RadioBGroup_m_connect_outputs_to_master;
-       m_connect_outputs_to_master = Gtk::manage(new class Gtk::RadioButton(_RadioBGroup_m_connect_outputs_to_master, _("Connect to Master Bus")));
-       m_connect_outputs_to_physical = Gtk::manage(new class Gtk::RadioButton(_RadioBGroup_m_connect_outputs_to_master, _("Connect to Physical Outputs")));
+       m_connect_outputs_to_master = Gtk::manage(new class Gtk::RadioButton(_RadioBGroup_m_connect_outputs_to_master, _("... to Master Bus")));
+       m_connect_outputs_to_physical = Gtk::manage(new class Gtk::RadioButton(_RadioBGroup_m_connect_outputs_to_master, _("... to Physical Outputs")));
        output_conn_vbox = Gtk::manage(new class Gtk::VBox(false, 0));
        output_vbox = Gtk::manage(new class Gtk::VBox(false, 0));
 
-       output_label = Gtk::manage(new class Gtk::Label(_("<b>Track/Bus Outputs</b>")));
        output_frame = Gtk::manage(new class Gtk::Frame());
        advanced_vbox = Gtk::manage(new class Gtk::VBox(false, 0));
        advanced_label = Gtk::manage(new class Gtk::Label(_("Advanced Options")));
@@ -92,24 +123,25 @@ NewSessionDialog::NewSessionDialog()
        m_treeview = Gtk::manage(new class Gtk::TreeView());
        recent_scrolledwindow = Gtk::manage(new class Gtk::ScrolledWindow());
 
-       recent_sesion_label = Gtk::manage(new class Gtk::Label(_("Open Recent Session")));
+       recent_sesion_label = Gtk::manage(new class Gtk::Label(_("Recent:")));
        recent_frame = Gtk::manage(new class Gtk::Frame());
        open_session_vbox = Gtk::manage(new class Gtk::VBox(false, 0));
        m_notebook = Gtk::manage(new class Gtk::Notebook());
        session_name_label->set_alignment(0, 0.5);
-       session_name_label->set_padding(0,0);
+       session_name_label->set_padding(6,0);
        session_name_label->set_line_wrap(false);
        session_name_label->set_selectable(false);
        m_name->set_editable(true);
        m_name->set_max_length(0);
        m_name->set_has_frame(true);
        m_name->set_activates_default(true);
+       m_name->set_width_chars (40);
        session_location_label->set_alignment(0,0.5);
-       session_location_label->set_padding(0,0);
+       session_location_label->set_padding(6,0);
        session_location_label->set_line_wrap(false);
        session_location_label->set_selectable(false);
        session_template_label->set_alignment(0,0.5);
-       session_template_label->set_padding(0,0);
+       session_template_label->set_padding(6,0);
        session_template_label->set_line_wrap(false);
        session_template_label->set_selectable(false);
        m_create_control_bus->set_flags(Gtk::CAN_FOCUS);
@@ -128,7 +160,7 @@ NewSessionDialog::NewSessionDialog()
        m_master_bus_channel_count->set_numeric(true);
        m_master_bus_channel_count->set_digits(0);
        m_master_bus_channel_count->set_wrap(false);
-       open_session_file_label = Gtk::manage(new class Gtk::Label(_("Open Session File :")));
+       open_session_file_label = Gtk::manage(new class Gtk::Label(_("Browse:")));
        open_session_file_label->set_alignment(0, 0.5);
        m_create_master_bus->set_flags(Gtk::CAN_FOCUS);
        m_create_master_bus->set_relief(Gtk::RELIEF_NORMAL);
@@ -137,11 +169,6 @@ NewSessionDialog::NewSessionDialog()
        m_create_master_bus->set_border_width(0);
        advanced_table->set_row_spacings(0);
        advanced_table->set_col_spacings(0);
-       advanced_table->attach(*chan_count_label, 1, 2, 0, 1, Gtk::AttachOptions(), Gtk::AttachOptions(), 0, 0);
-       advanced_table->attach(*m_create_control_bus, 0, 1, 2, 3, Gtk::FILL, Gtk::AttachOptions(), 0, 0);
-       advanced_table->attach(*m_control_bus_channel_count, 1, 2, 2, 3, Gtk::AttachOptions(), Gtk::AttachOptions(), 0, 0);
-       advanced_table->attach(*m_master_bus_channel_count, 1, 2, 1, 2, Gtk::AttachOptions(), Gtk::AttachOptions(), 0, 0);
-       advanced_table->attach(*m_create_master_bus, 0, 1, 1, 2, Gtk::FILL, Gtk::AttachOptions(), 0, 0);
 
        m_connect_inputs->set_flags(Gtk::CAN_FOCUS);
        m_connect_inputs->set_relief(Gtk::RELIEF_NORMAL);
@@ -161,14 +188,40 @@ NewSessionDialog::NewSessionDialog()
        m_input_limit_count->set_wrap(false);
        m_input_limit_count->set_sensitive(false);
 
+       bus_hbox = Gtk::manage (new Gtk::HBox (false, 0));
+       bus_hbox->pack_start (*bus_table, Gtk::PACK_SHRINK, 18);
+
+       bus_label->set_alignment(0, 0.5);
+       bus_label->set_padding(0,0);
+       bus_label->set_line_wrap(false);
+       bus_label->set_selectable(false);
+       bus_label->set_use_markup(true);
+       bus_frame->set_shadow_type(Gtk::SHADOW_NONE);
+       bus_frame->set_label_align(0,0.5);
+       bus_frame->add(*bus_hbox);
+       bus_frame->set_label_widget(*bus_label);
+       
+       bus_table->set_row_spacings (0);
+       bus_table->set_col_spacings (0);
+       bus_table->attach (*m_create_master_bus, 0, 1, 0, 1, Gtk::EXPAND|Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 0, 0);
+       bus_table->attach (*m_master_bus_channel_count, 1, 2, 0, 1, Gtk::EXPAND|Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 0, 0);
+       bus_table->attach (*chan_count_label_1, 2, 3, 0, 1, Gtk::EXPAND|Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 6, 0);
+       bus_table->attach (*m_create_control_bus, 0, 1, 1, 2, Gtk::EXPAND|Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 0, 0);
+       bus_table->attach (*m_control_bus_channel_count, 1, 2, 1, 2, Gtk::EXPAND|Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 0, 0);
+       bus_table->attach (*chan_count_label_2, 2, 3, 1, 2, Gtk::EXPAND|Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 6, 0);
+
        input_port_limit_hbox->pack_start(*m_limit_input_ports, Gtk::PACK_SHRINK, 6);
-       input_port_limit_hbox->pack_start(*m_input_limit_count, Gtk::PACK_EXPAND_PADDING, 0);
+       input_port_limit_hbox->pack_start(*m_input_limit_count, Gtk::PACK_SHRINK, 0);
+       input_port_limit_hbox->pack_start(*chan_count_label_3, Gtk::PACK_SHRINK, 6);
        input_port_vbox->pack_start(*m_connect_inputs, Gtk::PACK_SHRINK, 0);
        input_port_vbox->pack_start(*input_port_limit_hbox, Gtk::PACK_EXPAND_PADDING, 0);
        input_table->set_row_spacings(0);
        input_table->set_col_spacings(0);
        input_table->attach(*input_port_vbox, 0, 1, 0, 1, Gtk::EXPAND|Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 6, 6);
 
+       input_hbox = Gtk::manage (new Gtk::HBox (false, 0));
+       input_hbox->pack_start (*input_table, Gtk::PACK_SHRINK, 18);
+
        input_label->set_alignment(0, 0.5);
        input_label->set_padding(0,0);
        input_label->set_line_wrap(false);
@@ -176,8 +229,9 @@ NewSessionDialog::NewSessionDialog()
        input_label->set_use_markup(true);
        input_frame->set_shadow_type(Gtk::SHADOW_NONE);
        input_frame->set_label_align(0,0.5);
-       input_frame->add(*input_table);
+       input_frame->add(*input_hbox);
        input_frame->set_label_widget(*input_label);
+
        m_connect_outputs->set_flags(Gtk::CAN_FOCUS);
        m_connect_outputs->set_relief(Gtk::RELIEF_NORMAL);
        m_connect_outputs->set_mode(true);
@@ -195,9 +249,8 @@ NewSessionDialog::NewSessionDialog()
        m_output_limit_count->set_wrap(false);
        m_output_limit_count->set_sensitive(false);
        output_port_limit_hbox->pack_start(*m_limit_output_ports, Gtk::PACK_SHRINK, 6);
-       output_port_limit_hbox->pack_start(*m_output_limit_count, Gtk::PACK_EXPAND_PADDING, 0);
-       output_port_vbox->pack_start(*m_connect_outputs, Gtk::PACK_SHRINK, 0);
-       output_port_vbox->pack_start(*output_port_limit_hbox, Gtk::PACK_EXPAND_PADDING, 0);
+       output_port_limit_hbox->pack_start(*m_output_limit_count, Gtk::PACK_SHRINK, 0);
+       output_port_limit_hbox->pack_start(*chan_count_label_4, Gtk::PACK_SHRINK, 6);
        m_connect_outputs_to_master->set_flags(Gtk::CAN_FOCUS);
        m_connect_outputs_to_master->set_relief(Gtk::RELIEF_NORMAL);
        m_connect_outputs_to_master->set_mode(true);
@@ -208,11 +261,15 @@ NewSessionDialog::NewSessionDialog()
        m_connect_outputs_to_physical->set_mode(true);
        m_connect_outputs_to_physical->set_active(false);
        m_connect_outputs_to_physical->set_border_width(0);
+       output_conn_vbox->pack_start(*m_connect_outputs, Gtk::PACK_SHRINK, 0);
        output_conn_vbox->pack_start(*m_connect_outputs_to_master, Gtk::PACK_SHRINK, 0);
        output_conn_vbox->pack_start(*m_connect_outputs_to_physical, Gtk::PACK_SHRINK, 0);
        output_vbox->set_border_width(6);
-       output_vbox->pack_start(*output_port_vbox);
+
+       output_port_vbox->pack_start(*output_port_limit_hbox, Gtk::PACK_SHRINK, 0);
+
        output_vbox->pack_start(*output_conn_vbox);
+       output_vbox->pack_start(*output_port_vbox);
 
        output_label->set_alignment(0, 0.5);
        output_label->set_padding(0,0);
@@ -221,10 +278,16 @@ NewSessionDialog::NewSessionDialog()
        output_label->set_use_markup(true);
        output_frame->set_shadow_type(Gtk::SHADOW_NONE);
        output_frame->set_label_align(0,0.5);
-       output_frame->add(*output_vbox);
+
+       output_hbox = Gtk::manage (new Gtk::HBox (false, 0));
+       output_hbox->pack_start (*output_vbox, Gtk::PACK_SHRINK, 18);
+
+       output_frame->add(*output_hbox);
        output_frame->set_label_widget(*output_label);
+
        advanced_vbox->pack_start(*advanced_table, Gtk::PACK_SHRINK, 0);
-       advanced_vbox->pack_start(*input_frame, Gtk::PACK_SHRINK, 12);
+       advanced_vbox->pack_start(*bus_frame, Gtk::PACK_SHRINK, 6);
+       advanced_vbox->pack_start(*input_frame, Gtk::PACK_SHRINK, 6);
        advanced_vbox->pack_start(*output_frame, Gtk::PACK_SHRINK, 0);
        advanced_label->set_padding(0,0);
        advanced_label->set_line_wrap(false);
@@ -237,7 +300,7 @@ NewSessionDialog::NewSessionDialog()
        advanced_expander->add(*advanced_vbox);
        advanced_expander->set_label_widget(*advanced_label);
        new_session_table->set_border_width(12);
-       new_session_table->set_row_spacings(0);
+       new_session_table->set_row_spacings(6);
        new_session_table->set_col_spacings(0);
        new_session_table->attach(*session_name_label, 0, 1, 0, 1, Gtk::FILL, Gtk::FILL, 0, 0);
        new_session_table->attach(*m_name, 1, 2, 0, 1, Gtk::EXPAND|Gtk::FILL, Gtk::FILL, 0, 0);
@@ -245,11 +308,9 @@ NewSessionDialog::NewSessionDialog()
        new_session_table->attach(*m_folder, 1, 2, 1, 2, Gtk::EXPAND|Gtk::FILL, Gtk::FILL, 0, 0);
        new_session_table->attach(*session_template_label, 0, 1, 2, 3, Gtk::FILL, Gtk::FILL, 0, 0);
        new_session_table->attach(*m_template, 1, 2, 2, 3, Gtk::EXPAND|Gtk::FILL, Gtk::FILL, 0, 0);
-       new_session_table->attach(*advanced_expander, 0, 2, 3, 4, Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 0, 12);
-       chan_count_label->set_padding(0,0);
-       chan_count_label->set_line_wrap(false);
-       chan_count_label->set_selectable(false);
-       open_session_hbox->pack_start(*open_session_file_label, true, true, 12);
+       new_session_table->attach(*advanced_expander, 0, 2, 3, 4, Gtk::FILL, Gtk::EXPAND|Gtk::FILL, 0, 6);
+
+       open_session_hbox->pack_start(*open_session_file_label, false, false, 12);
        open_session_hbox->pack_start(*m_open_filechooser, true, true, 12);
        m_treeview->set_flags(Gtk::CAN_FOCUS);
        m_treeview->set_headers_visible(true);
@@ -258,7 +319,6 @@ NewSessionDialog::NewSessionDialog()
        m_treeview->set_enable_search(true);
        m_treeview->set_fixed_height_mode(false);
        m_treeview->set_hover_selection(false);
-       m_treeview->set_hover_expand(true);
        m_treeview->set_size_request(-1, 150);
        recent_scrolledwindow->set_flags(Gtk::CAN_FOCUS);
        recent_scrolledwindow->set_border_width(6);
@@ -271,11 +331,11 @@ NewSessionDialog::NewSessionDialog()
        recent_sesion_label->set_line_wrap(false);
        recent_sesion_label->set_selectable(false);
        recent_frame->set_border_width(12);
-       recent_frame->set_shadow_type(Gtk::SHADOW_IN);
+       recent_frame->set_shadow_type(Gtk::SHADOW_NONE);
        recent_frame->add(*recent_scrolledwindow);
        recent_frame->set_label_widget(*recent_sesion_label);
-       open_session_vbox->pack_start(*open_session_hbox, Gtk::PACK_SHRINK, 12);
        open_session_vbox->pack_start(*recent_frame, Gtk::PACK_EXPAND_WIDGET, 0);
+       open_session_vbox->pack_start(*open_session_hbox, Gtk::PACK_SHRINK, 12);
        m_notebook->set_flags(Gtk::CAN_FOCUS);
        m_notebook->set_scrollable(true);
        m_notebook->append_page(*new_session_table, _("New Session"));
@@ -285,7 +345,11 @@ NewSessionDialog::NewSessionDialog()
        get_vbox()->set_homogeneous(false);
        get_vbox()->set_spacing(0);
        get_vbox()->pack_start(*m_notebook, Gtk::PACK_SHRINK, 0);
-       set_title(_("ardour: session control"));
+
+       WindowTitle title(Glib::get_application_name());
+       title += _("Session Control");
+       set_title(title.get_string());
+
        //set_modal(false);
        //property_window_position().set_value(Gtk::WIN_POS_NONE);
        set_resizable(false);
@@ -302,18 +366,20 @@ NewSessionDialog::NewSessionDialog()
        m_treeview->set_headers_visible (false);
        m_treeview->get_selection()->set_mode (Gtk::SELECTION_SINGLE);
 
-       std::string path = ARDOUR::get_user_ardour_path();
-       if (path.empty()) {
-               path = ARDOUR::get_system_data_path();
+       if (is_directory (user_template_directory ()))
+       {
+               m_template->set_current_folder (user_template_directory().to_string());
        }
-       if (!path.empty()) {
-               m_template->set_current_folder (path + X_("templates/"));
+       else if (is_directory (system_template_directory ()))
+       {
+               m_template->set_current_folder (system_template_directory().to_string());
+       }
+
+       if (is_directory (system_template_directory ()))
+       {
+               m_template->add_shortcut_folder (system_template_directory().to_string());
        }
 
-       const std::string sys_templates_dir = ARDOUR::get_system_data_path() + X_("templates");
-       if (Glib::file_test(sys_templates_dir, Glib::FILE_TEST_IS_DIR))
-               m_template->add_shortcut_folder(sys_templates_dir);
-       
        m_template->set_title(_("select template"));
        Gtk::FileFilter* session_filter = manage (new (Gtk::FileFilter));
        session_filter->add_pattern(X_("*.ardour"));
@@ -331,7 +397,6 @@ NewSessionDialog::NewSessionDialog()
        m_folder->set_current_folder(getenv ("HOME"));
        m_folder->set_title(_("select directory"));
 
-       on_new_session_page = true;
        m_notebook->set_current_page(0);
        m_notebook->show();
        m_notebook->show_all_children();
@@ -354,7 +419,7 @@ NewSessionDialog::NewSessionDialog()
        m_limit_output_ports->signal_clicked().connect (mem_fun (*this, &NewSessionDialog::limit_outputs_clicked));
        m_create_master_bus->signal_clicked().connect (mem_fun (*this, &NewSessionDialog::master_bus_button_clicked));
        m_create_control_bus->signal_clicked().connect (mem_fun (*this, &NewSessionDialog::monitor_bus_button_clicked));
-       m_name->signal_key_release_event().connect(mem_fun (*this, &NewSessionDialog::entry_key_release));
+       m_name->signal_changed().connect(mem_fun (*this, &NewSessionDialog::on_new_session_name_entry_changed));
        m_notebook->signal_switch_page().connect (mem_fun (*this, &NewSessionDialog::notebook_page_changed));
        m_treeview->get_selection()->signal_changed().connect (mem_fun (*this, &NewSessionDialog::treeview_selection_changed));
        m_treeview->signal_row_activated().connect (mem_fun (*this, &NewSessionDialog::recent_row_activated));
@@ -372,7 +437,7 @@ NewSessionDialog::set_session_name(const Glib::ustring& name)
 std::string
 NewSessionDialog::session_name() const
 {
-        std::string str = Glib::filename_from_utf8(m_open_filechooser->get_filename());
+       std::string str = Glib::filename_from_utf8(m_open_filechooser->get_filename());
        std::string::size_type position = str.find_last_of ('/');
        str = str.substr (position+1);
        position = str.find_last_of ('.');
@@ -387,7 +452,7 @@ NewSessionDialog::session_name() const
        }         
        */
 
-       if (m_notebook->get_current_page() == 0) {
+       if (on_new_session_page ()) {
                return Glib::filename_from_utf8(m_name->get_text());
        } else {
                if (m_treeview->get_selection()->count_selected_rows() == 0) {
@@ -401,12 +466,12 @@ NewSessionDialog::session_name() const
 std::string
 NewSessionDialog::session_folder() const
 {
-        if (m_notebook->get_current_page() == 0) {
-               return Glib::filename_from_utf8(m_folder->get_current_folder());
+       if (on_new_session_page ()) {
+               return Glib::filename_from_utf8(m_folder->get_current_folder());
        } else {
               
                if (m_treeview->get_selection()->count_selected_rows() == 0) {
-                       return Glib::filename_from_utf8(m_open_filechooser->get_current_folder());
+                       return Glib::filename_from_utf8(m_open_filechooser->get_current_folder());
                }
                Gtk::TreeModel::iterator i = m_treeview->get_selection()->get_selected();
                return (*i)[recent_columns.fullpath];
@@ -416,7 +481,7 @@ NewSessionDialog::session_folder() const
 bool
 NewSessionDialog::use_session_template() const
 {
-        if(m_template->get_filename().empty() && (m_notebook->get_current_page() == 0)) return false;
+       if(m_template->get_filename().empty() && (m_notebook->get_current_page() == 0)) return false;
        return true;
 }
 
@@ -498,11 +563,16 @@ NewSessionDialog::connect_outs_to_physical() const
        return m_connect_outputs_to_physical->get_active();
 }
 
-int
-NewSessionDialog::get_current_page()
+bool
+NewSessionDialog::on_new_session_page() const
 {
-       return m_notebook->get_current_page();
-       
+       return (m_notebook->get_current_page() == 0);
+}
+
+void
+NewSessionDialog::set_current_page(int page)
+{
+       return m_notebook->set_current_page (page);
 }
 
 void
@@ -510,11 +580,10 @@ NewSessionDialog::reset_name()
 {
        m_name->set_text("");
        set_response_sensitive (Gtk::RESPONSE_OK, false);
-       
 }
 
-bool
-NewSessionDialog::entry_key_release (GdkEventKey* ev)
+void
+NewSessionDialog::on_new_session_name_entry_changed ()
 {
        if (m_name->get_text() != "") {
                set_response_sensitive (Gtk::RESPONSE_OK, true);
@@ -522,14 +591,12 @@ NewSessionDialog::entry_key_release (GdkEventKey* ev)
        } else {
                set_response_sensitive (Gtk::RESPONSE_OK, false);
        }
-       return true;
 }
 
 void
 NewSessionDialog::notebook_page_changed (GtkNotebookPage* np, uint pagenum)
 {
-       if (pagenum == 1) {
-               on_new_session_page = false;
+       if (!on_new_session_page ()) {
                m_okbutton->set_label(_("Open"));
                set_response_sensitive (Gtk::RESPONSE_NONE, false);
                m_okbutton->set_image (*(new Gtk::Image (Gtk::Stock::OPEN, Gtk::ICON_SIZE_BUTTON)));
@@ -539,7 +606,6 @@ NewSessionDialog::notebook_page_changed (GtkNotebookPage* np, uint pagenum)
                        set_response_sensitive (Gtk::RESPONSE_OK, true);
                }
        } else {
-               on_new_session_page = true;
                if (m_name->get_text() != "") {
                        set_response_sensitive (Gtk::RESPONSE_NONE, true);
                }
@@ -570,7 +636,7 @@ NewSessionDialog::treeview_selection_changed ()
 void
 NewSessionDialog::file_chosen ()
 {
-       if (on_new_session_page) return;
+       if (on_new_session_page ()) return;
 
        m_treeview->get_selection()->unselect_all();
 
@@ -597,68 +663,68 @@ NewSessionDialog::template_chosen ()
 void
 NewSessionDialog::recent_row_activated (const Gtk::TreePath& path, Gtk::TreeViewColumn* col)
 {
-        response (Gtk::RESPONSE_YES);
+       response (Gtk::RESPONSE_YES);
 }
 
 void
 NewSessionDialog::connect_inputs_clicked ()
 {
-        m_limit_input_ports->set_sensitive(m_connect_inputs->get_active());
+       m_limit_input_ports->set_sensitive(m_connect_inputs->get_active());
 
-               if (m_connect_inputs->get_active() && m_limit_input_ports->get_active()) {
-               m_input_limit_count->set_sensitive(true);
-               } else {
-               m_input_limit_count->set_sensitive(false);
-               }
+       if (m_connect_inputs->get_active() && m_limit_input_ports->get_active()) {
+               m_input_limit_count->set_sensitive(true);
+       } else {
+               m_input_limit_count->set_sensitive(false);
+       }
 }
 
 void
 NewSessionDialog::connect_outputs_clicked ()
 {
-        m_limit_output_ports->set_sensitive(m_connect_outputs->get_active());
+       m_limit_output_ports->set_sensitive(m_connect_outputs->get_active());
 
-               if (m_connect_outputs->get_active() && m_limit_output_ports->get_active()) {
-               m_output_limit_count->set_sensitive(true);
-               } else {
-               m_output_limit_count->set_sensitive(false);
-               }
+       if (m_connect_outputs->get_active() && m_limit_output_ports->get_active()) {
+               m_output_limit_count->set_sensitive(true);
+       } else {
+               m_output_limit_count->set_sensitive(false);
+       }
 }
 
 void
 NewSessionDialog::limit_inputs_clicked ()
 {
-        m_input_limit_count->set_sensitive(m_limit_input_ports->get_active());
+       m_input_limit_count->set_sensitive(m_limit_input_ports->get_active());
 }
 
 void
 NewSessionDialog::limit_outputs_clicked ()
 {
-        m_output_limit_count->set_sensitive(m_limit_output_ports->get_active());
+       m_output_limit_count->set_sensitive(m_limit_output_ports->get_active());
 }
 
 void
 NewSessionDialog::master_bus_button_clicked ()
 {
-        m_master_bus_channel_count->set_sensitive(m_create_master_bus->get_active());
+       m_master_bus_channel_count->set_sensitive(m_create_master_bus->get_active());
 }
 
 void
 NewSessionDialog::monitor_bus_button_clicked ()
 {
-        m_control_bus_channel_count->set_sensitive(m_create_control_bus->get_active());
+       m_control_bus_channel_count->set_sensitive(m_create_control_bus->get_active());
 }
 
 void
 NewSessionDialog::reset_template()
 {
-        m_template->set_filename("");
+       m_template->set_filename("");
 }
 
 void
 NewSessionDialog::reset_recent()
 {
-        /* Shamelessly ripped from ardour_ui.cc */
-        std::vector<string *> *sessions;
+       /* Shamelessly ripped from ardour_ui.cc */
+       std::vector<string *> *sessions;
        std::vector<string *>::iterator i;
        RecentSessionsSorter cmp;
        
@@ -677,21 +743,21 @@ NewSessionDialog::reset_recent()
        
        for (i = sessions->begin(); i != sessions->end(); ++i) {
 
-               std::vector<std::string*>* states;
+               std::vector<std::string*>* states;
                std::vector<const gchar*> item;
                std::string fullpath = *(*i);
                
                /* remove any trailing / */
                
                if (fullpath[fullpath.length()-1] == '/') {
-                       fullpath = fullpath.substr (0, fullpath.length()-1);
+                       fullpath = fullpath.substr (0, fullpath.length()-1);
                }
            
                /* now get available states for this session */
                  
                if ((states = ARDOUR::Session::possible_states (fullpath)) == 0) {
-                       /* no state file? */
-                       continue;
+                       /* no state file? */
+                       continue;
                }
            
                Gtk::TreeModel::Row row = *(recent_model->append());
@@ -699,17 +765,17 @@ NewSessionDialog::reset_recent()
                row[recent_columns.visible_name] = Glib::path_get_basename (fullpath);
                row[recent_columns.fullpath] = fullpath;
                
-               if (states->size() > 1) {
-                   
-                       /* add the children */
-                   
-                       for (std::vector<std::string*>::iterator i2 = states->begin(); i2 != states->end(); ++i2) {
-
-                               Gtk::TreeModel::Row child_row = *(recent_model->append (row.children()));
-                               
+               if (states->size()) {
+
+                       /* add the children */
+
+                       for (std::vector<std::string*>::iterator i2 = states->begin(); i2 != states->end(); ++i2) {
+
+                               Gtk::TreeModel::Row child_row = *(recent_model->append (row.children()));
+
                                child_row[recent_columns.visible_name] = **i2;
                                child_row[recent_columns.fullpath] = fullpath;
-                               
+
                                delete *i2;
                        }
                }