Re: [PATCH v10 0/6] MR6569: winex11: Refactor window state/config tracker to fix race conditions.

[Rémi Bernon (＠rbernon)]

On Thu Jan 16 11:31:22 2025 +0000, Patrick Laimbock wrote:
> Hi Rémi. With the change below (bug51357.patch) from
> https://bugs.winehq.org/show_bug.cgi?id=51357 the wrong mouse offset is
> fixed in the VST plugin's main window titled 'Spire'. But, e.g. clicking
> the 'menu' button shows the menu in the wrong place. See screenshot. The
> menu should appear below the 'menu' button (top row) in the window
> titled 'Spire'. So the change above does not fix the issue but hopefully
> it gives you an idea what the problem might be. If you need more
> information please let me know. Thanks!
> [bug51357.patch](/uploads/dea8a1e711846f7e7642c16eacd284b4/bug51357.patch)
>  ![Screenshot_From_2025-01-16_11-49-56](/uploads/decbb70d2abc1b99d3fd28e0b6382080/Screenshot_From_2025-01-16_11-49-56.png)

Thank you! Yes, I think we could consider doing something like and use window-relative coordinates all the time, now that we're better tracking the X window tree position, and stopped using XTranslateCoordinates. It would have to wait until Wine 10 is released though, I don't want to break something else this late.