[PATCH] winex11.drv: Let the Window Manager handle the off-screen window if it's managed.

Gabriel Ivăncescu gabrielopcode at gmail.com
Thu Oct 15 07:44:31 CDT 2020


On 14/10/2020 23:37, Marvin wrote:
> Thank you for your contribution to Wine!
> 
> This is an automated notification to let you know that your patch has
> been reviewed and its status set to "Rejected".
> 
> This means that the patch has been rejected by a reviewer. You should
> have received a mail explaining why it was rejected. You need to fix
> the issue and resend the patch, or if you are convinced that your
> patch is good as is, you should reply to the rejection message with
> your counterarguments.
> 
> If you do not understand the reason for this status, disagree with our
> assessment, or are simply not sure how to proceed next, please ask for
> clarification by replying to this email.
> 

Hi,

Why was this rejected? To me, as it stands, it makes no sense to try to 
unmap the window if it's offscreen when it is managed; we're overriding 
its job.

Some WMs implement their own virtual desktops/workspaces and "snap" and 
"shift" the windows when they're offscreen (e.g. if you move the window 
to the left, it snaps to the left workspace and shifts up to the right 
when you switch to it). This completely ignores that and hides them when 
they would *not* be hidden by the WM.



More information about the wine-devel mailing list