From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#11732: Follow-up to bug#11732 Date: Sat, 30 Jun 2018 14:32:37 +0300 Message-ID: <83d0w8mrvu.fsf@gnu.org> References: <87obohkxga.fsf@XAVIER-PC.i-did-not-set--mail-host-address--so-tickle-me> <83y3f0p6n7.fsf@gnu.org> <5B35F120.9040807@gmx.at> <83fu16nepn.fsf@gnu.org> <5B373A21.9050407@gmx.at> NNTP-Posting-Host: blaine.gmane.org X-Trace: blaine.gmane.org 1530358267 19684 195.159.176.226 (30 Jun 2018 11:31:07 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Sat, 30 Jun 2018 11:31:07 +0000 (UTC) Cc: 11732@debbugs.gnu.org, mhatta@gmail.com To: martin rudalics Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Sat Jun 30 13:31:03 2018 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1fZE5m-00053L-Sf for geb-bug-gnu-emacs@m.gmane.org; Sat, 30 Jun 2018 13:31:03 +0200 Original-Received: from localhost ([::1]:46479 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fZE7s-0005t9-MW for geb-bug-gnu-emacs@m.gmane.org; Sat, 30 Jun 2018 07:33:14 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:48373) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fZE7l-0005t2-Ru for bug-gnu-emacs@gnu.org; Sat, 30 Jun 2018 07:33:06 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fZE7i-0001C2-NW for bug-gnu-emacs@gnu.org; Sat, 30 Jun 2018 07:33:05 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:60726) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fZE7i-0001Bs-JV for bug-gnu-emacs@gnu.org; Sat, 30 Jun 2018 07:33:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1fZE7i-0005e6-E6 for bug-gnu-emacs@gnu.org; Sat, 30 Jun 2018 07:33:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sat, 30 Jun 2018 11:33:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 11732 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 11732-submit@debbugs.gnu.org id=B11732.153035835821671 (code B ref 11732); Sat, 30 Jun 2018 11:33:02 +0000 Original-Received: (at 11732) by debbugs.gnu.org; 30 Jun 2018 11:32:38 +0000 Original-Received: from localhost ([127.0.0.1]:40390 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1fZE7J-0005dS-Qd for submit@debbugs.gnu.org; Sat, 30 Jun 2018 07:32:38 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:57334) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1fZE7I-0005dD-CM for 11732@debbugs.gnu.org; Sat, 30 Jun 2018 07:32:36 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fZE79-0000S4-TW for 11732@debbugs.gnu.org; Sat, 30 Jun 2018 07:32:31 -0400 Original-Received: from fencepost.gnu.org ([2001:4830:134:3::e]:59498) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fZE79-0000Ra-Q3; Sat, 30 Jun 2018 07:32:27 -0400 Original-Received: from [176.228.60.248] (port=4074 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1fZE79-0007oY-1G; Sat, 30 Jun 2018 07:32:27 -0400 In-reply-to: <5B373A21.9050407@gmx.at> (message from martin rudalics on Sat, 30 Jun 2018 10:06:57 +0200) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:148008 Archived-At: > Date: Sat, 30 Jun 2018 10:06:57 +0200 > From: martin rudalics > CC: mhatta@gmail.com, 11732@debbugs.gnu.org > > > SWP_NOACTIVATE was just a copy-paste from similar calls elsewhere. > > The rationale for the code was to tell windows to put the frame from > > which the file-selection dialog popped behind the dialog. > > IMO these two calls are not entirely kosher - after all the dialog box > is in the topmost group and the selected frame not. So some other > application that interrupts the dialog might mess things up. Anyway, > I would try moving the SWP_NOACTIVATE from the dialog to the selected > frame call like > > SetWindowPos (dialog, HWND_TOPMOST, 0, 0, 0, 0, > SWP_NOMOVE | SWP_NOSIZE | SWP_NOOWNERZORDER); > SetWindowPos (FRAME_W32_WINDOW (SELECTED_FRAME ()), > dialog, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE > | SWP_NOACTIVATE); > > Note that I can't test it because nothing is broken here in the first > place. I think I tried with HWND_TOPMOST, but what it does (and I see it now with your suggestion) is it doesn't allow raising any other (non-Emacs) window above the file-selection dialog (in the z-order). The original code didn't behave that way, so I looked for a better option, and HWND_NOTOPMOST seemed to do the job... > > If > > w32_dialog_in_progress is meant to do that, I don't understand how it > > does that; can you explain? > > All w32_dialog_in_progress does is moving frames from and to the > topmost group. I don't like putting frames in the topmost group - but > if one doesn't use child frames and wants a support frame on top of a > normal frame, setting just the z-order is not enough: You run into an > eternal loop where Emacs tries to put the support frame above the > normal one and Windows immediately reverses that because I obviously > want the normal frame to retain focus. By "support frame" here you mean the dialog box? If not, then why is this function called every time we are about to show a dialog box? > > (Btw, as long as we are discussing this: the above-suspended value of > > the z-group frame parameter appears to be completely undocumented.) > > Conceptually, users should never see it: It is set only during > dialogs. But if you think it should be documented I'll do that. If it's an internal setting that should never be seen outside the C sources, then it should be documented in the commentary preceding x_set_z_group, and the comment should tell this value is internal, so that whoever reads the code will understand what each setting wants to achieve. > > + if (msg == WM_NOTIFY) > > + { > > + SetWindowPos (hdlg, HWND_NOTOPMOST, 0, 0, 0, 0, > > + SWP_NOMOVE | SWP_NOSIZE | SWP_NOACTIVATE > > + | SWP_NOOWNERZORDER); > > The HWND_NOTOPMOST doesn't look good - dialog boxes should be topmost. > Could you try with > > static UINT_PTR CALLBACK > font_dialog_callback (HWND hdlg, UINT msg, WPARAM wParam, LPARAM lParam) > { > static HWND cf_hwnd; > > if (msg == WM_INITDIALOG) > cf_hwnd = ((CHOOSEFONT *)lParam)->hwndOwner; > > if (msg == WM_NOTIFY) > { > SetWindowPos (hdlg, HWND_TOPMOST, 0, 0, 0, 0, > SWP_NOMOVE | SWP_NOSIZE | SWP_NOOWNERZORDER); > SetWindowPos (cf_hwnd, hdlg, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE > | SWP_NOACTIVATE); > } > return 0; > } This looks good on XP, I will try on Windows 7 later. Curiously, HWND_TOPMOST here doesn't prevent raising other windows above the dialog box, as it does with file selector. > It doesn't show any strange effects here, at least. I think the problems I saw were on Windows 7. Will check. Thanks.