X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fedit_note_dialog.cc;h=39a3fa4e789ad354e6f56fe4e20c0ecc5edf0232;hb=cf52d6e4b40111eb04b244ec054055a4ec15dbe0;hp=c4cfe11218703670a8570a9aca502576eaa86f65;hpb=060048685729cce62085c057ad97da3bdf351517;p=ardour.git

diff --git a/gtk2_ardour/edit_note_dialog.cc b/gtk2_ardour/edit_note_dialog.cc
index c4cfe11218..39a3fa4e78 100644
--- a/gtk2_ardour/edit_note_dialog.cc
+++ b/gtk2_ardour/edit_note_dialog.cc
@@ -19,96 +19,141 @@
 
 #include <gtkmm/stock.h>
 #include <gtkmm/table.h>
-#include "ardour/midi_model.h"
+
+#include "gtkmm2ext/utils.h"
+
 #include "edit_note_dialog.h"
-#include "canvas-note-event.h"
 #include "midi_region_view.h"
+#include "note_base.h"
 
+#include "pbd/i18n.h"
+
+using namespace std;
 using namespace Gtk;
+using namespace Gtkmm2ext;
 
 /**
  *    EditNoteDialog constructor.
  *
- *    @param n Note to edit.
+ *    @param n Notes to edit.
  */
 
-EditNoteDialog::EditNoteDialog (MidiRegionView* rv, Gnome::Canvas::CanvasNoteEvent* ev)
+EditNoteDialog::EditNoteDialog (MidiRegionView* rv, set<NoteBase*> n)
 	: ArdourDialog (_("Note"))
 	, _region_view (rv)
-	, _event (ev)
-	, _time_clock (X_("notetime"), true, X_("NoteTimeClock"), true, false)
-	, _length_clock (X_("notelength"), true, X_("NoteLengthClock"), true, false, true)
+	, _events (n)
+	, _channel_all (_("Set selected notes to this channel"))
+	, _pitch_all (_("Set selected notes to this pitch"))
+	, _velocity_all (_("Set selected notes to this velocity"))
+	, _time_clock (X_("notetime"), true, "", true, false)
+	, _time_all (_("Set selected notes to this time"))
+	, _length_clock (X_("notelength"), true, "", true, false, true)
+	, _length_all (_("Set selected notes to this length"))
 {
 	Table* table = manage (new Table (4, 2));
 	table->set_spacings (6);
 
 	int r = 0;
 
-	Label* l = manage (new Label (_("Channel")));
-	l->set_alignment (0, 0.5);
+	Label* l = manage (left_aligned_label (_("Channel")));
 	table->attach (*l, 0, 1, r, r + 1);
 	table->attach (_channel, 1, 2, r, r + 1);
+	table->attach (_channel_all, 2, 3, r, r + 1);
 	++r;
 
 	_channel.set_range (1, 16);
 	_channel.set_increments (1, 2);
-	_channel.set_value (ev->note()->channel () + 1);
+	_channel.set_value ((*_events.begin())->note()->channel () + 1);
 
-	l = manage (new Label (_("Pitch")));
-	l->set_alignment (0, 0.5);
+	l = manage (left_aligned_label (_("Pitch")));
 	table->attach (*l, 0, 1, r, r + 1);
 	table->attach (_pitch, 1, 2, r, r + 1);
+	table->attach (_pitch_all, 2, 3, r, r + 1);
 	++r;
 
 	_pitch.set_range (0, 127);
 	_pitch.set_increments (1, 10);
-	_pitch.set_value (ev->note()->note ());
+	_pitch.set_value ((*_events.begin())->note()->note());
 
-	l = manage (new Label (_("Velocity")));
-	l->set_alignment (0, 0.5);
+	l = manage (left_aligned_label (_("Velocity")));
 	table->attach (*l, 0, 1, r, r + 1);
 	table->attach (_velocity, 1, 2, r, r + 1);
+	table->attach (_velocity_all, 2, 3, r, r + 1);
 	++r;
 
 	_velocity.set_range (0, 127);
 	_velocity.set_increments (1, 10);
-	_velocity.set_value (ev->note()->velocity ());
+	_velocity.set_value ((*_events.begin())->note()->velocity ());
 
-	l = manage (new Label (_("Time")));
-	l->set_alignment (0, 0.5);
+	l = manage (left_aligned_label (_("Time")));
 	table->attach (*l, 0, 1, r, r + 1);
 	table->attach (_time_clock, 1, 2, r, r + 1);
+	table->attach (_time_all, 2, 3, r, r + 1);
 	++r;
 
 	_time_clock.set_session (_region_view->get_time_axis_view().session ());
 	_time_clock.set_mode (AudioClock::BBT);
-	_time_clock.set (_region_view->time_converter().to (ev->note()->time ()), true);
+	_time_clock.set (_region_view->source_relative_time_converter().to
+			 ((*_events.begin())->note()->time()) + (_region_view->region()->position() - _region_view->region()->start()), true);
 
-	l = manage (new Label (_("Length")));
-	l->set_alignment (0, 0.5);
+	l = manage (left_aligned_label (_("Length")));
 	table->attach (*l, 0, 1, r, r + 1);
 	table->attach (_length_clock, 1, 2, r, r + 1);
+	table->attach (_length_all, 2, 3, r, r + 1);
 	++r;
 
 	_length_clock.set_session (_region_view->get_time_axis_view().session ());
 	_length_clock.set_mode (AudioClock::BBT);
-	_length_clock.set (_region_view->time_converter().to (ev->note()->length ()), true);
+	_length_clock.set (_region_view->region_relative_time_converter().to ((*_events.begin())->note()->length ()), true);
+
+	/* Set up `set all notes...' buttons' sensitivity */
+
+	_channel_all.set_sensitive (false);
+	_pitch_all.set_sensitive (false);
+	_velocity_all.set_sensitive (false);
+	_time_all.set_sensitive (false);
+	_length_all.set_sensitive (false);
+
+	int test_channel = (*_events.begin())->note()->channel ();
+	int test_pitch = (*_events.begin())->note()->note ();
+	int test_velocity = (*_events.begin())->note()->velocity ();
+	Evoral::Beats test_time = (*_events.begin())->note()->time ();
+	Evoral::Beats test_length = (*_events.begin())->note()->length ();
+
+	for (set<NoteBase*>::iterator i = _events.begin(); i != _events.end(); ++i) {
+		if ((*i)->note()->channel() != test_channel) {
+			_channel_all.set_sensitive (true);
+		}
+
+		if ((*i)->note()->note() != test_pitch) {
+			_pitch_all.set_sensitive (true);
+		}
+
+		if ((*i)->note()->velocity() != test_velocity) {
+			_velocity_all.set_sensitive (true);
+		}
+
+		if ((*i)->note()->time () != test_time) {
+			_time_all.set_sensitive (true);
+		}
+
+		if ((*i)->note()->length () != test_length) {
+			_length_all.set_sensitive (true);
+		}
+	}
 
 	get_vbox()->pack_start (*table);
 
 	add_button (Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
 	add_button (Gtk::Stock::APPLY, Gtk::RESPONSE_ACCEPT);
 	set_default_response (Gtk::RESPONSE_ACCEPT);
-
-	show_all ();
 }
 
-int
-EditNoteDialog::run ()
+void
+EditNoteDialog::done (int r)
 {
-	int const r = Dialog::run ();
 	if (r != RESPONSE_ACCEPT) {
-		return r;
+                return;
 	}
 
 	/* These calls mean that if a value is entered using the keyboard
@@ -119,36 +164,57 @@ EditNoteDialog::run ()
 	_velocity.update ();
 
 	_region_view->start_note_diff_command (_("edit note"));
-	
+
 	bool had_change = false;
 
-	if (_channel.get_value_as_int() - 1 != _event->note()->channel()) {
-		_region_view->change_note_channel (_event, _channel.get_value_as_int () - 1);
-		had_change = true;
+	if (!_channel_all.get_sensitive() || _channel_all.get_active ()) {
+		for (set<NoteBase*>::iterator i = _events.begin(); i != _events.end(); ++i) {
+			if (_channel.get_value_as_int() - 1 != (*i)->note()->channel()) {
+				_region_view->change_note_channel (*i, _channel.get_value_as_int () - 1);
+				had_change = true;
+			}
+		}
 	}
 
-	if (_pitch.get_value_as_int() != _event->note()->note()) {
-		_region_view->change_note_note (_event, _pitch.get_value_as_int (), false);
-		had_change = true;
+	if (!_pitch_all.get_sensitive() || _pitch_all.get_active ()) {
+		for (set<NoteBase*>::iterator i = _events.begin(); i != _events.end(); ++i) {
+			if (_pitch.get_value_as_int() != (*i)->note()->note()) {
+				_region_view->change_note_note (*i, _pitch.get_value_as_int ());
+				had_change = true;
+			}
+		}
 	}
 
-	if (_velocity.get_value_as_int() != _event->note()->velocity()) {
-		_region_view->change_note_velocity (_event, _velocity.get_value_as_int (), false);
-		had_change = true;
+	if (!_velocity_all.get_sensitive() || _velocity_all.get_active ()) {
+		for (set<NoteBase*>::iterator i = _events.begin(); i != _events.end(); ++i) {
+			if (_velocity.get_value_as_int() != (*i)->note()->velocity()) {
+				_region_view->change_note_velocity (*i, _velocity.get_value_as_int ());
+				had_change = true;
+			}
+		}
 	}
 
-	double const t = _region_view->time_converter().from (_time_clock.current_time ());
-	
-	if (t != _event->note()->time()) {
-		_region_view->change_note_time (_event, t);
-		had_change = true;
+	Evoral::Beats const t = _region_view->source_relative_time_converter().from
+		(_time_clock.current_time() - (_region_view->region()->position() - _region_view->region()->start()));
+
+	if (!_time_all.get_sensitive() || _time_all.get_active ()) {
+		for (set<NoteBase*>::iterator i = _events.begin(); i != _events.end(); ++i) {
+			if (t != (*i)->note()->time()) {
+				_region_view->change_note_time (*i, t);
+				had_change = true;
+			}
+		}
 	}
 
-	double const d = _region_view->time_converter().from (_length_clock.current_duration ());
+	Evoral::Beats const d = _region_view->region_relative_time_converter().from (_length_clock.current_duration ());
 
-	if (d != _event->note()->length()) {
-		_region_view->change_note_length (_event, d);
-		had_change = true;
+	if (!_length_all.get_sensitive() || _length_all.get_active ()) {
+		for (set<NoteBase*>::iterator i = _events.begin(); i != _events.end(); ++i) {
+			if (d != (*i)->note()->length()) {
+				_region_view->change_note_length (*i, d);
+				had_change = true;
+			}
+		}
 	}
 
 	if (!had_change) {
@@ -157,7 +223,7 @@ EditNoteDialog::run ()
 
 	_region_view->apply_diff ();
 
-	_event->set_selected (_event->selected()); // change color 
-	
-	return r;
+	for (set<NoteBase*>::iterator i = _events.begin(); i != _events.end(); ++i) {
+		(*i)->set_selected ((*i)->selected()); // change color
+	}
 }