Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Date: Thu, 17 Apr 2003 23:27:34 -0400
From: Darrell Walisser
Subject: Yet another OS X cursor bug

The synopsis:

1. Call SDL_ShowCursor(0);
2. Call SDL_SetVideoMode();
3. Call SDL_GetEvent();
3. Call SDL_ShowCursor(1);

The result: Sometimes the cursor doesn't come back! Ack! Oddly enough,
it does come back when mousing over the dock or clicking in the menu
bar. But that's besides the point.

The reason why this is happening is a flaw in the handling of
activation/deactivation events. The short explanation is that the
HideCursor() and ShowCursor() calls must be balanced, but if the cursor
was initially hidden, HideCursor() was called again on the activate
event - so now the next ShowCursor() fails (as does the next, and the
next, for some reason).

So, here's the patch. All it does is keep track of the
HideCursor()/ShowCursor() calls so that they will always be balanced.
  • Loading branch information
slouken committed Apr 20, 2003
1 parent ea1f9d4 commit 5ed9bd0
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 7 deletions.
12 changes: 8 additions & 4 deletions src/video/quartz/SDL_QuartzEvents.m
Expand Up @@ -301,9 +301,10 @@ static void QZ_DoActivate (_THIS)
{
in_foreground = YES;

/* Hide the mouse cursor if was hidden */
if (!cursor_visible) {
/* Hide the cursor if it was hidden by SDL_ShowCursor() */
if (!cursor_visible && !cursor_hidden) {
HideCursor ();
cursor_hidden = YES;
}

/* Regrab input, only if it was previously grabbed */
Expand All @@ -330,8 +331,11 @@ static void QZ_DoDeactivate (_THIS) {
/* Reassociate mouse and cursor */
CGAssociateMouseAndMouseCursorPosition (1);

/* Show the cursor */
ShowCursor ();
/* Show the cursor if it was hidden by SDL_ShowCursor() */
if (!cursor_visible && cursor_hidden) {
ShowCursor ();
cursor_hidden = NO;
}

SDL_PrivateAppActive (0, SDL_APPINPUTFOCUS);
}
Expand Down
4 changes: 3 additions & 1 deletion src/video/quartz/SDL_QuartzVideo.h
Expand Up @@ -138,7 +138,8 @@ typedef struct SDL_PrivateVideoData {
Uint8 expect_mouse_up; /* used to determine when to send mouse up events */
Uint8 grab_state; /* used to manage grab behavior */
NSPoint cursor_loc; /* saved cursor coords, for activate/deactivate when grabbed */
BOOL cursor_visible; /* tells if cursor was hidden or not */
BOOL cursor_visible; /* tells if cursor was instructed to be hidden or not (SDL_ShowCursor) */
BOOL cursor_hidden; /* tells if cursor is *actually* hidden or not */
Uint8* sw_buffers[2]; /* pointers to the two software buffers for double-buffer emulation */
SDL_Thread *thread; /* thread for async updates to the screen */
SDL_sem *sem1, *sem2; /* synchronization for async screen updates */
Expand Down Expand Up @@ -183,6 +184,7 @@ typedef struct SDL_PrivateVideoData {
#define grab_state (this->hidden->grab_state)
#define cursor_loc (this->hidden->cursor_loc)
#define cursor_visible (this->hidden->cursor_visible)
#define cursor_hidden (this->hidden->cursor_hidden)
#define sw_buffers (this->hidden->sw_buffers)
#define thread (this->hidden->thread)
#define sem1 (this->hidden->sem1)
Expand Down
10 changes: 8 additions & 2 deletions src/video/quartz/SDL_QuartzWM.m
Expand Up @@ -72,15 +72,21 @@ static int QZ_ShowWMCursor (_THIS, WMcursor *cursor) {

if ( cursor == NULL) {
if ( cursor_visible ) {
HideCursor ();
if (!cursor_hidden) {
HideCursor ();
cursor_hidden = YES;
}
cursor_visible = NO;
QZ_ChangeGrabState (this, QZ_HIDECURSOR);
}
}
else {
SetCursor(&cursor->curs);
if ( ! cursor_visible ) {
ShowCursor ();
if (cursor_hidden) {
ShowCursor ();
cursor_hidden = NO;
}
cursor_visible = YES;
QZ_ChangeGrabState (this, QZ_SHOWCURSOR);
}
Expand Down

0 comments on commit 5ed9bd0

Please sign in to comment.