X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Foption_editor.cc;h=1088d1d9b4f27708f75e1a2f2c09ebba51882d46;hb=7745a373940a0bcabd618b5b66c6b8916045ea9e;hp=ad34e8ed0b38449ad117cff0686e1e7c7e4365d4;hpb=c69ef10067cbdde5fc87ff676097aa4ffe406f2c;p=ardour.git diff --git a/gtk2_ardour/option_editor.cc b/gtk2_ardour/option_editor.cc index ad34e8ed0b..1088d1d9b4 100644 --- a/gtk2_ardour/option_editor.cc +++ b/gtk2_ardour/option_editor.cc @@ -1,1006 +1,746 @@ /* - Copyright (C) 2001-2006 Paul Davis + Copyright (C) 2001-2009 Paul Davis - This program is free software; you can redistribute it and/or modify - it under the terms of the GNU General Public License as published by - the Free Software Foundation; either version 2 of the License, or - (at your option) any later version. + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 2 of the License, or + (at your option) any later version. - This program is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - GNU General Public License for more details. + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. - You should have received a copy of the GNU General Public License - along with this program; if not, write to the Free Software - Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. + You should have received a copy of the GNU General Public License + along with this program; if not, write to the Free Software + Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ +#include -#include +#include +#include +#include "gtkmm2ext/utils.h" -#include -#include -#include -#include -#include -#include -#include -#include -#include -#include +#include "ardour/dB.h" +#include "ardour/rc_configuration.h" +#include "ardour/session.h" +#include "ardour/types.h" +#include "ardour/utils.h" +#include "pbd/configuration.h" +#include "pbd/replace_all.h" +#include "pbd/strsplit.h" + +#include "gui_thread.h" +#include "option_editor.h" #include "public_editor.h" -#include "keyboard.h" -#include "mixer_ui.h" -#include "ardour_ui.h" -#include "io_selector.h" -#include "gain_meter.h" -#include "sfdb_ui.h" #include "utils.h" -#include "editing.h" -#include "option_editor.h" - #include "i18n.h" -using namespace ARDOUR; -using namespace PBD; +using namespace std; using namespace Gtk; -using namespace Editing; using namespace Gtkmm2ext; -using namespace std; - -static vector positional_sync_strings; - -OptionEditor::OptionEditor (ARDOUR_UI& uip, PublicEditor& ed, Mixer_UI& mixui) - : Dialog ("options editor"), - ui (uip), - editor (ed), - mixer (mixui), - - /* Paths */ - path_table (11, 2), - - /* Fades */ - - short_xfade_adjustment (0, 1.0, 500.0, 5.0, 100.0), - short_xfade_slider (short_xfade_adjustment), - destructo_xfade_adjustment (1.0, 1.0, 500.0, 1.0, 100.0), - destructo_xfade_slider (destructo_xfade_adjustment), - - /* Sync */ - - smpte_offset_clock (X_("smpteoffset"), false, X_("SMPTEOffsetClock"), true, true), - smpte_offset_negative_button (_("SMPTE offset is negative")), - - /* MIDI */ - - mmc_device_id_adjustment (0.0, 0.0, (double) 0x7f, 1.0, 16.0), - mmc_device_id_spinner (mmc_device_id_adjustment), - - /* Click */ - - click_table (2, 3), - click_browse_button (_("Browse")), - click_emphasis_browse_button (_("Browse")), - - /* kbd/mouse */ +using namespace ARDOUR; - keyboard_mouse_table (3, 4), - delete_button_adjustment (3, 1, 5), - delete_button_spin (delete_button_adjustment), - edit_button_adjustment (3, 1, 5), - edit_button_spin (edit_button_adjustment) - +void +OptionEditorComponent::add_widget_to_page (OptionEditorPage* p, Gtk::Widget* w) { - using namespace Notebook_Helpers; - - click_io_selector = 0; - auditioner_io_selector = 0; - session = 0; - - WindowTitle title(Glib::get_application_name()); - title += _("Options Editor"); - set_title(title.get_string()); - - set_default_size (300, 300); - set_wmclass (X_("ardour_option_editor"), "Ardour"); - - set_name ("OptionsWindow"); - add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK); - - VBox *vbox = get_vbox(); - set_border_width (3); - - vbox->set_spacing (4); - vbox->pack_start(notebook); - - signal_delete_event().connect (mem_fun(*this, &OptionEditor::wm_close)); - - notebook.set_show_tabs (true); - notebook.set_show_border (true); - notebook.set_name ("OptionsNotebook"); - - setup_sync_options(); - setup_path_options(); - setup_fade_options (); - setup_keyboard_options (); - setup_auditioner_editor (); - - notebook.pages().push_back (TabElem (sync_packer, _("Sync"))); - notebook.pages().push_back (TabElem (path_table, _("Paths/Files"))); - notebook.pages().push_back (TabElem (keyboard_mouse_table, _("Kbd/Mouse"))); - notebook.pages().push_back (TabElem (click_packer, _("Click"))); - notebook.pages().push_back (TabElem (audition_packer, _("Audition"))); - notebook.pages().push_back (TabElem (fade_packer, _("Layers & Fades"))); - - if (!MIDI::Manager::instance()->get_midi_ports().empty()) { - setup_midi_options (); - notebook.pages().push_back (TabElem (midi_packer, _("MIDI"))); + int const n = p->table.property_n_rows(); + int m = n + 1; + if (!_note.empty ()) { + ++m; } - set_session (0); - show_all_children(); + p->table.resize (m, 3); + p->table.attach (*w, 1, 3, n, n + 1, FILL | EXPAND); + + maybe_add_note (p, n + 1); } void -OptionEditor::set_session (Session *s) +OptionEditorComponent::add_widgets_to_page (OptionEditorPage* p, Gtk::Widget* wa, Gtk::Widget* wb) { - clear_click_editor (); - clear_auditioner_editor (); - - click_path_entry.set_text (""); - click_emphasis_path_entry.set_text (""); - session_raid_entry.set_text (""); - - click_path_entry.set_sensitive (false); - click_emphasis_path_entry.set_sensitive (false); - session_raid_entry.set_sensitive (false); - - short_xfade_slider.set_sensitive (false); - smpte_offset_negative_button.set_sensitive (false); - - smpte_offset_clock.set_session (s); - - if ((session = s) == 0) { - return; + int const n = p->table.property_n_rows(); + int m = n + 1; + if (!_note.empty ()) { + ++m; } - click_path_entry.set_sensitive (true); - click_emphasis_path_entry.set_sensitive (true); - session_raid_entry.set_sensitive (true); - short_xfade_slider.set_sensitive (true); - smpte_offset_negative_button.set_sensitive (true); - - smpte_offset_clock.set_session (s); - smpte_offset_clock.set (s->smpte_offset (), true); - - smpte_offset_negative_button.set_active (session->smpte_offset_negative()); - - /* set up port assignments */ - - std::map >::iterator res; - - if (session->mtc_port()) { - if ((res = port_toggle_buttons.find (session->mtc_port())) != port_toggle_buttons.end()) { - (*res).second[MtcIndex]->set_active (true); - } - } + p->table.resize (m, 3); + p->table.attach (*wa, 1, 2, n, n + 1, FILL); + p->table.attach (*wb, 2, 3, n, n + 1, FILL | EXPAND); - if (session->mmc_port ()) { - if ((res = port_toggle_buttons.find (session->mmc_port())) != port_toggle_buttons.end()) { - (*res).second[MmcIndex]->set_active (true); - } - } + maybe_add_note (p, n + 1); +} - if (session->midi_port()) { - if ((res = port_toggle_buttons.find (session->midi_port())) != port_toggle_buttons.end()) { - (*res).second[MidiIndex]->set_active (true); - } +void +OptionEditorComponent::maybe_add_note (OptionEditorPage* p, int n) +{ + if (!_note.empty ()) { + Gtk::Label* l = manage (new Gtk::Label (string_compose (X_("%1"), _note))); + l->set_use_markup (true); + p->table.attach (*l, 1, 3, n, n + 1, FILL | EXPAND); } +} - setup_click_editor (); - connect_audition_editor (); - - short_xfade_adjustment.set_value ((Crossfade::short_xfade_length() / (float) session->frame_rate()) * 1000.0); - - add_session_paths (); +void +OptionEditorComponent::set_note (string const & n) +{ + _note = n; } -OptionEditor::~OptionEditor () +OptionEditorHeading::OptionEditorHeading (string const & h) { + std::stringstream s; + s << "" << h << ""; + _label = manage (left_aligned_label (s.str())); + _label->set_use_markup (true); } void -OptionEditor::setup_path_options() +OptionEditorHeading::add_to_page (OptionEditorPage* p) { - Gtk::Label* label; - - path_table.set_homogeneous (false); - path_table.set_border_width (12); - path_table.set_row_spacings (5); - - session_raid_entry.set_name ("OptionsEntry"); - - session_raid_entry.signal_activate().connect (mem_fun(*this, &OptionEditor::raid_path_changed)); + int const n = p->table.property_n_rows(); + p->table.resize (n + 2, 3); - label = manage(new Label(_("session RAID path"))); - label->set_name ("OptionsLabel"); - path_table.attach (*label, 0, 1, 0, 1, FILL|EXPAND, FILL); - path_table.attach (session_raid_entry, 1, 3, 0, 1, Gtk::FILL|Gtk::EXPAND, FILL); - - path_table.show_all(); + p->table.attach (*manage (new Label ("")), 0, 3, n, n + 1, FILL | EXPAND); + p->table.attach (*_label, 0, 3, n + 1, n + 2, FILL | EXPAND); } void -OptionEditor::add_session_paths () +OptionEditorBox::add_to_page (OptionEditorPage* p) { - click_path_entry.set_sensitive (true); - click_emphasis_path_entry.set_sensitive (true); - session_raid_entry.set_sensitive (true); + add_widget_to_page (p, _box); +} - if (Config->get_click_sound().empty()) { - click_path_entry.set_text (_("internal")); - } else { - click_path_entry.set_text (Config->get_click_sound()); +RcActionButton::RcActionButton (std::string const & t, const Glib::SignalProxy0< void >::SlotType & slot, std::string const & l) + : _label (NULL) +{ + _button = manage (new Button (t)); + _button->signal_clicked().connect (slot); + if (!l.empty ()) { + _label = manage (right_aligned_label (l)); } +} - if (Config->get_click_emphasis_sound().empty()) { - click_emphasis_path_entry.set_text (_("internal")); +void +RcActionButton::add_to_page (OptionEditorPage *p) +{ + int const n = p->table.property_n_rows(); + int m = n + 1; + p->table.resize (m, 3); + if (_label) { + p->table.attach (*_label, 1, 2, n, n + 1, FILL | EXPAND); + p->table.attach (*_button, 2, 3, n, n + 1, FILL | EXPAND); } else { - click_emphasis_path_entry.set_text (Config->get_click_emphasis_sound()); + p->table.attach (*_button, 1, 3, n, n + 1, FILL | EXPAND); } +} - session_raid_entry.set_text(session->raid_path()); +RcConfigDisplay::RcConfigDisplay (string const & i, string const & n, sigc::slot g, char s) + : _get (g) + , _id (i) + , _sep (s) +{ + _label = manage (right_aligned_label (n)); + _info = manage (new Label); + _info-> set_line_wrap (true); + set_state_from_config (); } void -OptionEditor::setup_fade_options () +RcConfigDisplay::set_state_from_config () { - Gtk::HBox* hbox; - - Label* label = manage (new Label (_("Short crossfade length (msecs)"))); - label->set_name ("OptionsLabel"); - - hbox = manage (new HBox); - hbox->set_border_width (5); - hbox->set_spacing (10); - hbox->pack_start (*label, false, false); - hbox->pack_start (short_xfade_slider, true, true); - fade_packer.pack_start (*hbox, false, false); - - short_xfade_adjustment.signal_value_changed().connect (mem_fun(*this, &OptionEditor::short_xfade_adjustment_changed)); - - label = manage (new Label (_("Destructive crossfade length (msecs)"))); - label->set_name ("OptionsLabel"); - - hbox = manage (new HBox); - hbox->set_border_width (5); - hbox->set_spacing (10); - hbox->pack_start (*label, false, false); - hbox->pack_start (destructo_xfade_slider, true, true); - fade_packer.pack_start (*hbox, false, false); - - destructo_xfade_adjustment.signal_value_changed().connect (mem_fun(*this, &OptionEditor::destructo_xfade_adjustment_changed)); - - short_xfade_slider.set_update_policy (UPDATE_DISCONTINUOUS); - destructo_xfade_slider.set_update_policy (UPDATE_DISCONTINUOUS); - - destructo_xfade_adjustment.set_value (Config->get_destructive_xfade_msecs()); - - fade_packer.show_all (); + string p = _get(); + if (_sep) { + std::replace (p.begin(), p.end(), _sep, '\n'); + } + _info->set_text (p); } void -OptionEditor::short_xfade_adjustment_changed () +RcConfigDisplay::parameter_changed (std::string const & p) { - if (session) { - float val = short_xfade_adjustment.get_value(); - - /* val is in msecs */ - - Crossfade::set_short_xfade_length ((nframes_t) floor (session->frame_rate() * (val / 1000.0))); + if (p == _id) { + set_state_from_config (); } } void -OptionEditor::destructo_xfade_adjustment_changed () +RcConfigDisplay::add_to_page (OptionEditorPage *p) { - float val = destructo_xfade_adjustment.get_value(); - - /* val is in msecs */ + int const n = p->table.property_n_rows(); + int m = n + 1; + p->table.resize (m, 3); + p->table.attach (*_label, 1, 2, n, n + 1, FILL | EXPAND); + p->table.attach (*_info, 2, 3, n, n + 1, FILL | EXPAND); +} - - Config->set_destructive_xfade_msecs ((uint32_t) floor (val)); - if (session) { - SndFileSource::setup_standard_crossfades (session->frame_rate()); - } +BoolOption::BoolOption (string const & i, string const & n, sigc::slot g, sigc::slot s) + : Option (i, n), + _get (g), + _set (s) +{ + _button = manage (new CheckButton); + _label = manage (new Label); + _label->set_markup (n); + _button->add (*_label); + _button->set_active (_get ()); + _button->signal_toggled().connect (sigc::mem_fun (*this, &BoolOption::toggled)); } void -OptionEditor::setup_sync_options () +BoolOption::add_to_page (OptionEditorPage* p) { - HBox* hbox; - vector dumb; - - smpte_offset_clock.set_mode (AudioClock::SMPTE); - smpte_offset_clock.ValueChanged.connect (mem_fun(*this, &OptionEditor::smpte_offset_chosen)); - - smpte_offset_negative_button.set_name ("OptionEditorToggleButton"); - - smpte_offset_negative_button.unset_flags (Gtk::CAN_FOCUS); - - Label *smpte_offset_label = manage (new Label (_("SMPTE Offset"))); - smpte_offset_label->set_name("OptionsLabel"); - - hbox = manage (new HBox); - hbox->set_border_width (5); - hbox->set_spacing (10); - hbox->pack_start (*smpte_offset_label, false, false); - hbox->pack_start (smpte_offset_clock, false, false); - hbox->pack_start (smpte_offset_negative_button, false, false); - - sync_packer.pack_start (*hbox, false, false); - - smpte_offset_negative_button.signal_clicked().connect (mem_fun(*this, &OptionEditor::smpte_offset_negative_clicked)); + add_widget_to_page (p, _button); } void -OptionEditor::smpte_offset_negative_clicked () +BoolOption::set_state_from_config () { - if (session) { - session->set_smpte_offset_negative (smpte_offset_negative_button.get_active()); - } + _button->set_active (_get ()); } void -OptionEditor::smpte_offset_chosen() +BoolOption::toggled () { - if (session) { - nframes_t frames = smpte_offset_clock.current_duration(); - session->set_smpte_offset (frames); + if (!_set (_button->get_active ())) { + _button->set_active (_get ()); } } +RouteDisplayBoolOption::RouteDisplayBoolOption (string const & i, string const & n, sigc::slot g, sigc::slot s) + : BoolOption (i, n, g, s) +{ +} void -OptionEditor::setup_midi_options () +RouteDisplayBoolOption::toggled () { - HBox* hbox; - MIDI::Manager::PortMap::const_iterator i; - const MIDI::Manager::PortMap& ports = MIDI::Manager::instance()->get_midi_ports(); - int n; - ToggleButton* tb; - RadioButton* rb; - - Gtk::Table* table = manage (new Table (ports.size() + 4, 10)); - - table->set_row_spacings (6); - table->set_col_spacings (10); - - table->attach (*(manage (new Label (_("Port")))), 0, 1, 0, 1); - table->attach (*(manage (new Label (_("Offline")))), 1, 2, 0, 1); - table->attach (*(manage (new Label (_("Trace\nInput")))), 2, 3, 0, 1); - table->attach (*(manage (new Label (_("Trace\nOutput")))), 3, 4, 0, 1); - table->attach (*(manage (new Label (_("MTC")))), 4, 5, 0, 1); - table->attach (*(manage (new Label (_("MMC")))), 6, 7, 0, 1); - table->attach (*(manage (new Label (_("MIDI Parameter\nControl")))), 8, 9, 0, 1); - - table->attach (*(manage (new HSeparator())), 0, 9, 1, 2); - table->attach (*(manage (new VSeparator())), 5, 6, 0, 8); - table->attach (*(manage (new VSeparator())), 7, 8, 0, 8); - - table->attach (*(manage (new Label (_("MMC Device ID")))), 9, 10, 0, 1); - table->attach (mmc_device_id_spinner, 9, 10, 1, 2); - - mmc_device_id_adjustment.signal_value_changed().connect (mem_fun (*this, &OptionEditor::mmc_device_id_adjusted)); - - for (n = 0, i = ports.begin(); i != ports.end(); ++n, ++i) { - - pair > newpair; - - newpair.first = i->second; - - table->attach (*(manage (new Label (i->first))), 0, 1, n+2, n+3,FILL|EXPAND, FILL ); - tb = manage (new ToggleButton (_("online"))); - tb->set_name ("OptionEditorToggleButton"); + DisplaySuspender ds; + BoolOption::toggled (); +} - /* remember, we have to handle the i18n case where the relative - lengths of the strings in language N is different than in english. - */ +EntryOption::EntryOption (string const & i, string const & n, sigc::slot g, sigc::slot s) + : Option (i, n), + _get (g), + _set (s) +{ + _label = manage (left_aligned_label (n + ":")); + _entry = manage (new Entry); + _entry->signal_activate().connect (sigc::mem_fun (*this, &EntryOption::activated)); + _entry->signal_focus_out_event().connect (sigc::mem_fun (*this, &EntryOption::focus_out)); + _entry->signal_insert_text().connect (sigc::mem_fun (*this, &EntryOption::filter_text)); +} - if (strlen (_("offline")) > strlen (_("online"))) { - set_size_request_to_display_given_text (*tb, _("offline"), 15, 12); - } else { - set_size_request_to_display_given_text (*tb, _("online"), 15, 12); - } +void +EntryOption::add_to_page (OptionEditorPage* p) +{ + add_widgets_to_page (p, _label, _entry); +} - tb->set_active (!(*i).second->input()->offline()); - tb->signal_toggled().connect (bind (mem_fun(*this, &OptionEditor::port_online_toggled), (*i).second, tb)); - (*i).second->input()->OfflineStatusChanged.connect (bind (mem_fun(*this, &OptionEditor::map_port_online), (*i).second, tb)); - table->attach (*tb, 1, 2, n+2, n+3, FILL|EXPAND, FILL); - - tb = manage (new ToggleButton ()); - tb->set_name ("OptionEditorToggleButton"); - tb->signal_toggled().connect (bind (mem_fun(*this, &OptionEditor::port_trace_in_toggled), (*i).second, tb)); - tb->set_size_request (10, 10); - table->attach (*tb, 2, 3, n+2, n+3, FILL|EXPAND, FILL); - - tb = manage (new ToggleButton ()); - tb->set_name ("OptionEditorToggleButton"); - tb->signal_toggled().connect (bind (mem_fun(*this, &OptionEditor::port_trace_out_toggled), (*i).second, tb)); - tb->set_size_request (10, 10); - table->attach (*tb, 3, 4, n+2, n+3, FILL|EXPAND, FILL); - - rb = manage (new RadioButton ()); - newpair.second.push_back (rb); - rb->set_name ("OptionEditorToggleButton"); - if (n == 0) { - mtc_button_group = rb->get_group(); - } else { - rb->set_group (mtc_button_group); +void +EntryOption::set_state_from_config () +{ + _entry->set_text (_get ()); +} - } - table->attach (*rb, 4, 5, n+2, n+3, FILL|EXPAND, FILL); - rb->signal_toggled().connect (bind (mem_fun(*this, &OptionEditor::mtc_port_chosen), (*i).second, rb)); +void +EntryOption::set_sensitive (bool s) +{ + _entry->set_sensitive (s); +} - if (Config->get_mtc_port_name() == i->first) { - rb->set_active (true); - } - - rb = manage (new RadioButton ()); - newpair.second.push_back (rb); - rb->set_name ("OptionEditorToggleButton"); - if (n == 0) { - mmc_button_group = rb->get_group(); - } else { - rb->set_group (mmc_button_group); - } - table->attach (*rb, 6, 7, n+2, n+3, FILL|EXPAND, FILL); - rb->signal_toggled().connect (bind (mem_fun(*this, &OptionEditor::mmc_port_chosen), (*i).second, rb)); +void +EntryOption::filter_text (const Glib::ustring&, int*) +{ + std::string text = _entry->get_text (); + for (size_t i = 0; i < _invalid.length(); ++i) { + text.erase (std::remove(text.begin(), text.end(), _invalid.at(i)), text.end()); + } + if (text != _entry->get_text ()) { + _entry->set_text (text); + } +} - if (Config->get_mmc_port_name() == i->first) { - rb->set_active (true); - } +void +EntryOption::activated () +{ + _set (_entry->get_text ()); +} - rb = manage (new RadioButton ()); - newpair.second.push_back (rb); - rb->set_name ("OptionEditorToggleButton"); - if (n == 0) { - midi_button_group = rb->get_group(); - } else { - rb->set_group (midi_button_group); - } - table->attach (*rb, 8, 9, n+2, n+3, FILL|EXPAND, FILL); - rb->signal_toggled().connect (bind (mem_fun(*this, &OptionEditor::midi_port_chosen), (*i).second, rb)); +bool +EntryOption::focus_out (GdkEventFocus*) +{ + _set (_entry->get_text ()); + return true; +} - if (Config->get_midi_port_name() == i->first) { - rb->set_active (true); - } - - port_toggle_buttons.insert (newpair); - } +/** Construct a BoolComboOption. + * @param i id + * @param n User-visible name. + * @param t Text to give for the variable being true. + * @param f Text to give for the variable being false. + * @param g Slot to get the variable's value. + * @param s Slot to set the variable's value. + */ +BoolComboOption::BoolComboOption ( + string const & i, string const & n, string const & t, string const & f, + sigc::slot g, sigc::slot s + ) + : Option (i, n) + , _get (g) + , _set (s) +{ + _label = manage (new Label (n + ":")); + _label->set_alignment (0, 0.5); + _combo = manage (new ComboBoxText); - table->show_all (); + /* option 0 is the false option */ + _combo->append_text (f); + /* and option 1 is the true */ + _combo->append_text (t); - hbox = manage (new HBox); - hbox->set_border_width (6); - hbox->pack_start (*table, true, false); - midi_packer.pack_start (*hbox, false, false); + _combo->signal_changed().connect (sigc::mem_fun (*this, &BoolComboOption::changed)); } void -OptionEditor::mtc_port_chosen (MIDI::Port *port, Gtk::RadioButton* rb) +BoolComboOption::set_state_from_config () { - if (session) { - if (rb->get_active()) { - if (port) { - session->set_mtc_port (port->name()); - Config->set_mtc_port_name (port->name()); - } else { - session->set_mtc_port (""); - } - rb->set_active (true); - } - } + _combo->set_active (_get() ? 1 : 0); } void -OptionEditor::mmc_port_chosen (MIDI::Port* port, Gtk::RadioButton* rb) +BoolComboOption::add_to_page (OptionEditorPage* p) { - if (session) { - if (rb->get_active()) { - if (port) { - session->set_mmc_port (port->name()); - Config->set_mtc_port_name (port->name()); - } else { - session->set_mmc_port (""); - } - rb->set_active (true); - } - } + add_widgets_to_page (p, _label, _combo); } void -OptionEditor::midi_port_chosen (MIDI::Port* port, Gtk::RadioButton* rb) +BoolComboOption::changed () { - if (session) { - if (rb->get_active()) { - if (port) { - session->set_midi_port (port->name()); - Config->set_midi_port_name (port->name()); - } else { - session->set_midi_port (""); - } - rb->set_active (true); - } - } + _set (_combo->get_active_row_number () == 0 ? false : true); } void -OptionEditor::port_online_toggled (MIDI::Port* port, ToggleButton* tb) +BoolComboOption::set_sensitive (bool yn) { - bool wanted = tb->get_active(); - - if (wanted != port->input()->offline()) { - port->input()->set_offline (wanted); - } + _combo->set_sensitive (yn); } -void -OptionEditor::map_port_online (MIDI::Port* port, ToggleButton* tb) + + +FaderOption::FaderOption (string const & i, string const & n, sigc::slot g, sigc::slot s) + : Option (i, n) + , _db_adjustment (gain_to_slider_position_with_max (1.0, Config->get_max_gain()), 0, 1, 0.01, 0.1) + , _get (g) + , _set (s) { - bool bstate = tb->get_active (); + _db_slider = manage (new HSliderController (&_db_adjustment, boost::shared_ptr(), 115, 18)); - if (bstate != port->input()->offline()) { - if (port->input()->offline()) { - tb->set_label (_("offline")); - tb->set_active (false); - } else { - tb->set_label (_("online")); - tb->set_active (true); - } - } + _label.set_text (n + ":"); + _label.set_alignment (0, 0.5); + _label.set_name (X_("OptionsLabel")); + + _fader_centering_box.pack_start (*_db_slider, true, false); + + _box.set_spacing (4); + _box.set_homogeneous (false); + _box.pack_start (_fader_centering_box, false, false); + _box.pack_start (_db_display, false, false); + _box.show_all (); + + set_size_request_to_display_given_text (_db_display, "-99.00", 12, 12); + + _db_adjustment.signal_value_changed().connect (sigc::mem_fun (*this, &FaderOption::db_changed)); + _db_display.signal_activate().connect (sigc::mem_fun (*this, &FaderOption::on_activate)); + _db_display.signal_key_press_event().connect (sigc::mem_fun (*this, &FaderOption::on_key_press), false); } void -OptionEditor::mmc_device_id_adjusted () +FaderOption::set_state_from_config () { - uint8_t id = (uint8_t) mmc_device_id_spinner.get_value(); + gain_t const val = _get (); + _db_adjustment.set_value (gain_to_slider_position_with_max (val, Config->get_max_gain ())); - if (id != Config->get_mmc_device_id()) { - Config->set_mmc_device_id (id); + char buf[16]; + + if (val == 0.0) { + snprintf (buf, sizeof (buf), "-inf"); + } else { + snprintf (buf, sizeof (buf), "%.2f", accurate_coefficient_to_dB (val)); } + + _db_display.set_text (buf); } void -OptionEditor::port_trace_in_toggled (MIDI::Port* port, ToggleButton* tb) +FaderOption::db_changed () { - bool trace = tb->get_active(); - - if (port->input()->tracing() != trace) { - port->input()->trace (trace, &cerr, string (port->name()) + string (" input: ")); - } + _set (slider_position_to_gain_with_max (_db_adjustment.get_value (), Config->get_max_gain())); } void -OptionEditor::port_trace_out_toggled (MIDI::Port* port, ToggleButton* tb) +FaderOption::on_activate () { - bool trace = tb->get_active(); + float db_val = atof (_db_display.get_text ().c_str ()); + gain_t coeff_val = dB_to_coefficient (db_val); + + _db_adjustment.set_value (gain_to_slider_position_with_max (coeff_val, Config->get_max_gain ())); +} - if (port->output()->tracing() != trace) { - port->output()->trace (trace, &cerr, string (port->name()) + string (" output: ")); +bool +FaderOption::on_key_press (GdkEventKey* ev) +{ + if (ARDOUR_UI_UTILS::key_is_legal_for_numeric_entry (ev->keyval)) { + /* drop through to normal handling */ + return false; } + /* illegal key for gain entry */ + return true; } void -OptionEditor::save () +FaderOption::add_to_page (OptionEditorPage* p) { - /* XXX a bit odd that we save the entire session state here */ - - ui.save_state (""); + add_widgets_to_page (p, &_label, &_box); } -gint -OptionEditor::wm_close (GdkEventAny *ev) +ClockOption::ClockOption (string const & i, string const & n, sigc::slot g, sigc::slot s) + : Option (i, n) + , _clock (X_("timecode-offset"), true, X_(""), true, false, true, false) + , _get (g) + , _set (s) { - save (); - hide (); - return TRUE; + _label.set_text (n + ":"); + _label.set_alignment (0, 0.5); + _label.set_name (X_("OptionsLabel")); + _clock.ValueChanged.connect (sigc::mem_fun (*this, &ClockOption::save_clock_time)); } void -OptionEditor::raid_path_changed () +ClockOption::set_state_from_config () { - if (session) { - Config->set_raid_path (session_raid_entry.get_text()); + Timecode::Time TC; + framepos_t when; + if (!Timecode::parse_timecode_format(_get(), TC)) { + _clock.set (0, true); } + TC.rate = _session->frames_per_timecode_frame(); + TC.drop = _session->timecode_drop_frames(); + _session->timecode_to_sample(TC, when, false, false); + if (TC.negative) { when=-when; } + _clock.set (when, true); } void -OptionEditor::click_browse_clicked () +ClockOption::save_clock_time () { - SoundFileChooser sfdb (_("Choose Click"), session); - - int result = sfdb.run (); - - if (result == Gtk::RESPONSE_OK) { - click_chosen(sfdb.get_filename()); - } + Timecode::Time TC; + _session->sample_to_timecode(_clock.current_time(), TC, false, false); + _set (Timecode::timecode_format_time(TC)); } void -OptionEditor::click_chosen (const string & path) +ClockOption::add_to_page (OptionEditorPage* p) { - click_path_entry.set_text (path); - click_sound_changed (); + add_widgets_to_page (p, &_label, &_clock); } void -OptionEditor::click_emphasis_browse_clicked () +ClockOption::set_session (Session* s) { - SoundFileChooser sfdb (_("Choose Click Emphasis"), session); - - int result = sfdb.run (); - - if (result == Gtk::RESPONSE_OK) { - click_emphasis_chosen (sfdb.get_filename()); - } + _session = s; + _clock.set_session (s); } -void -OptionEditor::click_emphasis_chosen (const string & path) -{ - click_emphasis_path_entry.set_text (path); - click_emphasis_sound_changed (); +OptionEditorPage::OptionEditorPage (Gtk::Notebook& n, std::string const & t) + : table (1, 3) +{ + table.set_spacings (4); + table.set_col_spacing (0, 32); + box.pack_start (table, false, false); + box.set_border_width (4); + n.append_page (box, t); } -void -OptionEditor::click_sound_changed () +/** Construct an OptionEditor. + * @param o Configuration to edit. + * @param t Title for the dialog. + */ +OptionEditor::OptionEditor (PBD::Configuration* c) + : _config (c) + , option_tree (TreeStore::create (option_columns)) + , option_treeview (option_tree) { - if (session) { - string path = click_path_entry.get_text(); + using namespace Notebook_Helpers; - if (path == Config->get_click_sound()) { - return; - } + _notebook.set_show_tabs (false); + _notebook.set_show_border (true); + _notebook.set_name ("OptionsNotebook"); - strip_whitespace_edges (path); + option_treeview.append_column ("", option_columns.name); + option_treeview.set_enable_search(true); + option_treeview.set_search_column(0); + option_treeview.set_name ("OptionsTreeView"); + option_treeview.set_headers_visible (false); - if (path == _("internal")) { - Config->set_click_sound (""); - } else { - Config->set_click_sound (path); + option_treeview.get_selection()->set_mode (Gtk::SELECTION_SINGLE); + option_treeview.get_selection()->signal_changed().connect (sigc::mem_fun (*this, &OptionEditor::treeview_row_selected)); + + /* Watch out for changes to parameters */ + _config->ParameterChanged.connect (config_connection, invalidator (*this), boost::bind (&OptionEditor::parameter_changed, this, _1), gui_context()); +} + +OptionEditor::~OptionEditor () +{ + for (std::map::iterator i = _pages.begin(); i != _pages.end(); ++i) { + for (std::list::iterator j = i->second->components.begin(); j != i->second->components.end(); ++j) { + delete *j; } + delete i->second; } } +/** Called when a configuration parameter has been changed. + * @param p Parameter name. + */ void -OptionEditor::click_emphasis_sound_changed () +OptionEditor::parameter_changed (std::string const & p) { - if (session) { - string path = click_emphasis_path_entry.get_text(); + ENSURE_GUI_THREAD (*this, &OptionEditor::parameter_changed, p) - if (path == Config->get_click_emphasis_sound()) { - return; - } - - strip_whitespace_edges (path); - - if (path == _("internal")) { - Config->set_click_emphasis_sound (""); - } else { - Config->set_click_emphasis_sound (path); + for (std::map::iterator i = _pages.begin(); i != _pages.end(); ++i) { + for (std::list::iterator j = i->second->components.begin(); j != i->second->components.end(); ++j) { + (*j)->parameter_changed (p); } } } void -OptionEditor::clear_click_editor () +OptionEditor::treeview_row_selected () { - if (click_io_selector) { - click_packer.remove (*click_io_selector); - click_packer.remove (*click_gpm); - delete click_io_selector; - delete click_gpm; - click_io_selector = 0; - click_gpm = 0; + Glib::RefPtr selection = option_treeview.get_selection(); + TreeModel::iterator iter = selection->get_selected(); + if(iter) { + TreeModel::Row row = *iter; + Gtk::Widget* w = row[option_columns.widget]; + if (w) { + _notebook.set_current_page (_notebook.page_num (*w)); + } } } void -OptionEditor::setup_click_editor () +OptionEditor::add_path_to_treeview (std::string const & pn, Gtk::Widget& widget) { - Label* label; - HBox* hpacker = manage (new HBox); + option_treeview.set_model (Glib::RefPtr()); - click_path_entry.set_sensitive (true); - click_emphasis_path_entry.set_sensitive (true); + if (pn.find ('/') == std::string::npos) { + /* new top level item in tree */ - click_path_entry.set_name ("OptionsEntry"); - click_emphasis_path_entry.set_name ("OptionsEntry"); - - click_path_entry.signal_activate().connect (mem_fun(*this, &OptionEditor::click_sound_changed)); - click_emphasis_path_entry.signal_activate().connect (mem_fun(*this, &OptionEditor::click_emphasis_sound_changed)); + TreeModel::iterator new_row = option_tree->append (); + TreeModel::Row row = *new_row; + row[option_columns.name] = pn; + row[option_columns.widget] = &widget; - click_path_entry.signal_focus_out_event().connect (bind (mem_fun(*this, &OptionEditor::focus_out_event_handler), &OptionEditor::click_sound_changed)); - click_emphasis_path_entry.signal_focus_out_event().connect (bind (mem_fun(*this, &OptionEditor::focus_out_event_handler), &OptionEditor::click_emphasis_sound_changed)); + } else { - click_browse_button.set_name ("EditorGTKButton"); - click_emphasis_browse_button.set_name ("EditorGTKButton"); - click_browse_button.signal_clicked().connect (mem_fun(*this, &OptionEditor::click_browse_clicked)); - click_emphasis_browse_button.signal_clicked().connect (mem_fun(*this, &OptionEditor::click_emphasis_browse_clicked)); + /* find parent */ - click_packer.set_border_width (12); - click_packer.set_spacing (5); + /* split page name, which is actually a path, into each + * component + */ - click_io_selector = new IOSelector (*session, session->click_io(), false); - click_gpm = new GainMeter (session->click_io(), *session); + std::vector components; + split (pn, components, '/'); - click_table.set_col_spacings (10); - - label = manage(new Label(_("Click audio file"))); - label->set_name ("OptionsLabel"); - click_table.attach (*label, 0, 1, 0, 1, FILL|EXPAND, FILL); - click_table.attach (click_path_entry, 1, 2, 0, 1, Gtk::FILL|Gtk::EXPAND, FILL); - click_table.attach (click_browse_button, 2, 3, 0, 1, FILL|EXPAND, FILL); - - label = manage(new Label(_("Click emphasis audiofile"))); - label->set_name ("OptionsLabel"); - click_table.attach (*label, 0, 1, 1, 2, FILL|EXPAND, FILL); - click_table.attach (click_emphasis_path_entry, 1, 2, 1, 2, Gtk::FILL|Gtk::EXPAND, FILL); - click_table.attach (click_emphasis_browse_button, 2, 3, 1, 2, FILL|EXPAND, FILL); + /* start with top level children */ - hpacker->set_spacing (10); - hpacker->pack_start (*click_io_selector, false, false); - hpacker->pack_start (*click_gpm, false, false); + typedef Gtk::TreeModel::Children type_children; //minimise code length. + type_children children = option_tree->children(); - click_packer.pack_start (click_table, false, false); - click_packer.pack_start (*hpacker, false, false); + /* foreach path component ... */ - click_packer.show_all (); -} + for (std::vector::const_iterator s = components.begin(); s != components.end(); ) { -void -OptionEditor::clear_auditioner_editor () -{ - if (auditioner_io_selector) { - audition_hpacker.remove (*auditioner_io_selector); - audition_hpacker.remove (*auditioner_gpm); - delete auditioner_io_selector; - delete auditioner_gpm; - auditioner_io_selector = 0; - auditioner_gpm = 0; - } -} + bool component_found = false; -void -OptionEditor::setup_auditioner_editor () -{ - audition_packer.set_border_width (12); - audition_packer.set_spacing (5); - audition_hpacker.set_spacing (10); + type_children::iterator iter; - audition_label.set_name ("OptionEditorAuditionerLabel"); - audition_label.set_text (_("The auditioner is a dedicated mixer strip used\n" - "for listening to specific regions outside the context\n" - "of the overall mix. It can be connected just like any\n" - "other mixer strip.")); - - audition_packer.pack_start (audition_label, false, false, 10); - audition_packer.pack_start (audition_hpacker, false, false); -} + /* look through the children at this level */ -void -OptionEditor::connect_audition_editor () -{ - auditioner_io_selector = new IOSelector (*session, session->the_auditioner(), false); - auditioner_gpm = new GainMeter (session->the_auditioner(), *session); + for (iter = children.begin(); iter != children.end(); ++iter) { + Gtk::TreeModel::Row row = *iter; - audition_hpacker.pack_start (*auditioner_io_selector, false, false); - audition_hpacker.pack_start (*auditioner_gpm, false, false); + std::string row_name = row[option_columns.name]; + if (row_name == (*s)) { - auditioner_io_selector->show_all (); - auditioner_gpm->show_all (); -} + /* found it */ -bool -OptionEditor::focus_out_event_handler (GdkEventFocus* ev, void (OptionEditor::*pmf)()) -{ - (this->*pmf)(); - return false; -} + component_found = true; -static const struct { - const char *name; - guint modifier; -} modifiers[] = { - { "Shift", GDK_SHIFT_MASK }, - { "Control", GDK_CONTROL_MASK }, - { "Alt (Mod1)", GDK_MOD1_MASK }, - { "Control-Shift", GDK_CONTROL_MASK|GDK_SHIFT_MASK }, - { "Control-Alt", GDK_CONTROL_MASK|GDK_MOD1_MASK }, - { "Shift-Alt", GDK_SHIFT_MASK|GDK_MOD1_MASK }, - { "Control-Shift-Alt", GDK_CONTROL_MASK|GDK_SHIFT_MASK|GDK_MOD1_MASK }, - { "Mod2", GDK_MOD2_MASK }, - { "Mod3", GDK_MOD3_MASK }, - { "Mod4", GDK_MOD4_MASK }, - { "Mod5", GDK_MOD5_MASK }, - { 0, 0 } -}; + /* reset children to point to + * the children of this row + */ -void -OptionEditor::setup_keyboard_options () -{ - vector dumb; - Label* label; + children = row.children(); + break; + } + } - keyboard_mouse_table.set_border_width (12); - keyboard_mouse_table.set_row_spacings (5); - keyboard_mouse_table.set_col_spacings (5); + if (!component_found) { - /* internationalize and prepare for use with combos */ + /* missing component. If it is the last + * one, append a real page. Otherwise + * just put an entry in the tree model + * since it is a navigational/sorting + * component. + */ - for (int i = 0; modifiers[i].name; ++i) { - dumb.push_back (_(modifiers[i].name)); - } + TreeModel::iterator new_row = option_tree->append (children); + TreeModel::Row row = *new_row; + row[option_columns.name] = *s; - set_popdown_strings (edit_modifier_combo, dumb); - edit_modifier_combo.signal_changed().connect (mem_fun(*this, &OptionEditor::edit_modifier_chosen)); + ++s; - for (int x = 0; modifiers[x].name; ++x) { - if (modifiers[x].modifier == Keyboard::edit_modifier ()) { - edit_modifier_combo.set_active_text (_(modifiers[x].name)); - break; - } - } + if (s == components.end()) { + row[option_columns.widget] = &widget; + } else { + row[option_columns.widget] = 0; + } - label = manage (new Label (_("Edit using"))); - label->set_name ("OptionsLabel"); - label->set_alignment (1.0, 0.5); - - keyboard_mouse_table.attach (*label, 0, 1, 0, 1, Gtk::FILL|Gtk::EXPAND, FILL); - keyboard_mouse_table.attach (edit_modifier_combo, 1, 2, 0, 1, Gtk::FILL|Gtk::EXPAND, FILL); - - label = manage (new Label (_("+ button"))); - label->set_name ("OptionsLabel"); - - keyboard_mouse_table.attach (*label, 3, 4, 0, 1, Gtk::FILL|Gtk::EXPAND, FILL); - keyboard_mouse_table.attach (edit_button_spin, 4, 5, 0, 1, Gtk::FILL|Gtk::EXPAND, FILL); - - edit_button_spin.set_name ("OptionsEntry"); - edit_button_adjustment.set_value (Keyboard::edit_button()); - edit_button_adjustment.signal_value_changed().connect (mem_fun(*this, &OptionEditor::edit_button_changed)); - - set_popdown_strings (delete_modifier_combo, dumb); - delete_modifier_combo.signal_changed().connect (mem_fun(*this, &OptionEditor::delete_modifier_chosen)); - - for (int x = 0; modifiers[x].name; ++x) { - if (modifiers[x].modifier == Keyboard::delete_modifier ()) { - delete_modifier_combo.set_active_text (_(modifiers[x].name)); - break; - } - } + children = row.children (); - label = manage (new Label (_("Delete using"))); - label->set_name ("OptionsLabel"); - label->set_alignment (1.0, 0.5); - - keyboard_mouse_table.attach (*label, 0, 1, 1, 2, Gtk::FILL|Gtk::EXPAND, FILL); - keyboard_mouse_table.attach (delete_modifier_combo, 1, 2, 1, 2, Gtk::FILL|Gtk::EXPAND, FILL); - - label = manage (new Label (_("+ button"))); - label->set_name ("OptionsLabel"); - - keyboard_mouse_table.attach (*label, 3, 4, 1, 2, Gtk::FILL|Gtk::EXPAND, FILL); - keyboard_mouse_table.attach (delete_button_spin, 4, 5, 1, 2, Gtk::FILL|Gtk::EXPAND, FILL); - - delete_button_spin.set_name ("OptionsEntry"); - delete_button_adjustment.set_value (Keyboard::delete_button()); - delete_button_adjustment.signal_value_changed().connect (mem_fun(*this, &OptionEditor::delete_button_changed)); - - set_popdown_strings (snap_modifier_combo, dumb); - snap_modifier_combo.signal_changed().connect (mem_fun(*this, &OptionEditor::snap_modifier_chosen)); - - for (int x = 0; modifiers[x].name; ++x) { - if (modifiers[x].modifier == (guint) Keyboard::snap_modifier ()) { - snap_modifier_combo.set_active_text (_(modifiers[x].name)); - break; + } else { + + /* component found, just move on to the + * next one. children has already been + * reset appropriately. + */ + + ++s; + } } + } - label = manage (new Label (_("Ignore snap using"))); - label->set_name ("OptionsLabel"); - label->set_alignment (1.0, 0.5); - - keyboard_mouse_table.attach (*label, 0, 1, 2, 3, Gtk::FILL|Gtk::EXPAND, FILL); - keyboard_mouse_table.attach (snap_modifier_combo, 1, 2, 2, 3, Gtk::FILL|Gtk::EXPAND, FILL); + option_treeview.set_model (option_tree); + option_treeview.expand_all (); } +/** Add a component to a given page. + * @param pn Page name (will be created if it doesn't already exist) + * @param o Component. + */ void -OptionEditor::edit_modifier_chosen () +OptionEditor::add_option (std::string const & pn, OptionEditorComponent* o) { - string txt; - - txt = edit_modifier_combo.get_active_text(); + if (_pages.find (pn) == _pages.end()) { + OptionEditorPage* oep = new OptionEditorPage (_notebook, pn); + _pages[pn] = oep; - for (int i = 0; modifiers[i].name; ++i) { - if (txt == _(modifiers[i].name)) { - Keyboard::set_edit_modifier (modifiers[i].modifier); - break; - } + add_path_to_treeview (pn, oep->box); } + + OptionEditorPage* p = _pages[pn]; + p->components.push_back (o); + + o->add_to_page (p); + o->set_state_from_config (); } +/** Add a new page + * @param pn Page name (will be created if it doesn't already exist) + * @param w widget that fills the page + */ void -OptionEditor::delete_modifier_chosen () +OptionEditor::add_page (std::string const & pn, Gtk::Widget& w) { - string txt; - - txt = delete_modifier_combo.get_active_text(); - - for (int i = 0; modifiers[i].name; ++i) { - if (txt == _(modifiers[i].name)) { - Keyboard::set_delete_modifier (modifiers[i].modifier); - break; - } + if (_pages.find (pn) == _pages.end()) { + OptionEditorPage* oep = new OptionEditorPage (_notebook, pn); + _pages[pn] = oep; + add_path_to_treeview (pn, oep->box); } + + OptionEditorPage* p = _pages[pn]; + p->box.pack_start (w, true, true); } void -OptionEditor::snap_modifier_chosen () +OptionEditor::set_current_page (string const & p) { - string txt; - - txt = snap_modifier_combo.get_active_text(); - - for (int i = 0; modifiers[i].name; ++i) { - if (txt == _(modifiers[i].name)) { - Keyboard::set_snap_modifier (modifiers[i].modifier); - break; + int i = 0; + while (i < _notebook.get_n_pages ()) { + if (_notebook.get_tab_label_text (*_notebook.get_nth_page (i)) == p) { + _notebook.set_current_page (i); + return; } + + ++i; } } + +DirectoryOption::DirectoryOption (string const & i, string const & n, sigc::slot g, sigc::slot s) + : Option (i, n) + , _get (g) + , _set (s) +{ + _file_chooser.set_action (Gtk::FILE_CHOOSER_ACTION_SELECT_FOLDER); + _file_chooser.signal_selection_changed().connect (sigc::mem_fun (*this, &DirectoryOption::selection_changed)); +} + + void -OptionEditor::delete_button_changed () +DirectoryOption::set_state_from_config () { - Keyboard::set_delete_button ((guint) delete_button_adjustment.get_value()); + _file_chooser.set_current_folder (poor_mans_glob(_get ())); } void -OptionEditor::edit_button_changed () +DirectoryOption::add_to_page (OptionEditorPage* p) { - Keyboard::set_edit_button ((guint) edit_button_adjustment.get_value()); + Gtk::Label *label = manage (new Label (_name)); + label->set_alignment (0, 0.5); + label->set_name (X_("OptionsLabel")); + add_widgets_to_page (p, label, &_file_chooser); } void -OptionEditor::fixup_combo_size (Gtk::ComboBoxText& combo, vector& strings) +DirectoryOption::selection_changed () { - /* find the widest string */ + _set (poor_mans_glob(_file_chooser.get_filename ())); +} - string::size_type maxlen = 0; - string maxstring; +/*--------------------------*/ - for (vector::iterator i = strings.begin(); i != strings.end(); ++i) { - string::size_type l; +OptionEditorContainer::OptionEditorContainer (PBD::Configuration* c, string const& str) + : OptionEditor (c) +{ + set_border_width (4); + hpacker.pack_start (treeview(), false, false); + hpacker.pack_start (notebook(), true, true); + pack_start (hpacker, true, true); - if ((l = (*i).length()) > maxlen) { - maxlen = l; - maxstring = *i; - } - } + hpacker.show_all (); + show (); +} - /* try to include ascenders and descenders */ +OptionEditorWindow::OptionEditorWindow (PBD::Configuration* c, string const& str) + : OptionEditor (c) + , ArdourWindow (str) +{ + container.set_border_width (4); + hpacker.pack_start (treeview(), false, false); + hpacker.pack_start (notebook(), true, true); - if (maxstring.length() > 2) { - maxstring[0] = 'g'; - maxstring[1] = 'l'; - } + container.pack_start (hpacker, true, true); - const guint32 FUDGE = 10; // Combo's are stupid - they steal space from the entry for the button + hpacker.show_all (); + container.show (); - set_size_request_to_display_given_text (combo, maxstring.c_str(), 10 + FUDGE, 10); + add (container); } -