get current tab on startup correct.
authorPaul Davis <paul@linuxaudiosystems.com>
Mon, 27 Jul 2015 21:12:35 +0000 (17:12 -0400)
committerPaul Davis <paul@linuxaudiosystems.com>
Mon, 22 Feb 2016 20:31:23 +0000 (15:31 -0500)
Note that Gtk::Notebook will not switch to a page with an invisible widget. This took me TOO LONG to discover.
Also move code around between files

gtk2_ardour/ardour_ui.cc
gtk2_ardour/ardour_ui2.cc
gtk2_ardour/ardour_ui_dependents.cc
gtk2_ardour/ardour_ui_ed.cc
gtk2_ardour/ardour_ui_mixer.cc
gtk2_ardour/editor.cc

index 1b6f5eaf6d6093a2224b206bfdd5696c1ced8130..0d6cef5a0ef35df74e82285081c5a5e5a24f8921 100644 (file)
@@ -1052,6 +1052,8 @@ ARDOUR_UI::starting ()
                Glib::signal_timeout().connect (sigc::bind (sigc::ptr_fun (_hide_splash), this), 1000);
        }
 
+       /* all other dialogs are created conditionally */
+
        return 0;
 }
 
index eaae3c0e03a6e045e92dd8e4bb10c9e327938112..2e86fc703b404334eff77769e6810b085448fb64 100644 (file)
@@ -69,16 +69,6 @@ using namespace Glib;
 using namespace ARDOUR_UI_UTILS;
 
 
-static GtkNotebook*
-tab_window_root_drop (GtkNotebook* src,
-                     GtkWidget* w,
-                     gint x,
-                     gint y,
-                     gpointer user_data)
-{
-       return ARDOUR_UI::instance()->tab_window_root_drop (src, w, x, y, user_data);
-}
-
 bool
 ARDOUR_UI::tabs_button_event (GdkEventButton* ev)
 {
@@ -115,136 +105,6 @@ ARDOUR_UI::tabs_button_event (GdkEventButton* ev)
        return false;
 }
 
-int
-ARDOUR_UI::setup_windows ()
-{
-       /* we don't use a widget with its own window for the tab close button,
-          which makes it impossible to rely on GTK+ to generate signals for
-          events occuring "in" this widget. Instead, we pre-connect a
-          handler to the relevant events on the notebook and then check
-          to see if the event coordinates tell us that it occured "in"
-          the close button.
-       */
-       _tabs.signal_button_press_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_button_event), false);
-       _tabs.signal_button_release_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_button_event), false);
-       
-       if (create_editor ()) {
-               error << _("UI: cannot setup editor") << endmsg;
-               return -1;
-       }
-
-       if (create_mixer ()) {
-               error << _("UI: cannot setup mixer") << endmsg;
-               return -1;
-       }
-
-       if (create_meterbridge ()) {
-               error << _("UI: cannot setup meterbridge") << endmsg;
-               return -1;
-       }
-
-       rc_option_editor = new RCOptionEditor;
-       rc_option_editor->StateChange.connect (sigc::mem_fun (*this, &ARDOUR_UI::tabbable_state_change));
-       rc_option_editor->add_to_notebook (_tabs, _("Preferences"));
-       
-       /* all other dialogs are created conditionally */
-
-       we_have_dependents ();
-
-#ifdef TOP_MENUBAR
-       EventBox* status_bar_event_box = manage (new EventBox);
-
-       status_bar_event_box->add (status_bar_label);
-       status_bar_event_box->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
-       status_bar_label.set_size_request (300, -1);
-
-       status_bar_label.show ();
-       status_bar_event_box->show ();
-
-       status_bar_event_box->signal_button_press_event().connect (mem_fun (*this, &ARDOUR_UI::status_bar_button_press));
-
-       status_bar_hpacker.pack_start (*status_bar_event_box, true, true, 6);
-       status_bar_hpacker.pack_start (menu_bar_base, false, false, 2);
-#else
-       top_packer.pack_start (menu_bar_base, false, false);
-#endif
-
-       main_vpacker.pack_start (top_packer, false, false);
-
-       /* now add the transport frame to the top of main window */
-       
-       main_vpacker.pack_start (transport_frame, false, false);
-       main_vpacker.pack_start (_tabs, true, true);
-
-#ifdef TOP_MENUBAR
-       main_vpacker.pack_start (status_bar_hpacker, false, false);
-#endif
-
-       setup_transport();
-       build_menu_bar ();
-       setup_tooltips ();
-
-       _main_window.signal_delete_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::main_window_delete_event));
-       
-       /* pack the main vpacker into the main window and show everything
-        */
-
-       _main_window.add (main_vpacker);
-       transport_frame.show_all ();
-
-       const XMLNode* mnode = main_window_settings ();
-
-       if (mnode) {
-               const XMLProperty* prop;
-               gint x = -1;
-               gint y = -1;
-               gint w = -1;
-               gint h = -1;
-
-               if ((prop = mnode->property (X_("x"))) != 0) {
-                       x = atoi (prop->value());
-               }
-
-               if ((prop = mnode->property (X_("y"))) != 0) {
-                       y = atoi (prop->value());
-               } 
-
-               if ((prop = mnode->property (X_("w"))) != 0) {
-                       w = atoi (prop->value());
-               } 
-               
-               if ((prop = mnode->property (X_("h"))) != 0) {
-                       h = atoi (prop->value());
-               }
-
-               if (x >= 0 && y >= 0 && w >= 0 && h >= 0) {
-                       _main_window.set_position (Gtk::WIN_POS_NONE);
-               }
-               
-               if (x >= 0 && y >= 0) {
-                       _main_window.move (x, y);
-               }
-               
-               if (w > 0 && h > 0) {
-                       _main_window.set_default_size (w, h);
-               }
-       }
-       
-       _main_window.show_all ();
-       setup_toplevel_window (_main_window, "", this);
-       
-       _tabs.signal_switch_page().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_switch));
-       _tabs.signal_page_removed().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_page_removed));
-       _tabs.signal_page_added().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_page_added));
-
-       /* It would be nice if Gtkmm had wrapped this rather than just
-        * deprecating the old set_window_creation_hook() method, but oh well...
-        */
-       g_signal_connect (_tabs.gobj(), "create-window", (GCallback) ::tab_window_root_drop, this);
-
-       return 0;
-}
-
 void
 ARDOUR_UI::tabs_page_removed (Gtk::Widget*, guint)
 {
@@ -268,13 +128,6 @@ ARDOUR_UI::tabs_page_added (Gtk::Widget*, guint)
 void
 ARDOUR_UI::tabs_switch (GtkNotebookPage*, guint page_number)
 {
-       if (page_number == 2) {
-               if (!rc_option_editor) {
-                       rc_option_editor = new RCOptionEditor;
-                       rc_option_editor_placeholder.pack_start (*rc_option_editor, true, true);
-                       rc_option_editor_placeholder.show_all ();
-               }
-       }
 }
 
 void
@@ -850,11 +703,12 @@ ARDOUR_UI::restore_editing_space ()
 void
 ARDOUR_UI::show_ui_prefs ()
 {
-       tabs().set_current_page (2);
-       rc_option_editor->set_current_page (_("GUI"));
+       if (rc_option_editor) {
+               show_tabbable (rc_option_editor);
+               rc_option_editor->set_current_page (_("GUI"));
+       }
 }
 
-
 bool
 ARDOUR_UI::click_button_clicked (GdkEventButton* ev)
 {
@@ -863,7 +717,7 @@ ARDOUR_UI::click_button_clicked (GdkEventButton* ev)
                return false;
        }
 
-       tabs().set_current_page (2);
+       show_tabbable (rc_option_editor);
        rc_option_editor->set_current_page (_("Misc"));
        return true;
 }
index 0d7b5a13532460febeb27f68a859990589598c11..42816dd482ae9db9d16b938317ef38d56cfac192 100644 (file)
@@ -193,3 +193,163 @@ ARDOUR_UI::main_window_delete_event (GdkEventAny* ev)
        
        return true;
 }
+
+static GtkNotebook*
+tab_window_root_drop (GtkNotebook* src,
+                     GtkWidget* w,
+                     gint x,
+                     gint y,
+                     gpointer user_data)
+{
+       return ARDOUR_UI::instance()->tab_window_root_drop (src, w, x, y, user_data);
+}
+
+int
+ARDOUR_UI::setup_windows ()
+{
+       /* we don't use a widget with its own window for the tab close button,
+          which makes it impossible to rely on GTK+ to generate signals for
+          events occuring "in" this widget. Instead, we pre-connect a
+          handler to the relevant events on the notebook and then check
+          to see if the event coordinates tell us that it occured "in"
+          the close button.
+       */
+       _tabs.signal_button_press_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_button_event), false);
+       _tabs.signal_button_release_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_button_event), false);
+
+       rc_option_editor = new RCOptionEditor;
+       rc_option_editor->StateChange.connect (sigc::mem_fun (*this, &ARDOUR_UI::tabbable_state_change));
+
+       if (create_editor ()) {
+               error << _("UI: cannot setup editor") << endmsg;
+               return -1;
+       }
+
+       if (create_mixer ()) {
+               error << _("UI: cannot setup mixer") << endmsg;
+               return -1;
+       }
+
+       if (create_meterbridge ()) {
+               error << _("UI: cannot setup meterbridge") << endmsg;
+               return -1;
+       }
+
+       /* order of addition affects order seen in initial window display */
+       
+       rc_option_editor->add_to_notebook (_tabs, _("Preferences"));
+       mixer->add_to_notebook (_tabs, _("Mixer"));
+       editor->add_to_notebook (_tabs, _("Editor"));
+
+       /* all other dialogs are created conditionally */
+
+       we_have_dependents ();
+
+#ifdef TOP_MENUBAR
+       EventBox* status_bar_event_box = manage (new EventBox);
+
+       status_bar_event_box->add (status_bar_label);
+       status_bar_event_box->add_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
+       status_bar_label.set_size_request (300, -1);
+
+       status_bar_label.show ();
+       status_bar_event_box->show ();
+
+       status_bar_event_box->signal_button_press_event().connect (mem_fun (*this, &ARDOUR_UI::status_bar_button_press));
+
+       status_bar_hpacker.pack_start (*status_bar_event_box, true, true, 6);
+       status_bar_hpacker.pack_start (menu_bar_base, false, false, 2);
+#else
+       top_packer.pack_start (menu_bar_base, false, false);
+#endif
+
+       main_vpacker.pack_start (top_packer, false, false);
+
+       /* now add the transport frame to the top of main window */
+       
+       main_vpacker.pack_start (transport_frame, false, false);
+       main_vpacker.pack_start (_tabs, true, true);
+
+#ifdef TOP_MENUBAR
+       main_vpacker.pack_start (status_bar_hpacker, false, false);
+#endif
+
+       setup_transport();
+       build_menu_bar ();
+       setup_tooltips ();
+
+       _main_window.signal_delete_event().connect (sigc::mem_fun (*this, &ARDOUR_UI::main_window_delete_event));
+       
+       /* pack the main vpacker into the main window and show everything
+        */
+
+       _main_window.add (main_vpacker);
+       transport_frame.show_all ();
+
+       const XMLNode* mnode = main_window_settings ();
+
+       if (mnode) {
+               const XMLProperty* prop;
+               gint x = -1;
+               gint y = -1;
+               gint w = -1;
+               gint h = -1;
+
+               if ((prop = mnode->property (X_("x"))) != 0) {
+                       x = atoi (prop->value());
+               }
+
+               if ((prop = mnode->property (X_("y"))) != 0) {
+                       y = atoi (prop->value());
+               } 
+
+               if ((prop = mnode->property (X_("w"))) != 0) {
+                       w = atoi (prop->value());
+               } 
+               
+               if ((prop = mnode->property (X_("h"))) != 0) {
+                       h = atoi (prop->value());
+               }
+
+               if (x >= 0 && y >= 0 && w >= 0 && h >= 0) {
+                       _main_window.set_position (Gtk::WIN_POS_NONE);
+               }
+               
+               if (x >= 0 && y >= 0) {
+                       _main_window.move (x, y);
+               }
+               
+               if (w > 0 && h > 0) {
+                       _main_window.set_default_size (w, h);
+               }
+
+               std::string current_tab;
+               
+               if ((prop = mnode->property (X_("current-tab"))) != 0) {
+                       current_tab = prop->value();
+               } else {
+                       current_tab = "editor";
+               }
+               if (mixer && current_tab == "mixer") {
+                       _tabs.set_current_page (_tabs.page_num (mixer->contents()));
+               } else if (rc_option_editor && current_tab == "preferences") {
+                       _tabs.set_current_page (_tabs.page_num (rc_option_editor->contents()));
+               } else if (editor) {
+                       _tabs.set_current_page (_tabs.page_num (editor->contents()));
+               }
+       }
+       
+       _main_window.show_all ();
+       setup_toplevel_window (_main_window, "", this);
+       
+       _tabs.signal_switch_page().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_switch));
+       _tabs.signal_page_removed().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_page_removed));
+       _tabs.signal_page_added().connect (sigc::mem_fun (*this, &ARDOUR_UI::tabs_page_added));
+
+       /* It would be nice if Gtkmm had wrapped this rather than just
+        * deprecating the old set_window_creation_hook() method, but oh well...
+        */
+       g_signal_connect (_tabs.gobj(), "create-window", (GCallback) ::tab_window_root_drop, this);
+
+       return 0;
+}
index 299a5d7d69e2f754c87ea0fef61baf7b64fe5291..f763b313e54a709006b16eb203d659e7e2556830 100644 (file)
@@ -80,7 +80,6 @@ ARDOUR_UI::create_editor ()
        try {
                editor = new Editor ();
                editor->StateChange.connect (sigc::mem_fun (*this, &ARDOUR_UI::tabbable_state_change));
-               editor->add_to_notebook (_tabs, _("Editor"));
        }
 
        catch (failed_constructor& err) {
@@ -668,6 +667,18 @@ ARDOUR_UI::save_ardour_state ()
        main_window_node.add_property (X_("w"), PBD::to_string (mw, std::dec));
        main_window_node.add_property (X_("h"), PBD::to_string (mh, std::dec));
 
+       string current_tab;
+       int current_page_number = _tabs.get_current_page ();
+       if (editor && (current_page_number == _tabs.page_num (editor->contents()))) {
+               current_tab = "editor";
+       } else if (mixer && (current_page_number == _tabs.page_num (mixer->contents()))) {
+               current_tab = "mixer";
+       } else if (rc_option_editor && (current_page_number == _tabs.page_num (rc_option_editor->contents()))) {
+               current_tab == "preferences";
+       }
+
+       main_window_node.add_property (X_("current-tab"), current_tab);
+       
        /* Windows */
 
        WM::Manager::instance().add_state (*window_node);
index a0b0519a26740507fcf1b319f60136c94b8f1e4c..6279f4c789439ecad791d644ad29747a33f56b56 100644 (file)
@@ -42,7 +42,6 @@ ARDOUR_UI::create_mixer ()
        try {
                mixer = Mixer_UI::instance ();
                mixer->StateChange.connect (sigc::mem_fun (*this, &ARDOUR_UI::tabbable_state_change));
-               mixer->add_to_notebook (_tabs, _("Mixer"));
        }
 
        catch (failed_constructor& err) {
index c8c96f3d878fb87a5adda937be5539f8baf2160c..2e4dbbb4384f722f4a8f39d2673d9175b15c1ce0 100644 (file)
@@ -747,6 +747,11 @@ Editor::Editor ()
        global_vpacker.pack_start (*hbox, true, true);
        global_hpacker.pack_start (global_vpacker, true, true);
 
+       /* need to show the "contents" widget so that notebook will show if tab is switched to
+        */
+
+       global_hpacker.show (); 
+
        /* register actions now so that set_state() can find them and set toggles/checks etc */
 
        register_actions ();