Skip to content
Permalink
Browse files

Final fix for bug #373

------- Comment #13 From Christian Walther 2007-07-15 10:04:56 [reply] -------
Created an attachment (id=229) [details]
patch to reinstate the SDL cursor after an NSApplicationActivatedEvent

Oops, you're right. My testcursor.c has diverged so far from the original that
I missed that. In fact, now that I think about it, that issue isn't even
addressed by the cursor rectangle patch - it is only concerned with windowed
mode (and fixes the problem there as a side effect). The problem in fullscreen
is still the same as before. How about the attachment for a fix? (I can confirm
that [NSCursor set] doesn't seem to have any effect when called from
QZ_DoActivate().)
  • Loading branch information
slouken committed Jul 15, 2007
1 parent 857f271 commit 195cfdb0b8676f180da3a577df1e4021515bc074
Showing with 10 additions and 1 deletion.
  1. +10 −1 src/video/quartz/SDL_QuartzEvents.m
@@ -945,7 +945,16 @@ disassociated from the mouse (and therefore
break;
case NSFlagsChanged:
break;
/* case NSAppKitDefined: break; */
case NSAppKitDefined:
[ NSApp sendEvent:event ];
if ([ event subtype ] == NSApplicationActivatedEventType && (mode_flags & SDL_FULLSCREEN)) {
/* the default handling of this event seems to reset any cursor set by [NSCursor set] (used by SDL_SetCursor() in fullscreen mode) to the default system arrow cursor */
SDL_Cursor *sdlc = SDL_GetCursor();
if (sdlc != NULL && sdlc->wm_cursor != NULL) {
[ sdlc->wm_cursor->nscursor set ];
}
}
break;
/* case NSApplicationDefined: break; */
/* case NSPeriodic: break; */
/* case NSCursorUpdate: break; */

0 comments on commit 195cfdb

Please sign in to comment.