Use general VisibilityGroup class to manage visibility
[ardour.git] / gtk2_ardour / ardour_ui_ed.cc
index 0c67d4b59d31a57259aac3f0fded1e0326651eef..8ab5a9aa50fdea1d90f11db14589949556c61104 100644 (file)
 
 */
 
+#ifdef WAF_BUILD
+#include "gtk2ardour-config.h"
+#endif
+
 /* This file contains any ARDOUR_UI methods that require knowledge of
    the editor, and exists so that no compilation dependency exists
    between the main ARDOUR_UI modules and the PublicEditor class. This
@@ -94,7 +98,7 @@ ARDOUR_UI::install_actions ()
        /* menus + submenus that need action items */
 
        ActionManager::register_action (main_actions, X_("Session"), _("Session"));
-       act = ActionManager::register_action (main_actions, X_("Cleanup"), _("Cleanup"));
+       act = ActionManager::register_action (main_actions, X_("Cleanup"), _("Clean-up"));
        ActionManager::write_sensitive_actions.push_back (act);
        ActionManager::register_action (main_actions, X_("Sync"), _("Sync"));
        ActionManager::register_action (main_actions, X_("TransportOptions"), _("Options"));
@@ -143,6 +147,10 @@ ARDOUR_UI::install_actions ()
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::write_sensitive_actions.push_back (act);
 
+       act = ActionManager::register_action (main_actions, X_("Rename"), _("Rename..."), sigc::mem_fun(*this, &ARDOUR_UI::rename_session));
+       ActionManager::session_sensitive_actions.push_back (act);
+       ActionManager::write_sensitive_actions.push_back (act);
+
        act = ActionManager::register_action (main_actions, X_("SaveTemplate"), _("Save Template..."),  sigc::mem_fun(*this, &ARDOUR_UI::save_template));
        ActionManager::session_sensitive_actions.push_back (act);
 
@@ -158,10 +166,13 @@ ARDOUR_UI::install_actions ()
        act = ActionManager::register_action (main_actions, X_("ExportAudio"), _("Export To Audio File(s)..."),  sigc::mem_fun (*editor, &PublicEditor::export_audio));
        ActionManager::session_sensitive_actions.push_back (act);
 
+       act = ActionManager::register_action (main_actions, X_("StemExport"), _("Stem export..."),  sigc::mem_fun (*editor, &PublicEditor::stem_export));
+       ActionManager::session_sensitive_actions.push_back (act);
+
        act = ActionManager::register_action (main_actions, X_("Export"), _("Export"));
        ActionManager::session_sensitive_actions.push_back (act);
 
-       act = ActionManager::register_action (main_actions, X_("CleanupUnused"), _("Cleanup Unused Sources..."),  sigc::mem_fun (*(ARDOUR_UI::instance()), &ARDOUR_UI::cleanup));
+       act = ActionManager::register_action (main_actions, X_("CleanupUnused"), _("Clean-up Unused Sources..."),  sigc::mem_fun (*(ARDOUR_UI::instance()), &ARDOUR_UI::cleanup));
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::write_sensitive_actions.push_back (act);
 
@@ -183,23 +194,23 @@ ARDOUR_UI::install_actions ()
 
        RadioAction::Group jack_latency_group;
 
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency32"), X_("32"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 32));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency32"), X_("32"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 32));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency64"), X_("64"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 64));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency64"), X_("64"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 64));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency128"), X_("128"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 128));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency128"), X_("128"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 128));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency256"), X_("256"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 256));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency256"), X_("256"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 256));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency512"), X_("512"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 512));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency512"), X_("512"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 512));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency1024"), X_("1024"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 1024));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency1024"), X_("1024"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 1024));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency2048"), X_("2048"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 2048));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency2048"), X_("2048"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 2048));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency4096"), X_("4096"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 4096));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency4096"), X_("4096"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 4096));
        ActionManager::jack_sensitive_actions.push_back (act);
-       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency8192"), X_("8192"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (nframes_t) 8192));
+       act = ActionManager::register_radio_action (jack_actions, jack_latency_group, X_("JACKLatency8192"), X_("8192"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::set_jack_buffer_size), (pframes_t) 8192));
        ActionManager::jack_sensitive_actions.push_back (act);
 
        /* these actions are intended to be shared across all windows */
@@ -208,15 +219,14 @@ ARDOUR_UI::install_actions ()
        ActionManager::register_action (main_actions, X_("WindowMenu"), _("Window"));
        ActionManager::register_action (common_actions, X_("Quit"), _("Quit"), (hide_return (sigc::mem_fun(*this, &ARDOUR_UI::finish))));
 
-        /* windows visibility actions */
+       /* windows visibility actions */
 
        ActionManager::register_toggle_action (common_actions, X_("ToggleMaximalEditor"), _("Maximise Editor Space"), sigc::mem_fun (*this, &ARDOUR_UI::toggle_editing_space));
        act = ActionManager::register_toggle_action (common_actions, X_("KeepTearoffs"), _("Toolbars when Maximised"), mem_fun (*this, &ARDOUR_UI::toggle_keep_tearoffs));
        ActionManager::session_sensitive_actions.push_back (act);
 
-       ActionManager::register_action (common_actions, X_("goto-editor"), _("Show Editor"),  sigc::mem_fun(*this, &ARDOUR_UI::goto_editor_window));
-       ActionManager::register_action (common_actions, X_("goto-mixer"), _("Show Mixer"),  sigc::mem_fun(*this, &ARDOUR_UI::goto_mixer_window));
-       ActionManager::register_action (common_actions, X_("toggle-editor-mixer-on-top"), _("Toggle Editor Mixer on Top"),  sigc::mem_fun(*this, &ARDOUR_UI::toggle_editor_mixer_on_top));
+       ActionManager::register_toggle_action (common_actions, X_("toggle-mixer"), _("Mixer"),  sigc::mem_fun(*this, &ARDOUR_UI::toggle_mixer_window));
+       ActionManager::register_toggle_action (common_actions, X_("toggle-mixer-on-top"), _("Mixer on Top"),  sigc::mem_fun(*this, &ARDOUR_UI::toggle_mixer_on_top));
        ActionManager::register_toggle_action (common_actions, X_("ToggleRCOptionsEditor"), _("Preferences"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_rc_options_window));
        ActionManager::register_toggle_action (common_actions, X_("ToggleSessionOptionsEditor"), _("Properties"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_session_options_window));
        act = ActionManager::register_toggle_action (common_actions, X_("ToggleInspector"), _("Tracks and Busses"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_route_params_window));
@@ -226,6 +236,8 @@ ARDOUR_UI::install_actions ()
        ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_toggle_action (common_actions, X_("ToggleBigClock"), _("Big Clock"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_big_clock_window));
        ActionManager::session_sensitive_actions.push_back (act);
+       act = ActionManager::register_toggle_action (common_actions, X_("toggle-speaker-config"), _("Speaker Configuration"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_speaker_config_window));
+       ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_toggle_action (common_actions, X_("toggle-audio-connection-manager"), _("Audio Connection Manager"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::toggle_global_port_matrix), ARDOUR::DataType::AUDIO));
        ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_toggle_action (common_actions, X_("toggle-midi-connection-manager"), _("MIDI Connection Manager"), sigc::bind (sigc::mem_fun(*this, &ARDOUR_UI::toggle_global_port_matrix), ARDOUR::DataType::MIDI));
@@ -234,8 +246,8 @@ ARDOUR_UI::install_actions ()
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::register_action (common_actions, X_("About"), _("About"),  sigc::mem_fun(*this, &ARDOUR_UI::show_about));
        ActionManager::register_action (common_actions, X_("Chat"), _("Chat"),  sigc::mem_fun(*this, &ARDOUR_UI::launch_chat));
-        ActionManager::register_action (common_actions, X_("Manual"), _("Manual"),  mem_fun(*this, &ARDOUR_UI::launch_manual));
-        ActionManager::register_action (common_actions, X_("Reference"), _("Reference"),  mem_fun(*this, &ARDOUR_UI::launch_reference));
+       ActionManager::register_action (common_actions, X_("Manual"), _("Manual"),  mem_fun(*this, &ARDOUR_UI::launch_manual));
+       ActionManager::register_action (common_actions, X_("Reference"), _("Reference"),  mem_fun(*this, &ARDOUR_UI::launch_reference));
        ActionManager::register_toggle_action (common_actions, X_("ToggleThemeManager"), _("Theme Manager"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_theme_manager));
        ActionManager::register_toggle_action (common_actions, X_("ToggleKeyEditor"), _("Key Bindings"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_key_editor));
        ActionManager::register_toggle_action (common_actions, X_("ToggleBundleManager"), _("Bundle Manager"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_bundle_manager));
@@ -300,7 +312,7 @@ ARDOUR_UI::install_actions ()
        act = ActionManager::register_action (transport_actions, X_("Loop"), _("Play Loop Range"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_session_auto_loop));
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::transport_sensitive_actions.push_back (act);
-       act = ActionManager::register_action (transport_actions, X_("PlaySelection"), _("Play Selection"), sigc::mem_fun(*this, &ARDOUR_UI::transport_play_selection));
+       act = ActionManager::register_action (transport_actions, X_("PlaySelection"), _("Play Selected Range"), sigc::mem_fun(*this, &ARDOUR_UI::transport_play_selection));
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::transport_sensitive_actions.push_back (act);
 
@@ -346,31 +358,34 @@ ARDOUR_UI::install_actions ()
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::transport_sensitive_actions.push_back (act);
 
+       act = ActionManager::register_action (transport_actions, X_("primary-clock-timecode"), _("Timecode"), sigc::bind (sigc::mem_fun(primary_clock, &AudioClock::set_mode), AudioClock::Timecode));
+       ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_action (transport_actions, X_("primary-clock-bbt"), _("Bars & Beats"), sigc::bind (sigc::mem_fun(primary_clock, &AudioClock::set_mode), AudioClock::BBT));
        ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_action (transport_actions, X_("primary-clock-minsec"), _("Minutes & Seconds"), sigc::bind (sigc::mem_fun(primary_clock, &AudioClock::set_mode), AudioClock::MinSec));
        ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_action (transport_actions, X_("primary-clock-samples"), _("Samples"), sigc::bind (sigc::mem_fun(primary_clock, &AudioClock::set_mode), AudioClock::Frames));
        ActionManager::session_sensitive_actions.push_back (act);
-       act = ActionManager::register_action (transport_actions, X_("primary-clock-off"), _("Off"), sigc::bind (sigc::mem_fun(primary_clock, &AudioClock::set_mode), AudioClock::Off));
-       ActionManager::session_sensitive_actions.push_back (act);
 
+       act = ActionManager::register_action (transport_actions, X_("secondary-clock-timecode"), _("Timecode"), sigc::bind (sigc::mem_fun(secondary_clock, &AudioClock::set_mode), AudioClock::Timecode));
+       ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_action (transport_actions, X_("secondary-clock-bbt"), _("Bars & Beats"), sigc::bind (sigc::mem_fun(secondary_clock, &AudioClock::set_mode), AudioClock::BBT));
        ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_action (transport_actions, X_("secondary-clock-minsec"), _("Minutes & Seconds"), sigc::bind (sigc::mem_fun(secondary_clock, &AudioClock::set_mode), AudioClock::MinSec));
        ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_action (transport_actions, X_("secondary-clock-samples"), _("Samples"), sigc::bind (sigc::mem_fun(secondary_clock, &AudioClock::set_mode), AudioClock::Frames));
        ActionManager::session_sensitive_actions.push_back (act);
-       act = ActionManager::register_action (transport_actions, X_("secondary-clock-off"), _("Off"), sigc::bind (sigc::mem_fun(secondary_clock, &AudioClock::set_mode), AudioClock::Off));
-       ActionManager::session_sensitive_actions.push_back (act);
 
        act = ActionManager::register_toggle_action (transport_actions, X_("TogglePunchIn"), _("Punch In"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_punch_in));
+       act->set_short_label (_("In"));
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::transport_sensitive_actions.push_back (act);
        act = ActionManager::register_toggle_action (transport_actions, X_("TogglePunchOut"), _("Punch Out"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_punch_out));
+       act->set_short_label (_("Out"));
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::transport_sensitive_actions.push_back (act);
        act = ActionManager::register_toggle_action (transport_actions, X_("TogglePunch"), _("Punch In/Out"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_punch));
+       act->set_short_label (_("In/Out"));
        ActionManager::session_sensitive_actions.push_back (act);
        ActionManager::transport_sensitive_actions.push_back (act);
        act = ActionManager::register_toggle_action (transport_actions, X_("ToggleClick"), _("Click"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_click));
@@ -390,7 +405,7 @@ ARDOUR_UI::install_actions ()
        ActionManager::session_sensitive_actions.push_back (act);
        act = ActionManager::register_toggle_action (transport_actions, X_("ToggleTimeMaster"), _("Time Master"), sigc::mem_fun(*this, &ARDOUR_UI::toggle_time_master));
        ActionManager::session_sensitive_actions.push_back (act);
-       act = ActionManager::register_toggle_action (transport_actions, X_("ToggleExternalSync"), _(""), sigc::mem_fun(*this, &ARDOUR_UI::toggle_external_sync));
+       act = ActionManager::register_toggle_action (transport_actions, X_("ToggleExternalSync"), "", sigc::mem_fun(*this, &ARDOUR_UI::toggle_external_sync));
        ActionManager::session_sensitive_actions.push_back (act);
 
        for (int i = 1; i <= 32; ++i) {
@@ -419,15 +434,15 @@ ARDOUR_UI::install_actions ()
        ActionManager::session_sensitive_actions.push_back (act);
 
 
-        if (getenv ("ARDOUR_BUNDLED")) {
-                act = ActionManager::register_toggle_action (main_actions, X_("EnableTranslation"), _("Enable Translations"), mem_fun (*this, &ARDOUR_UI::toggle_translations));
-                if (act) {
-                        RefPtr<ToggleAction> ract = RefPtr<ToggleAction>::cast_dynamic (act);
-                        if (ract) {
-                                ract->set_active (!ARDOUR::translations_are_disabled());
-                        }
-                }
-        }
+       if (getenv ("ARDOUR_BUNDLED")) {
+               act = ActionManager::register_toggle_action (main_actions, X_("EnableTranslation"), _("Enable Translations"), mem_fun (*this, &ARDOUR_UI::toggle_translations));
+               if (act) {
+                       RefPtr<ToggleAction> ract = RefPtr<ToggleAction>::cast_dynamic (act);
+                       if (ract) {
+                               ract->set_active (!ARDOUR::translations_are_disabled());
+                       }
+               }
+       }
 
        ActionManager::add_action_group (shuttle_actions);
        ActionManager::add_action_group (option_actions);
@@ -438,7 +453,7 @@ ARDOUR_UI::install_actions ()
 }
 
 void
-ARDOUR_UI::set_jack_buffer_size (nframes_t nframes)
+ARDOUR_UI::set_jack_buffer_size (pframes_t nframes)
 {
        Glib::RefPtr<Action> action;
        const char* action_name = 0;
@@ -507,6 +522,13 @@ ARDOUR_UI::build_menu_bar ()
        // so use this instead ...
        gtk_settings_set_long_property (gtk_settings_get_default(), "gtk-can-change-accels", 1, "Ardour:designers");
 
+       EventBox* ev = manage (new EventBox);
+       ev->show ();
+       HBox* hbox = manage (new HBox);
+       hbox->show ();
+       
+       ev->add (*hbox);
+
        wall_clock_box.add (wall_clock_label);
        wall_clock_box.set_name ("WallClock");
        wall_clock_label.set_name ("WallClock");
@@ -514,39 +536,67 @@ ARDOUR_UI::build_menu_bar ()
        disk_space_box.add (disk_space_label);
        disk_space_box.set_name ("WallClock");
        disk_space_label.set_name ("WallClock");
+       disk_space_label.set_use_markup ();
 
        cpu_load_box.add (cpu_load_label);
        cpu_load_box.set_name ("CPULoad");
        cpu_load_label.set_name ("CPULoad");
+       cpu_load_label.set_use_markup ();
+       set_size_request_to_display_given_text (cpu_load_label, "DSP: 100.0%", 2, 2);
 
        buffer_load_box.add (buffer_load_label);
        buffer_load_box.set_name ("BufferLoad");
        buffer_load_label.set_name ("BufferLoad");
+       buffer_load_label.set_use_markup ();
+       set_size_request_to_display_given_text (buffer_load_label, "Buffers: p:100% c:100%", 2, 2);
 
        sample_rate_box.add (sample_rate_label);
        sample_rate_box.set_name ("SampleRate");
        sample_rate_label.set_name ("SampleRate");
+       sample_rate_label.set_use_markup ();
+
+       format_box.add (format_label);
+       format_box.set_name ("Format");
+       format_label.set_name ("Format");
+       format_label.set_use_markup ();
 
 #ifndef TOP_MENUBAR
-       menu_hbox.pack_start (*menu_bar, true, true);
+       menu_hbox.pack_start (*menu_bar, false, false);
 #else
        use_menubar_as_top_menubar ();
 #endif
 
+       bool wall_clock = false;
+       bool disk_space = false;
+
        if (!Profile->get_small_screen()) {
 #ifndef GTKOSX
                // OSX provides its own wallclock, thank you very much
-               menu_hbox.pack_end (wall_clock_box, false, false, 2);
+               wall_clock = true;
 #endif
-               menu_hbox.pack_end (disk_space_box, false, false, 4);
+               disk_space = true;
        }
+       
+       hbox->pack_end (wall_clock_box, false, false, 2);
+       hbox->pack_end (disk_space_box, false, false, 4);
+       hbox->pack_end (cpu_load_box, false, false, 4);
+       hbox->pack_end (buffer_load_box, false, false, 4);
+       hbox->pack_end (sample_rate_box, false, false, 4);
+       hbox->pack_end (format_box, false, false, 4);
 
-       menu_hbox.pack_end (cpu_load_box, false, false, 4);
-       menu_hbox.pack_end (buffer_load_box, false, false, 4);
-       menu_hbox.pack_end (sample_rate_box, false, false, 4);
+       menu_hbox.pack_end (*ev, false, false);
 
        menu_bar_base.set_name ("MainMenuBar");
        menu_bar_base.add (menu_hbox);
+
+       _status_bar_visibility.add (&wall_clock_box,  X_("WallClock"), _("Wall Clock"), wall_clock);
+       _status_bar_visibility.add (&disk_space_box,  X_("Disk"),      _("Disk Space"), disk_space);
+       _status_bar_visibility.add (&cpu_load_box,    X_("DSP"),       _("DSP"));
+       _status_bar_visibility.add (&buffer_load_box, X_("Buffers"),   _("Buffers"));
+       _status_bar_visibility.add (&sample_rate_box, X_("JACK"),      _("JACK Sampling Rate and Latency"));
+       _status_bar_visibility.add (&format_box,      X_("Format"),    _("File Format"));
+
+       ev->signal_button_press_event().connect (sigc::mem_fun (_status_bar_visibility, &VisibilityGroup::button_press_event));
 }
 
 void
@@ -583,10 +633,9 @@ ARDOUR_UI::setup_clock ()
 
        big_clock_window->get()->set_keep_above (true);
        big_clock_window->get()->set_border_width (0);
-       big_clock_window->get()->add (big_clock);
+       big_clock_window->get()->add (*big_clock);
 
        big_clock_window->get()->set_title (_("Big Clock"));
-       big_clock_window->get()->set_type_hint (Gdk::WINDOW_TYPE_HINT_UTILITY);
        big_clock_window->get()->signal_realize().connect (sigc::mem_fun (*this, &ARDOUR_UI::big_clock_realized));
        big_clock_window->get()->signal_unmap().connect (sigc::bind (sigc::ptr_fun(&ActionManager::uncheck_toggleaction), X_("<Actions>/Common/ToggleBigClock")));
        big_clock_window->get()->signal_key_press_event().connect (sigc::bind (sigc::ptr_fun (relay_key_press), big_clock_window->get()), false);
@@ -603,15 +652,15 @@ ARDOUR_UI::big_clock_realized ()
        set_decoration (big_clock_window->get(), (Gdk::DECOR_BORDER|Gdk::DECOR_RESIZEH));
        big_clock_window->get()->get_window()->get_geometry (x, y, w, big_clock_height, d);
 
-        original_big_clock_height = big_clock_height;
-        original_big_clock_width = w;
+       original_big_clock_height = big_clock_height;
+       original_big_clock_width = w;
+
+       Pango::FontDescription fd (big_clock->get_style()->get_font());
+       original_big_clock_font_size = fd.get_size ();
 
-        Pango::FontDescription fd (big_clock.get_style()->get_font());
-        original_big_clock_font_size = fd.get_size ();
-        
-        if (!fd.get_size_is_absolute ()) {
-                original_big_clock_font_size /= PANGO_SCALE;
-        }
+       if (!fd.get_size_is_absolute ()) {
+               original_big_clock_font_size /= PANGO_SCALE;
+       }
 
        big_clock_window->setup ();
 }
@@ -643,9 +692,9 @@ ARDOUR_UI::idle_big_clock_text_resizer (int, int)
        big_clock_resize_in_progress = false;
 
        Glib::RefPtr<Gdk::Window> win = big_clock_window->get()->get_window();
-       Pango::FontDescription fd (big_clock.get_style()->get_font());
+       Pango::FontDescription fd (big_clock->get_style()->get_font());
        int current_size = fd.get_size ();
-        int x, y, w, h, d;
+       int x, y, w, h, d;
 
        if (!fd.get_size_is_absolute ()) {
                current_size /= PANGO_SCALE;
@@ -653,28 +702,28 @@ ARDOUR_UI::idle_big_clock_text_resizer (int, int)
 
        win->get_geometry (x, y, w, h, d);
 
-        double scale  = min (((double) w / (double) original_big_clock_width), 
-                             ((double) h / (double) original_big_clock_height));
-
-        int size = (int) lrintf (original_big_clock_font_size * scale);
-
-        if (size != current_size) {
-
-                string family = fd.get_family();
-                char buf[family.length()+16];
-                snprintf (buf, family.length()+16, "%s %d", family.c_str(), size);
-                
-                try { 
-                        Pango::FontDescription fd (buf);
-                        Glib::RefPtr<Gtk::RcStyle> rcstyle = big_clock.get_modifier_style ();
-                        rcstyle->set_font (fd);
-                        big_clock.modify_style (rcstyle);
-                } 
-                
-                catch (...) {
-                        /* oh well, do nothing */
-                }
-        }
+       double scale  = min (((double) w / (double) original_big_clock_width),
+                            ((double) h / (double) original_big_clock_height));
+
+       int size = (int) lrintf (original_big_clock_font_size * scale);
+
+       if (size != current_size) {
+
+               string family = fd.get_family();
+               char buf[family.length()+16];
+               snprintf (buf, family.length()+16, "%s %d", family.c_str(), size);
+
+               try {
+                       Pango::FontDescription fd (buf);
+                       Glib::RefPtr<Gtk::RcStyle> rcstyle = big_clock->get_modifier_style ();
+                       rcstyle->set_font (fd);
+                       big_clock->modify_style (rcstyle);
+               }
+
+               catch (...) {
+                       /* oh well, do nothing */
+               }
+       }
 
        return false;
 }
@@ -694,48 +743,51 @@ ARDOUR_UI::save_ardour_state ()
        Config->add_extra_xml (*node);
        Config->add_extra_xml (get_transport_controllable_state());
 
-        XMLNode* window_node = new XMLNode (X_("UI"));
+       XMLNode* window_node = new XMLNode (X_("UI"));
+       window_node->add_property (_status_bar_visibility.get_state_name().c_str(), _status_bar_visibility.get_state_value ());
 
        for (list<WindowProxyBase*>::iterator i = _window_proxies.begin(); i != _window_proxies.end(); ++i) {
                if ((*i)->rc_configured()) {
                        window_node->add_child_nocopy (*((*i)->get_state ()));
                }
        }
-        
-        /* tearoffs */
 
-        XMLNode* tearoff_node = new XMLNode (X_("Tearoffs"));
+       /* tearoffs */
+
+       XMLNode* tearoff_node = new XMLNode (X_("Tearoffs"));
 
-        if (transport_tearoff) {
-                XMLNode* t = new XMLNode (X_("transport"));
-                transport_tearoff->add_state (*t);
-                tearoff_node->add_child_nocopy (*t);
-        } 
+       if (transport_tearoff) {
+               XMLNode* t = new XMLNode (X_("transport"));
+               transport_tearoff->add_state (*t);
+               tearoff_node->add_child_nocopy (*t);
+       }
+
+       if (mixer && mixer->monitor_section()) {
+               XMLNode* t = new XMLNode (X_("monitor-section"));
+               mixer->monitor_section()->tearoff().add_state (*t);
+               tearoff_node->add_child_nocopy (*t);
+       }
 
-        if (mixer && mixer->monitor_section()) {
-                XMLNode* t = new XMLNode (X_("monitor-section"));
-                mixer->monitor_section()->tearoff().add_state (*t);
-                tearoff_node->add_child_nocopy (*t);
-        } 
+       if (editor && editor->mouse_mode_tearoff()) {
+               XMLNode* t = new XMLNode (X_("mouse-mode"));
+               editor->mouse_mode_tearoff ()->add_state (*t);
+               tearoff_node->add_child_nocopy (*t);
+       }
 
-        if (editor && editor->mouse_mode_tearoff()) {
-                XMLNode* t = new XMLNode (X_("mouse-mode"));
-                editor->mouse_mode_tearoff ()->add_state (*t);
-                tearoff_node->add_child_nocopy (*t);
-        } 
-        
-        window_node->add_child_nocopy (*tearoff_node);
+       window_node->add_child_nocopy (*tearoff_node);
 
-        Config->add_extra_xml (*window_node);
+       Config->add_extra_xml (*window_node);
 
        if (_startup && _startup->engine_control() && _startup->engine_control()->was_used()) {
                Config->add_extra_xml (_startup->engine_control()->get_state());
        }
        Config->save_state();
-       ui_config->save_state ();
+       if (ui_config->dirty()) {
+               ui_config->save_state ();
+       }
 
-       XMLNode enode(static_cast<Stateful*>(editor)->get_state());
-       XMLNode mnode(mixer->get_state());
+       XMLNode& enode (static_cast<Stateful*>(editor)->get_state());
+       XMLNode& mnode (mixer->get_state());
 
        if (_session) {
                _session->add_instant_xml (enode);
@@ -752,7 +804,7 @@ void
 ARDOUR_UI::toggle_global_port_matrix (ARDOUR::DataType t)
 {
        std::string const action = string_compose ("toggle-%1-connection-manager", t.to_string ());
-       
+
        if (_global_port_matrix[t]->get() == 0) {
                _global_port_matrix[t]->set (new GlobalPortMatrixWindow (_session, t));
                _global_port_matrix[t]->get()->signal_unmap().connect(sigc::bind (sigc::ptr_fun (&ActionManager::uncheck_toggleaction), string_compose (X_("<Actions>/Common/%1"), action)));
@@ -769,4 +821,3 @@ ARDOUR_UI::toggle_global_port_matrix (ARDOUR::DataType t)
                }
        }
 }
-