From 833cfe534a2052126bad696e33eec469972babe5 Mon Sep 17 00:00:00 2001 From: "Pierre-Loup A. Griffais" Date: Wed, 26 Mar 2014 12:54:51 -0700 Subject: [PATCH] Remove the RaiseWindow call from OnWindowRestored for now. It seems like a net improvement in all the scenarios Sam and I could think of, and looking at hg history it was added for fullscreen window management specifically. Much of that code has changed since then, but maybe it needs to stay there for that and simply be moved to a fullscreen condition check. It would solve this issue: https://bugzilla.libsdl.org/show_bug.cgi?id=2439 As well as cases where on SteamOS, we hide/show specific Steam overlay windows while expecting them to stay in the background, since changing the window stacking order really angers the NVIDIA driver. CR: Sam. --- src/video/SDL_video.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/video/SDL_video.c b/src/video/SDL_video.c index 18f65d0834948..c1f379140d414 100644 --- a/src/video/SDL_video.c +++ b/src/video/SDL_video.c @@ -2129,7 +2129,13 @@ SDL_OnWindowMinimized(SDL_Window * window) void SDL_OnWindowRestored(SDL_Window * window) { - SDL_RaiseWindow(window); + /* + * FIXME: Is this fine to just remove this, or should it be preserved just + * for the fullscreen case? In principle it seems like just hiding/showing + * windows shouldn't affect the stacking order; maybe the right fix is to + * re-decouple OnWindowShown and OnWindowRestored. + */ + //SDL_RaiseWindow(window); if (FULLSCREEN_VISIBLE(window)) { SDL_UpdateFullscreenMode(window, SDL_TRUE);