Skip to content

Commit

Permalink
Merge pull request #18628 from ramezgerges/x11_window_activation
Browse files Browse the repository at this point in the history
fix(x11): Window.Activate() should immediately raise the window
  • Loading branch information
jeromelaban authored Oct 31, 2024
2 parents a61a17f + 1f06b1e commit 8bfd517
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/Uno.UI.Runtime.Skia.X11/X11WindowWrapper.cs
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ public override void Activate()
var x11Window = _host.RootX11Window;
using var lockDiposable = X11Helper.XLock(x11Window.Display);
_ = XLib.XRaiseWindow(x11Window.Display, x11Window.Window);
_ = XLib.XFlush(x11Window.Display); // Important! Otherwise X commands will sit waiting to be flushed, and since the window is not activated, there are no new X commands being sent to force a flush.

// We could send _NET_ACTIVE_WINDOW as well, although it doesn't seem to be needed (and only works with EWMH-compliant WMs)
// XClientMessageEvent xclient = default;
Expand Down

0 comments on commit 8bfd517

Please sign in to comment.