X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fardour_window.cc;h=628999124fb7d62bc164f72dffccc01cb01fb3dc;hb=b04f89be519b6c3257cbe5707992b514771bce63;hp=808fcd9fd700ce2f6417afc1c1c90e1a4fa36aca;hpb=57613dc0cc0d47b5eabeae45d369b8c221ea67f6;p=ardour.git

diff --git a/gtk2_ardour/ardour_window.cc b/gtk2_ardour/ardour_window.cc
index 808fcd9fd7..628999124f 100644
--- a/gtk2_ardour/ardour_window.cc
+++ b/gtk2_ardour/ardour_window.cc
@@ -24,6 +24,7 @@
 
 #include "ardour_window.h"
 #include "ardour_ui.h"
+#include "ui_config.h"
 #include "keyboard.h"
 #include "utils.h"
 
@@ -38,14 +39,15 @@ ArdourWindow::ArdourWindow (string title)
 {
 	set_title (title);
 	init ();
-	set_position (Gtk::WIN_POS_CENTER);
+	set_position (Gtk::WIN_POS_MOUSE);
 }
 
-ArdourWindow::ArdourWindow (Gtk::Window& parent, string /*title*/)
+ArdourWindow::ArdourWindow (Gtk::Window& parent, string title)
 	: Window ()
 	, VisibilityTracker (*((Gtk::Window*)this))
 {
 	init ();
+	set_title (title);
 	set_transient_for (parent);
 	set_position (Gtk::WIN_POS_CENTER_ON_PARENT);
 }
@@ -58,14 +60,15 @@ ArdourWindow::~ArdourWindow ()
 bool
 ArdourWindow::on_key_press_event (GdkEventKey* ev)
 {
-	if (get_modal()) {
-		return Gtk::Window::on_key_press_event (ev);
-	}
+	bool handled = Gtk::Window::on_key_press_event (ev);
 
-	if (!relay_key_press (ev, this)) {
-		return Gtk::Window::on_key_press_event (ev);
+	if (!handled) {
+		if (!get_modal()) {
+			handled = relay_key_press (ev, this);
+		}
 	}
-	return true;
+
+	return handled;
 }
 
 bool
@@ -103,28 +106,40 @@ ArdourWindow::init ()
 	set_border_width (10);
 	add_events (Gdk::FOCUS_CHANGE_MASK);
 
-      /* ArdourWindows are not dialogs (they have no "OK" or "Close" button) but
+        /* ArdourWindows are not dialogs (they have no "OK" or "Close" button) but
            they should be considered part of the same "window level" as a dialog. This
-           works on X11 and Quartz, in that:
-           
-           (a) utility & dialog windows are considered to be part of the same level
+           works on X11 in that: 
+
+           (a) there are no window "levels"
            (b) they will float above normal windows without any particular effort
 	   (c) present()-ing them will make a utility float over a dialog or
                vice versa.
+
+           Some X11 Window managers (e.g. KDE) get this wrong, and so we allow the user
+           to select what type of window hint is used.
+
+           GTK+ on OS X uses different levels for DIALOG and UTILITY, and Cocoa has a bug/design
+           issue that it will not transfer keyboard focus across levels when hiding a window.
+           So on OS X, we use DIALOG for all ArdourWindows to ensure that keyboard focus
+           will return to the main window(s) when this window is hidden.
         */
 
-	if (ARDOUR_UI::config()->get_all_floating_windows_are_dialogs()) {
+#ifdef __APPLE__
+        set_type_hint (Gdk::WINDOW_TYPE_HINT_DIALOG);
+#else
+	if (UIConfiguration::instance().get_all_floating_windows_are_dialogs()) {
 		set_type_hint (Gdk::WINDOW_TYPE_HINT_DIALOG);
 	} else {
 		set_type_hint (Gdk::WINDOW_TYPE_HINT_UTILITY);
 	}
+#endif
 
 	Gtk::Window* parent = WM::Manager::instance().transient_parent();
 
 	if (parent) {
 		set_transient_for (*parent);
 	}
-	
+
 	ARDOUR_UI::CloseAllDialogs.connect (sigc::mem_fun (*this, &ArdourWindow::hide));
 
 	proxy = new WM::ProxyTemporary (get_title(), this);