From 35b1a52b1e298026dba25aa84b2e833b01078cda Mon Sep 17 00:00:00 2001
From: Jonathan Wilkes <jon.w.wilkes@gmail.com>
Date: Wed, 6 Jan 2016 23:39:13 -0500
Subject: [PATCH] Give window focus when asking if we need to save before
closing. Using a (possibly buggy) "restore" method instead of the more
obvious "focus" method, so this needs more testing
---
pd/nw/pdgui.js | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/pd/nw/pdgui.js b/pd/nw/pdgui.js
index 2493afb0e..f69ce4373 100644
--- a/pd/nw/pdgui.js
+++ b/pd/nw/pdgui.js
@@ -520,15 +520,11 @@ function canvas_menuclose_callback(cid_for_dialog, cid, force) {
no_button.onclick = function() {
patchwin[cid_for_dialog].window.canvas_events.close_without_saving(cid, force);
};
+ // Boy does this seem wrong-- restore() brings the window to the front of
+ // the stacking order. But that is really the job of focus(). This works
+ // under Ubuntu-- need to test it on OSX...
+ patchwin[cid_for_dialog].restore();
dialog.showModal();
-// reply = patchwin[cid_for_dialog].window
-// .confirm("Save changes to " + title + "?");
-// if (reply) {
-// pdsend(cid_for_dialog + " menusave");
-// } else {
-// pdsend(cid_for_dialog + " dirty 0");
-// pdsend(cid + " menuclose " + force);
-// }
}
function gui_canvas_menuclose(cid_for_dialog, cid, force) {
--
GitLab