diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2015-07-27 17:12:35 -0400 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2016-02-22 15:31:23 -0500 |
commit | 8933d53271c902aa2ab3ef90910e2228671bd77b (patch) | |
tree | 648250f84aebfdf58fe091ee20da9453091127eb /gtk2_ardour | |
parent | 1be30e040169863006eb37d8ee2b8ad919febfa3 (diff) |
get current tab on startup correct.
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
Diffstat (limited to 'gtk2_ardour')
-rw-r--r-- | gtk2_ardour/ardour_ui.cc | 2 | ||||
-rw-r--r-- | gtk2_ardour/ardour_ui2.cc | 156 | ||||
-rw-r--r-- | gtk2_ardour/ardour_ui_dependents.cc | 160 | ||||
-rw-r--r-- | gtk2_ardour/ardour_ui_ed.cc | 13 | ||||
-rw-r--r-- | gtk2_ardour/ardour_ui_mixer.cc | 1 | ||||
-rw-r--r-- | gtk2_ardour/editor.cc | 5 |
6 files changed, 184 insertions, 153 deletions
diff --git a/gtk2_ardour/ardour_ui.cc b/gtk2_ardour/ardour_ui.cc index 1b6f5eaf6d..0d6cef5a0e 100644 --- a/gtk2_ardour/ardour_ui.cc +++ b/gtk2_ardour/ardour_ui.cc @@ -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; } diff --git a/gtk2_ardour/ardour_ui2.cc b/gtk2_ardour/ardour_ui2.cc index eaae3c0e03..2e86fc703b 100644 --- a/gtk2_ardour/ardour_ui2.cc +++ b/gtk2_ardour/ardour_ui2.cc @@ -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; } diff --git a/gtk2_ardour/ardour_ui_dependents.cc b/gtk2_ardour/ardour_ui_dependents.cc index 0d7b5a1353..42816dd482 100644 --- a/gtk2_ardour/ardour_ui_dependents.cc +++ b/gtk2_ardour/ardour_ui_dependents.cc @@ -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; +} diff --git a/gtk2_ardour/ardour_ui_ed.cc b/gtk2_ardour/ardour_ui_ed.cc index 299a5d7d69..f763b313e5 100644 --- a/gtk2_ardour/ardour_ui_ed.cc +++ b/gtk2_ardour/ardour_ui_ed.cc @@ -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); diff --git a/gtk2_ardour/ardour_ui_mixer.cc b/gtk2_ardour/ardour_ui_mixer.cc index a0b0519a26..6279f4c789 100644 --- a/gtk2_ardour/ardour_ui_mixer.cc +++ b/gtk2_ardour/ardour_ui_mixer.cc @@ -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) { diff --git a/gtk2_ardour/editor.cc b/gtk2_ardour/editor.cc index c8c96f3d87..2e4dbbb438 100644 --- a/gtk2_ardour/editor.cc +++ b/gtk2_ardour/editor.cc @@ -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 (); |