From 3973ce81b37a9ba6fcd2e7ff95ca730b0aa32252 Mon Sep 17 00:00:00 2001
From: Paul Davis <paul@linuxaudiosystems.com>
Date: Thu, 2 May 2013 18:13:35 -0400
Subject: remove all of Gtk::Window::set_position (WIN_POS_MOUSE) for anything
 deriving from ArdourDialog or ArdourWindow; move the set_position() call into
 those classes' constructors, so that they are called before the windows are
 realized, and thus it actually works

---
 gtk2_ardour/edit_note_dialog.cc | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

(limited to 'gtk2_ardour/edit_note_dialog.cc')

diff --git a/gtk2_ardour/edit_note_dialog.cc b/gtk2_ardour/edit_note_dialog.cc
index 17827693e7..5aacc59f3c 100644
--- a/gtk2_ardour/edit_note_dialog.cc
+++ b/gtk2_ardour/edit_note_dialog.cc
@@ -146,16 +146,13 @@ EditNoteDialog::EditNoteDialog (MidiRegionView* rv, set<ArdourCanvas::CanvasNote
 	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
@@ -227,6 +224,4 @@ EditNoteDialog::run ()
 	for (set<ArdourCanvas::CanvasNoteEvent*>::iterator i = _events.begin(); i != _events.end(); ++i) {
 		(*i)->set_selected ((*i)->selected()); // change color
 	}
-
-	return r;
 }
-- 
cgit v1.2.3