On Mon, 28 Jan 2002, Ori Pessach wrote:
- Changed is_window_managed()'s default return value to TRUE
Perhaps remove the cruft this leaves behind, then.
- In tooltips.c, TOOLTIPS_NCCreate(), I set the WS_EX_TOOLWINDOW bit in the
tooltip's dwExStyle field. This, I believe, is the right thing to do regardless of WM considerations, since MSDN specifies that tooltip windows have that bit set. (http://msdn.microsoft.com/library/default.asp?url=/library/en-us/shellcc/pla...)
Then I suggest you submit this piece of the patch to wine-patches independently of the rest of this.
- In menu.c, I replaced CreateWindowA() with CreateWindowExA(), passing
WS_EX_TOOLWINDOW as the first parameter. I'm not so sure about this one. I don't know whether Windows does this (Win16 surely doesn't), and if it doesn't, it might break apps that examine those bits.
Since this is probably not the case, perhaps it's better to check for the popup menu class in is_window_managed(), something like
if (GetClassLongA(win->hwndSelf, GCW_ATOM) == POPUPMENU_CLASS_ATOM) return FALSE;
With these changes, Pajama Sam works correctly, except for the KDE panel problem.
I'm currently looking at that issue.