We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 281

Summary: Can't minimize windows after running fullscreen
Product: SDL Reporter: Frank Becker <crittermail2005>
Component: videoAssignee: Christian Walther <cwalther>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: major    
Priority: P2 CC: crittermail2005, info
Version: 1.2.10   
Hardware: PowerPC   
OS: Mac OS X 10.4 (PPC)   
Attachments: proposed patch

Description Frank Becker 2006-07-16 16:12:26 UTC 
When you quit an SDL application while in fullscreen mode, the open windows of other applications start refusing to minimize (their minimize control is grayed out).

Newly opened windows do not suffer from the problem, and running the SDL application again, but quitting it while in windowed mode also fixes the old windows again.

This was introduced in 1.2.10. 1.2.9 is ok.

The related email thread is "OSX: Can't minimize windows after running fullscreen".
Comment 1 Christian Walther 2006-07-18 15:21:06 UTC 
Seems I was wrong about newly opened windows not being affected, and running the SDL app again fixing the old ones. I can't reproduce that now - the minimize buttons of old and new windows stay disabled until you close and reopen their application.

So far, I verified that the bug was indeed introduced in revision 1885 with the new fading code. The next step, hopefully tomorrow, will be finding what the exact problem is - it looks like something isn't getting cleaned up properly when you quit out of fullscreen.
Comment 2 Christian Walther 2006-07-23 07:37:57 UTC 
Created attachment 151 [details]
proposed patch

Wow, that was an interesting bug to chase. It was a timing issue: it seems that for some reason, a certain time must pass between ShowMenuBar() being called in QZ_UnsetVideoMode() and the application quitting. Before rev. 1885, this delay was provided by the slow hand-coded fade. With the asynchronous Core Graphics fading introduced in rev. 1885, that delay was no longer present (most of the time) and the bug became apparent. Adding an SDL_Delay(100) somewhere between ShowMenuBar() and the end of QZ_VideoQuit() lowered the frequency of the bug appearing from "almost every time" to "very rarely" here.

However, there is another solution: doing the ShowMenuBar() before releasing the captured display instead of afterwards. Apparently, no delay is necessary in that case, and it looks nicer to me anyway because it is the reverse order of the way things are set up in the beginning: capture display - set video mode - hide menu bar - ... - show menu bar - reset video mode - release captured display. So, this is what the attached patch does.

In addition, I've taken the liberty of
- removing some unused code that I forgot to remove in rev. 1885,
- fixing two warnings about undeclared functions in SDL_QuartzVideo.m by including OpenGL.h (whose name is a bit misleading - it only declares CGL stuff, so there's no interference with SDL_opengl.h).
Comment 3 Christian Walther 2006-07-30 02:43:19 UTC 
Any comments? Frank, can you try this?
Comment 4 Frank Becker 2006-08-02 01:27:25 UTC 
The patch works for me. Thanks!
Comment 5 Sam Lantinga 2006-09-23 21:27:31 UTC 
Thanks Christian, your patch is included in subversion.

Hey, if you want to take a look at SDL 1.3, I could use a hand with the fullscreen code.  I don't really know what I'm doing poking around in there. :)
Comment 6 Mike Blaguszewski 2007-04-06 17:53:25 UTC 
*** Bug 404 has been marked as a duplicate of this bug. ***