http://bugs.winehq.org/show_bug.cgi?id=26693
Summary: Neverwinter Nights 2 installer window is obscured by installer background image Product: Wine Version: 1.3.17 Platform: x86 OS/Version: Linux Status: NEW Keywords: Installer, regression Severity: normal Priority: P2 Component: winex11.drv AssignedTo: wine-bugs@winehq.org ReportedBy: arethusa26@gmail.com CC: dmitry@codeweavers.com
Created an attachment (id=33993) --> (http://bugs.winehq.org/attachment.cgi?id=33993) Neverwinter Nights 2 +x11drv,+win log
When the installer for Neverwinter Nights 2 is launched on wine-1.3.17-173-g6eed4c4, the installer window always appears under the installer background image, which makes it impossible to perform the installation. The window appears correctly in wine-1.3.16, and a regression test indicates:
5ef58c11ce28c963908672f5c5c1fde866272ba5 is the first bad commit commit 5ef58c11ce28c963908672f5c5c1fde866272ba5 Author: Dmitry Timoshkov dmitry@codeweavers.com Date: Tue Mar 22 15:11:40 2011 +0800
winex11.drv: Do not use any window types except _NET_WM_WINDOW_TYPE_NORMAL.
Main window of the Atom Zombie Smasher game has WS_DLGFRAME style set, but Metacity allows making fullscreen ONLY normal windows, so do not use any window types except _NET_WM_WINDOW_TYPE_NORMAL.
:040000 040000 65ae9d82c83e54b32c7fa38754c5e5f4becd923c 96ed2dd11fbd04b352b0d6d7a42c9b88736b799f M dlls
Reverting the commit makes the window appear correctly again. A +x11drv,+win log is attached.
http://bugs.winehq.org/show_bug.cgi?id=26693
Andrew Nguyen arethusa26@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #33993|Neverwinter Nights 2 |Neverwinter Nights 2 description|+x11drv,+win log |installer +x11drv,+win log
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #1 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-08 22:35:41 CDT --- What Window Manager are you using? Is there a free download/demo available?
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #2 from Andrew Nguyen arethusa26@gmail.com 2011-04-09 04:11:25 CDT --- I am using Metacity 2.30.3 as shipped in Fedora 14. Unfortunately, there's no download available for Neverwinter Nights 2.
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #3 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-10 02:00:58 CDT --- Is this the game http://nwn.bioware.com/downloads/demo.html ?
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #4 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-10 04:41:35 CDT --- Created an attachment (id=34024) --> (http://bugs.winehq.org/attachment.cgi?id=34024) Set the transient_for hint after the window type
Does the attached patch make any difference?
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #5 from Andrew Nguyen arethusa26@gmail.com 2011-04-10 09:25:02 CDT --- (In reply to comment #3)
Is this the game http://nwn.bioware.com/downloads/demo.html ?
No, it's the installer for the Neverwinter Nights demo. No Neverwinter Nights 2 demo exists.
(In reply to comment #4)
Created an attachment (id=34024)
--> (http://bugs.winehq.org/attachment.cgi?id=34024) [details]
Set the transient_for hint after the window type
Does the attached patch make any difference?
Unfortunately, it doesn't help the NWN2 installer. Are there any additional traces that I should make?
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #6 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-10 21:32:42 CDT --- (In reply to comment #5)
Unfortunately, it doesn't help the NWN2 installer. Are there any additional traces that I should make?
Probably comparing +win,+x11drv,+event logs before and after the patch could help.
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #7 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-10 21:33:47 CDT --- Also, is the installer background image fullscreen? Could you please attach a screenshot?
http://bugs.winehq.org/show_bug.cgi?id=26693
GyB gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gyebro69@gmail.com
--- Comment #8 from GyB gyebro69@gmail.com 2011-04-11 13:24:31 CDT --- This regression affects some other installers, too. The Bionic Heart demo has the smallest download size (44 MB), so here's the link to it: http://www.bigdownload.com/games/bionic-heart/pc/bionic-heart-demo/
Fedora 14 Gnome 2.32.0 + Metacity 2.30.3
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #9 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-12 09:28:41 CDT --- The problem is that Metacity ignores the transient_for hint if the window owner is a fullscreen window, it places the fullscreen window on top of the owned window. Somehow dialog type changes internal WM semantics and compensates for that Metacity bug.
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #10 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-12 09:31:36 CDT --- I've added support for _NET_WM_STATE_MODAL (for owned windows) but that didn't help.
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #11 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-18 03:46:06 CDT --- I've filed a bug report for Metacity: https://bugzilla.gnome.org/show_bug.cgi?id=648079
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #12 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-18 04:26:29 CDT --- And here is another one: https://bugzilla.gnome.org/show_bug.cgi?id=648080
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #13 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-19 07:06:07 CDT --- I sent a workaround for the two Metacity bugs mentioned above: http://www.winehq.org/pipermail/wine-patches/2011-April/101072.html
http://bugs.winehq.org/show_bug.cgi?id=26693
--- Comment #14 from Dmitry Timoshkov dmitry@codeweavers.com 2011-04-19 21:39:35 CDT --- Here is a better version: http://www.winehq.org/pipermail/wine-patches/2011-April/101075.html
http://bugs.winehq.org/show_bug.cgi?id=26693
Andrew Nguyen arethusa26@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #15 from Andrew Nguyen arethusa26@gmail.com 2011-04-20 12:58:32 CDT --- The Neverwinter Nights 2 installer and the Bionic Hearts demo installer both work correctly after fd043c56443430ee993c7d7281845b9b726b48d6, so I'm marking this fixed.
http://bugs.winehq.org/show_bug.cgi?id=26693
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #16 from Alexandre Julliard julliard@winehq.org 2011-04-29 13:15:34 CDT --- Closing bugs fixed in 1.3.19.