Fix session-open after selecting new, template, then back
[ardour.git] / gtk2_ardour / midi_time_axis.cc
index 1679039990b1cebb3c2a5774d0bafd15551344bf..ffff6a96981a7b0e2aea69c6a7eaba673c1b0f43 100644 (file)
@@ -25,6 +25,9 @@
 
 #include <sigc++/bind.h>
 
+#include <gtkmm/separator.h>
+#include <gtkmm/stock.h>
+
 #include "pbd/error.h"
 #include "pbd/ffs.h"
 #include "pbd/stl_delete.h"
 #include "pbd/stateful_diff_command.h"
 
 #include "gtkmm2ext/gtk_ui.h"
-#include "gtkmm2ext/selector.h"
-#include "gtkmm2ext/bindable_button.h"
 #include "gtkmm2ext/utils.h"
 
+#include "widgets/tooltips.h"
+
 #include "ardour/event_type_map.h"
 #include "ardour/midi_patch_manager.h"
 #include "ardour/midi_playlist.h"
@@ -61,7 +64,6 @@
 #include "ardour/track.h"
 #include "ardour/types.h"
 
-#include "ardour_button.h"
 #include "automation_line.h"
 #include "automation_time_axis.h"
 #include "editor.h"
 #include "midi_streamview.h"
 #include "midi_region_view.h"
 #include "midi_time_axis.h"
+#include "patch_change_dialog.h"
 #include "piano_roll_header.h"
 #include "playlist_selector.h"
 #include "plugin_selector.h"
 #include "plugin_ui.h"
 #include "point_selection.h"
-#include "prompter.h"
 #include "region_view.h"
 #include "rgb_macros.h"
 #include "selection.h"
 #include "step_editor.h"
-#include "tooltips.h"
 #include "utils.h"
 #include "note_base.h"
 
@@ -93,7 +94,6 @@
 #include "pbd/i18n.h"
 
 using namespace ARDOUR;
-using namespace ARDOUR_UI_UTILS;
 using namespace PBD;
 using namespace Gtk;
 using namespace Gtkmm2ext;
@@ -153,7 +153,7 @@ MidiTimeAxisView::set_route (boost::shared_ptr<Route> rt)
        */
        RouteTimeAxisView::set_route (rt);
 
-       _view->apply_color (gdk_color_to_rgba (color()), StreamView::RegionColor);
+       _view->apply_color (ARDOUR_UI_UTILS::gdk_color_to_rgba (color()), StreamView::RegionColor);
 
        subplugin_menu.set_name ("ArdourContextMenu");
 
@@ -260,8 +260,8 @@ MidiTimeAxisView::set_route (boost::shared_ptr<Route> rt)
                }
        }
 
-       set_tooltip (_midnam_model_selector, _("External MIDI Device"));
-       set_tooltip (_midnam_custom_device_mode_selector, _("External Device Mode"));
+       ArdourWidgets::set_tooltip (_midnam_model_selector, _("External MIDI Device"));
+       ArdourWidgets::set_tooltip (_midnam_custom_device_mode_selector, _("External Device Mode"));
 
        _midi_controls_box.pack_start (_midnam_model_selector, false, false, 2);
        _midi_controls_box.pack_start (_midnam_custom_device_mode_selector, false, false, 2);
@@ -313,7 +313,7 @@ MidiTimeAxisView::set_route (boost::shared_ptr<Route> rt)
                        *i, route_id, has_parameter, parameter);
                if (p && route_id == _route->id () && has_parameter) {
                        const std::string& visible = gui_object_state().get_string (*i, X_("visible"));
-                       create_automation_child (parameter, string_is_affirmative (visible));
+                       create_automation_child (parameter, string_to<bool> (visible));
                }
        }
 }
@@ -559,7 +559,8 @@ MidiTimeAxisView::append_extra_display_menu_items ()
        items.push_back (MenuElem (_("Channel Selector"),
                                   sigc::mem_fun(*this, &MidiTimeAxisView::toggle_channel_selector)));
 
-       items.push_back (MenuElem (_("Select Patch"), *build_patch_menu()));
+       items.push_back (MenuElem (_("Select Patch..."),
+                               sigc::mem_fun(*this, &MidiTimeAxisView::send_patch_change)));
 
        color_mode_menu = build_color_mode_menu();
        if (color_mode_menu) {
@@ -1076,98 +1077,23 @@ MidiTimeAxisView::build_color_mode_menu()
        return mode_menu;
 }
 
-Gtk::Menu*
-MidiTimeAxisView::build_patch_menu()
-{
-       using namespace MIDI::Name;
-       using namespace Menu_Helpers;
-
-       boost::shared_ptr<MasterDeviceNames> device_names = get_device_names();
-       const std::string device_mode = gui_property (X_("midnam-custom-device-mode"));
-
-       Menu* pc_menu = manage (new Menu);
-       MenuList& pc_items = pc_menu->items();
-
-       for (uint32_t chn = 0; chn < 16; ++chn) {
-               boost::shared_ptr<ChannelNameSet> channel_name_set = device_names->channel_name_set_by_channel (device_mode, chn);
-               // see also PatchChange::initialize_popup_menus
-               if (!channel_name_set) {
-                       continue;
-               }
-               const ChannelNameSet::PatchBanks& patch_banks = channel_name_set->patch_banks();
-               if (patch_banks.size () == 0) {
-                       continue;
-               }
-
-               Gtk::Menu& chan_menu = *manage(new Gtk::Menu());
-
-               if (patch_banks.size() > 1) {
-
-                       for (ChannelNameSet::PatchBanks::const_iterator bank = patch_banks.begin();
-                                       bank != patch_banks.end();
-                                       ++bank) {
-                               Glib::RefPtr<Glib::Regex> underscores = Glib::Regex::create("_");
-                               std::string replacement(" ");
-
-                               Gtk::Menu& patch_bank_menu = *manage(new Gtk::Menu());
-
-                               const PatchNameList& patches = (*bank)->patch_name_list();
-                               Gtk::Menu::MenuList& patch_menus = patch_bank_menu.items();
-
-                               for (PatchNameList::const_iterator patch = patches.begin();
-                                               patch != patches.end();
-                                               ++patch) {
-                                       std::string name = underscores->replace((*patch)->name().c_str(), -1, 0, replacement);
-
-                                       patch_menus.push_back(
-                                                       Gtk::Menu_Helpers::MenuElem(
-                                                               name,
-                                                               sigc::bind(
-                                                                       sigc::mem_fun(*this, &MidiTimeAxisView::on_patch_menu_selected),
-                                                                       chn, (*patch)->patch_primary_key())) );
-                               }
-
-
-                               std::string name = underscores->replace((*bank)->name().c_str(), -1, 0, replacement);
-
-                               chan_menu.items().push_back(
-                                               Gtk::Menu_Helpers::MenuElem(
-                                                       name,
-                                                       patch_bank_menu) );
-                       }
-               } else {
-                       /* only one patch bank, so make it the initial menu */
-
-                       const PatchNameList& patches = patch_banks.front()->patch_name_list();
-
-                       for (PatchNameList::const_iterator patch = patches.begin();
-                                       patch != patches.end();
-                                       ++patch) {
-                               std::string name = (*patch)->name();
-                               boost::replace_all (name, "_", " ");
-
-                               chan_menu.items().push_back (
-                                               Gtk::Menu_Helpers::MenuElem (
-                                                       name,
-                                                       sigc::bind (sigc::mem_fun(*this, &MidiTimeAxisView::on_patch_menu_selected),
-                                                               chn, (*patch)->patch_primary_key())));
-                       }
-               }
-
-               pc_items.push_back(
-                               Gtk::Menu_Helpers::MenuElem(
-                                       string_compose (_("Channel %1"), chn + 1),
-                                       chan_menu));
-       }
-       return pc_menu;
-}
-
 void
-MidiTimeAxisView::on_patch_menu_selected (int chn, const MIDI::Name::PatchPrimaryKey& key)
+MidiTimeAxisView::send_patch_change ()
 {
        if (!_route) {
                return;
        }
+
+       Evoral::PatchChange<Evoral::Beats> empty (Evoral::Beats(), 0, 0, 0);
+       PatchChangeDialog d (0, 0, empty, _route->instrument_info(), Gtk::Stock::OK);
+
+       if (d.run() == RESPONSE_CANCEL) {
+               return;
+       }
+       Evoral::PatchChange<Evoral::Beats> p (d.patch ());
+
+       uint8_t chn = p.channel();
+
        boost::shared_ptr<AutomationControl> bank_msb = _route->automation_control(Evoral::Parameter (MidiCCAutomation, chn, MIDI_CTL_MSB_BANK), true);
        boost::shared_ptr<AutomationControl> bank_lsb = _route->automation_control(Evoral::Parameter (MidiCCAutomation, chn, MIDI_CTL_LSB_BANK), true);
        boost::shared_ptr<AutomationControl> program = _route->automation_control(Evoral::Parameter (MidiPgmChangeAutomation, chn), true);
@@ -1175,9 +1101,10 @@ MidiTimeAxisView::on_patch_menu_selected (int chn, const MIDI::Name::PatchPrimar
        if (!bank_msb || ! bank_lsb || !program) {
                return;
        }
-       bank_msb->set_value ((key.bank() >> 7) & 0x7f, Controllable::NoGroup);
-       bank_lsb->set_value (key.bank() & 0x7f, Controllable::NoGroup);
-       program->set_value (key.program(), Controllable::NoGroup);
+
+       bank_msb->set_value (p.bank_msb (), Controllable::NoGroup);
+       bank_lsb->set_value (p.bank_lsb (), Controllable::NoGroup);
+       program->set_value  (p.program () , Controllable::NoGroup);
 }
 
 void
@@ -1239,13 +1166,6 @@ MidiTimeAxisView::set_note_range (MidiStreamView::VisibleNoteRange range, bool a
 void
 MidiTimeAxisView::update_range()
 {
-       MidiGhostRegion* mgr;
-
-       for (list<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-               if ((mgr = dynamic_cast<MidiGhostRegion*>(*i)) != 0) {
-                       mgr->update_range();
-               }
-       }
 }
 
 void
@@ -1404,6 +1324,10 @@ MidiTimeAxisView::create_automation_child (const Evoral::Parameter& param, bool
                }
 
                add_automation_child (param, track, show);
+               if (selected ()) {
+                       reshow_selection (_editor.get_selection().time);
+               }
+
                break;
 
        case PanWidthAutomation:
@@ -1653,9 +1577,11 @@ MidiTimeAxisView::automation_child_menu_item (Evoral::Parameter param)
 }
 
 boost::shared_ptr<MidiRegion>
-MidiTimeAxisView::add_region (framepos_t pos, framecnt_t length, bool commit, const int32_t sub_num)
+MidiTimeAxisView::add_region (framepos_t f, framecnt_t length, bool commit)
 {
        Editor* real_editor = dynamic_cast<Editor*> (&_editor);
+       MusicFrame pos (f, 0);
+
        if (commit) {
                real_editor->begin_reversible_command (Operations::create_region);
        }
@@ -1672,8 +1598,8 @@ MidiTimeAxisView::add_region (framepos_t pos, framecnt_t length, bool commit, co
 
        boost::shared_ptr<Region> region = (RegionFactory::create (src, plist));
        /* sets beat position */
-       region->set_position (pos, sub_num);
-       playlist()->add_region (region, pos, 1.0, false, sub_num);
+       region->set_position (pos.frame, pos.division);
+       playlist()->add_region (region, pos.frame, 1.0, false, pos.division);
        _session->add_command (new StatefulDiffCommand (playlist()));
 
        if (commit) {