Re: winex11.drv: Don't add owned windows to the taskbar
Dmitry Timoshkov escribió: > "Alex Villacís Lasso" <[EMAIL PROTECTED]> wrote: > > >>> The problem with IDA is that the button you see in the Windows task bar >>> does not belong to the main IDA window, it belongs to a zero sized visible >>> window with the same caption. Since it's zero sized Wine doesn't map it all. >>> So, while formally it's a regression, technically it's not: main IDA window >>> has never been added to the task bar before my patch. This all can be easily >>> discovered with spy++ and a simple test app. >>> >>> >>> >> This also affects many, if not all, VB6 applications (and possibly many >> Delphi applications). Personally, I consider this a rather serious >> regression and propose backing out of commit >> cb3ff54789c652b02d852f40db71cafd9fd67ca9 as a fix. >> >> BTW, this reminds me of bug 1598. If the explanation of the bug for IDA >> above is correct, then the "buggy" behavior of not assigning >> minimize/maximize buttons to the window that appears in the panel is >> sort of consistent with this regression. Maybe the behavior should be >> that visible windows owned by an invisible window should get mapped in >> the panel. >> > > 1598 is closed as fixed, and I have no idea how missing minimize/maximize > buttons are related to owned windows, and why you think that it's somehow > related to the visibility of an owner. > > What you need to be fixed is making a zero sized window with WS_VISIBLE > style set mapped, but X11/WMs doesn't handle that gracefully. > > Bug 1598 (even if closed and fixed) is relevant because it was also triggered by having an invisible owner window, as my own analysis in comment #16 suggested. This bug (1598) also made the window button disappear from the taskbar, with the same application that now exhibited the regression. I was just preparing a patch to fix this very issue over the weekend, but it seems Jacek Caban beat me to it. I am currently recompiling my tree to test the patch. -- perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'
Re: winex11.drv: Don't add owned windows to the taskbar
"Alex Villacís Lasso" <[EMAIL PROTECTED]> wrote: >> The problem with IDA is that the button you see in the Windows task bar >> does not belong to the main IDA window, it belongs to a zero sized visible >> window with the same caption. Since it's zero sized Wine doesn't map it all. >> So, while formally it's a regression, technically it's not: main IDA window >> has never been added to the task bar before my patch. This all can be easily >> discovered with spy++ and a simple test app. >> >> > This also affects many, if not all, VB6 applications (and possibly many > Delphi applications). Personally, I consider this a rather serious > regression and propose backing out of commit > cb3ff54789c652b02d852f40db71cafd9fd67ca9 as a fix. > > BTW, this reminds me of bug 1598. If the explanation of the bug for IDA > above is correct, then the "buggy" behavior of not assigning > minimize/maximize buttons to the window that appears in the panel is > sort of consistent with this regression. Maybe the behavior should be > that visible windows owned by an invisible window should get mapped in > the panel. 1598 is closed as fixed, and I have no idea how missing minimize/maximize buttons are related to owned windows, and why you think that it's somehow related to the visibility of an owner. What you need to be fixed is making a zero sized window with WS_VISIBLE style set mapped, but X11/WMs doesn't handle that gracefully. -- Dmitry.
Re: winex11.drv: Don't add owned windows to the taskbar
Dmitry Timoshkov escribio': > "Alexander Dorofeyev" <[EMAIL PROTECTED]> wrote: > > >> There's a regression in IDA Pro disassembler. Nothing is added to the >> taskbar at >> all, furthermore, when you are starting debugger, before there was a window >> appearing that asks pass control to the application or not. Now it's not >> appearing, and can't be found because there's nothing on the taskbar, which >> is >> pretty bad because the app doesn't respond to any other input at that >> moment. >> Minimizing the main window makes the app disappear altogether, so it's >> pretty >> unusable. >> > > The problem with IDA is that the button you see in the Windows task bar > does not belong to the main IDA window, it belongs to a zero sized visible > window with the same caption. Since it's zero sized Wine doesn't map it all. > So, while formally it's a regression, technically it's not: main IDA window > has never been added to the task bar before my patch. This all can be easily > discovered with spy++ and a simple test app. > > This also affects many, if not all, VB6 applications (and possibly many Delphi applications). Personally, I consider this a rather serious regression and propose backing out of commit cb3ff54789c652b02d852f40db71cafd9fd67ca9 as a fix. BTW, this reminds me of bug 1598. If the explanation of the bug for IDA above is correct, then the "buggy" behavior of not assigning minimize/maximize buttons to the window that appears in the panel is sort of consistent with this regression. Maybe the behavior should be that visible windows owned by an invisible window should get mapped in the panel. -- perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'
Re: winex11.drv: Don't add owned windows to the taskbar
"Alexander Dorofeyev" <[EMAIL PROTECTED]> wrote: > There's a regression in IDA Pro disassembler. Nothing is added to the taskbar > at > all, furthermore, when you are starting debugger, before there was a window > appearing that asks pass control to the application or not. Now it's not > appearing, and can't be found because there's nothing on the taskbar, which > is > pretty bad because the app doesn't respond to any other input at that moment. > Minimizing the main window makes the app disappear altogether, so it's > pretty > unusable. The problem with IDA is that the button you see in the Windows task bar does not belong to the main IDA window, it belongs to a zero sized visible window with the same caption. Since it's zero sized Wine doesn't map it all. So, while formally it's a regression, technically it's not: main IDA window has never been added to the task bar before my patch. This all can be easily discovered with spy++ and a simple test app. -- Dmitry.
Re: winex11.drv: Don't add owned windows to the taskbar
Hi. There's a regression in IDA Pro disassembler. Nothing is added to the taskbar at all, furthermore, when you are starting debugger, before there was a window appearing that asks pass control to the application or not. Now it's not appearing, and can't be found because there's nothing on the taskbar, which is pretty bad because the app doesn't respond to any other input at that moment. Minimizing the main window makes the app disappear altogether, so it's pretty unusable. I've KDE 3.5. You can get free version (that has same symptoms) from http://www.datarescue.com/ to help tweaking this. Dmitry Timoshkov wrote: > Hello, > > this patch fixes the bug 12160. According to my tests explorer in XP > doesn't add owned windows to the taskbar. > > Changelog: > winex11.drv: Don't add owned windows to the taskbar. > ---
Re: winex11.drv: Don't add owned windows to the taskbar
Dmitry Timoshkov <[EMAIL PROTECTED]> writes: > this patch fixes the bug 12160. According to my tests explorer in XP > doesn't add owned windows to the taskbar. Do we really want to remove them from the pager too? -- Alexandre Julliard [EMAIL PROTECTED]