/*
- * freeglut_menu.c
+ * fg_menu.c
*
* Pull-down menu creation and handling.
*
* These variables are for rendering the freeglut menu items.
*
* The choices are fore- and background, with and without h for Highlighting.
- * Old GLUT appeared to be system-dependant for its colors (sigh) so we are
+ * Old GLUT appeared to be system-dependent for its colors (sigh) so we are
* too. These variables should be stuffed into global state and initialized
* via the glutInit*() system.
*/
if( menuEntry != menu->ActiveEntry )
{
- menu->Window->State.Redisplay = GL_TRUE;
+ menu->Window->State.WorkMask |= GLUT_DISPLAY_WORK;
if( menu->ActiveEntry )
menu->ActiveEntry->IsActive = GL_FALSE;
}
menu->ActiveEntry = menuEntry;
- menu->IsActive = GL_TRUE; /* XXX Do we need this? */
+ menu->IsActive = GL_TRUE;
/*
- * OKi, we have marked that entry as active, but it would be also
+ * OK, we have marked that entry as active, but it would be also
* nice to have its contents updated, in case it's a sub menu.
* Also, ignore the return value of the check function:
*/
}
/* Activate it because its parent entry is active */
- menuEntry->SubMenu->IsActive = GL_TRUE; /* XXX Do we need this? */
+ menuEntry->SubMenu->IsActive = GL_TRUE;
}
/* Report back that we have caught the menu cursor */
( !menu->ActiveEntry->SubMenu ||
!menu->ActiveEntry->SubMenu->IsActive ) )
{
- menu->Window->State.Redisplay = GL_TRUE;
+ menu->Window->State.WorkMask |= GLUT_DISPLAY_WORK;
menu->ActiveEntry->IsActive = GL_FALSE;
menu->ActiveEntry = NULL;
}
fgState.MenuStateCallback(GLUT_MENU_IN_USE);
if (fgState.MenuStatusCallback)
/* window->State.MouseX and window->State.MouseY are relative to client area origin, as needed */
- fgState.MenuStatusCallback(GLUT_MENU_IN_USE, window->State.MouseX, window->State.MouseY);
+ fgState.MenuStatusCallback(GLUT_MENU_IN_USE, window->State.MouseX, window->State.MouseY, fgState.MenuStatusCallbackData);
}
fgSetWindow( menu->Window );
/*
* Update Highlight states of the menu
- *
- * Current mouse position is in menu->Window->State.MouseX/Y.
+ * NB: Current mouse position is in menu->Window->State.MouseX/Y
*/
void fgUpdateMenuHighlight ( SFG_Menu *menu )
{
* location.
* - Down-click any button outside the menu, menu active:
* deactivate the menu, and potentially activate a new menu
- * at the new mouse location
+ * at the new mouse location. This includes clicks in
+ * different windows of course
* - Down-click any button inside the menu, menu active:
* select the menu entry and deactivate the menu
* - Up-click the menu button, menu not active: nothing happens
* window to the window whose menu this is
*/
SFG_Window *save_window = fgStructure.CurrentWindow;
- SFG_Menu *save_menu = fgStructure.CurrentMenu, *active_menu = window->ActiveMenu;
- SFG_MenuEntry *active_entry = active_menu->ActiveEntry;
+ SFG_Menu *save_menu = fgStructure.CurrentMenu, *active_menu = window->ActiveMenu; /* active menu is always the one with the mouse in it, due to fghCheckMenuStatus */
+ SFG_MenuEntry *active_entry = active_menu->ActiveEntry; /* currently highlighted item -> must be the one that was just clicked */
SFG_Window *parent_window = window->ActiveMenu->ParentWindow;
- /* get clicked entry */
- while (active_entry->SubMenu)
+ /* ignore clicks on the submenu entry */
+ if (!active_entry->SubMenu)
{
- active_menu = active_entry->SubMenu;
- active_entry = active_menu->ActiveEntry;
- }
+ fgSetWindow( parent_window );
+ fgStructure.CurrentMenu = active_menu;
- fgSetWindow( parent_window );
- fgStructure.CurrentMenu = active_menu;
+ /* Deactivate menu and then call callback (we don't want menu to stay in view while callback is executing, and user should be able to change menus in callback) */
+ fgDeactivateMenu( parent_window );
+ active_menu->Callback( active_entry->ID, active_menu->CallbackData );
- /* Deactivate menu and then call callback (we don't want menu to stay in view while callback is executing) */
- fgDeactivateMenu( parent_window );
- active_menu->Callback( active_entry->ID );
-
- /* Restore the current window and menu */
- fgSetWindow( save_window );
- fgStructure.CurrentMenu = save_menu;
+ /* Restore the current window and menu */
+ fgSetWindow( save_window );
+ fgStructure.CurrentMenu = save_menu;
+ }
is_clicked = GL_TRUE; /* Don't reopen... */
}
/* Could reopen again in different location, as is_clicked remains false */
}
- /*
- * XXX Why does an active menu require a redisplay at
- * XXX this point? If this can come out cleanly, then
- * XXX it probably should do so; if not, a comment should
- * XXX explain it.
- */
- if( ! window->IsMenu )
- window->State.Redisplay = GL_TRUE;
-
is_handled = GL_TRUE;
}
+ else if ( fgState.ActiveMenus ) /* Don't have to check whether this was a downpress or an uppress, there is no way to get an uppress in another window before a downpress... */
+ {
+ /* if another window than the one clicked in has an open menu, close it */
+ SFG_Menu *menu = fgGetActiveMenu();
+ if ( menu ) /* any open menu? */
+ fgDeactivateMenu( menu->ParentWindow );
+
+ /* Leave is_handled to false, we didn't do anything relevant from the perspective of the window that was clicked */
+ }
/* No active menu, let's check whether we need to activate one. */
if( !is_clicked &&
if (window->State.MouseX>0 && window->State.MouseY>0 &&
window->State.MouseX<window->State.Width && window->State.MouseY<window->State.Height)
{
- /* XXX Posting a requisite Redisplay seems bogus. */
- window->State.Redisplay = GL_TRUE;
fghActivateMenu( window, button );
is_handled = GL_TRUE;
}
SFG_XYUse mouse_pos;
fghPlatformGetCursorPos(parent_window, GL_TRUE, &mouse_pos);
- fgState.MenuStatusCallback(GLUT_MENU_NOT_IN_USE, mouse_pos.X, mouse_pos.Y);
+ fgState.MenuStatusCallback(GLUT_MENU_NOT_IN_USE, mouse_pos.X, mouse_pos.Y, fgState.MenuStatusCallbackData);
}
}
}
/*
* If the entry is a submenu, then it needs to be wider to
- * accomodate the arrow. JCJ 31 July 2003
+ * accommodate the arrow.
*/
- if (menuEntry->SubMenu )
+ if (menuEntry->SubMenu)
menuEntry->Width += glutBitmapLength(
fgStructure.CurrentMenu->Font,
(unsigned char *)"_"
/*
* Creates a new menu object, adding it to the freeglut structure
*/
+int FGAPIENTRY glutCreateMenuUcall( FGCBMenuUC callback, FGCBUserData userData )
+{
+ /* The menu object creation code resides in fg_structure.c */
+ FREEGLUT_EXIT_IF_NOT_INITIALISED ( "glutCreateMenuUcall" );
+ if (fgState.ActiveMenus)
+ {
+ fgError( "Menu manipulation not allowed while menus in use." );
+ }
+
+ return fgCreateMenu( callback, userData )->ID;
+}
+
+/* Standard glutCreateMenu */
+static void fghCreateMenuCallback( int menu, FGCBUserData userData )
+{
+ FGCBMenu* callback = (FGCBMenu*)&userData;
+ (*callback)( menu );
+}
+
int FGAPIENTRY glutCreateMenu( FGCBMenu callback )
{
- /* The menu object creation code resides in freeglut_structure.c */
FREEGLUT_EXIT_IF_NOT_INITIALISED ( "glutCreateMenu" );
- if (fgGetActiveMenu())
- fgError("Menu manipulation not allowed while menus in use.");
-
- return fgCreateMenu( callback )->ID;
+ if (!callback)
+ {
+ return glutCreateMenuUcall( NULL, NULL );
+ }
+ FGCBMenu* reference = &callback;
+ return glutCreateMenuUcall( fghCreateMenuCallback, *((FGCBUserData*)reference) );
}
/*
menu = fgMenuByID( menuID );
freeglut_return_if_fail( menu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
- /* The menu object destruction code resides in freeglut_structure.c */
+ /* The menu object destruction code resides in fg_structure.c */
fgDestroyMenu( menu );
}
menuEntry = (SFG_MenuEntry *)calloc( sizeof(SFG_MenuEntry), 1 );
freeglut_return_if_fail( fgStructure.CurrentMenu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
menuEntry->Text = strdup( label );
subMenu = fgMenuByID( subMenuID );
freeglut_return_if_fail( fgStructure.CurrentMenu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
freeglut_return_if_fail( subMenu );
/*
* Changes the current menu's font
*/
-void FGAPIENTRY glutSetMenuFont( void* fontID )
+void FGAPIENTRY glutSetMenuFont( int menuID, void* fontID )
{
SFG_Font* font;
+ SFG_Menu* menu;
FREEGLUT_EXIT_IF_NOT_INITIALISED ( "glutSetMenuFont" );
- freeglut_return_if_fail( fgStructure.CurrentMenu );
+ menu = fgMenuByID( menuID );
+ freeglut_return_if_fail( menu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
font = fghFontByID( fontID );
if (!font)
+ {
fgWarning("glutChangeMenuFont: bitmap font 0x%08x not found. Make sure you're not passing a stroke font. Ignoring...\n",fontID);
- freeglut_return_if_fail( font );
+ return;
+ }
fgStructure.CurrentMenu->Font = fontID;
fghCalculateMenuBoxSize( );
FREEGLUT_EXIT_IF_NOT_INITIALISED ( "glutChangeToMenuEntry" );
freeglut_return_if_fail( fgStructure.CurrentMenu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
/* Get n-th menu entry in the current menu, starting from one: */
FREEGLUT_EXIT_IF_NOT_INITIALISED ( "glutChangeToSubMenu" );
freeglut_return_if_fail( fgStructure.CurrentMenu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
/* Get handle to sub menu */
FREEGLUT_EXIT_IF_NOT_INITIALISED ( "glutRemoveMenuItem" );
freeglut_return_if_fail( fgStructure.CurrentMenu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
/* Get n-th menu entry in the current menu, starting from one: */
freeglut_return_if_fail( fgStructure.CurrentWindow );
freeglut_return_if_fail( fgStructure.CurrentMenu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
freeglut_return_if_fail( button >= 0 );
freeglut_return_if_fail( fgStructure.CurrentWindow );
freeglut_return_if_fail( fgStructure.CurrentMenu );
- if (fgGetActiveMenu())
+ if (fgState.ActiveMenus)
fgError("Menu manipulation not allowed while menus in use.");
freeglut_return_if_fail( button >= 0 );