Fix bug where navigating menus with arrows left items highlighted (ported from mvsdk)

aufau · aufau · commit 62ed63263085 · 2025-03-29T11:54:05.000+01:00
diff --git a/code/ui/ui_shared.c b/code/ui/ui_shared.c
@@ -3203,6 +3203,8 @@ itemDef_t *Menu_SetPrevCursorItem(menuDef_t *menu) {
 
 		if (Item_SetFocus(menu->items[menu->cursorItem], DC->cursorx, DC->cursory)) {
 			Menu_HandleMouseMove(menu, menu->items[menu->cursorItem]->window.rect.x + 1, menu->items[menu->cursorItem]->window.rect.y + 1);
+			if ( oldCursor > 0 && menu->items[oldCursor] )
+				Item_MouseLeave(menu->items[oldCursor]); // Stop highlighting the last element.
       return menu->items[menu->cursorItem];
     }
   }
@@ -3231,6 +3233,8 @@ itemDef_t *Menu_SetNextCursorItem(menuDef_t *menu) {
     }
 		if (Item_SetFocus(menu->items[menu->cursorItem], DC->cursorx, DC->cursory)) {
 			Menu_HandleMouseMove(menu, menu->items[menu->cursorItem]->window.rect.x + 1, menu->items[menu->cursorItem]->window.rect.y + 1);
+			if ( oldCursor > 0 && menu->items[oldCursor] )
+				Item_MouseLeave(menu->items[oldCursor]); // Stop highlighting the last element.
       return menu->items[menu->cursorItem];
     }
 

Original file line number	Diff line number	Diff line change
`@@ -3203,6 +3203,8 @@ itemDef_t Menu_SetPrevCursorItem(menuDef_t menu) {`
`3203`	`3203`
`3204`	`3204`	`if (Item_SetFocus(menu->items[menu->cursorItem], DC->cursorx, DC->cursory)) {`
`3205`	`3205`	`Menu_HandleMouseMove(menu, menu->items[menu->cursorItem]->window.rect.x + 1, menu->items[menu->cursorItem]->window.rect.y + 1);`
	`3206`	`+ if ( oldCursor > 0 && menu->items[oldCursor] )`
	`3207`	`+ Item_MouseLeave(menu->items[oldCursor]); // Stop highlighting the last element.`
`3206`	`3208`	`return menu->items[menu->cursorItem];`
`3207`	`3209`	`}`
`3208`	`3210`	`}`
`@@ -3231,6 +3233,8 @@ itemDef_t Menu_SetNextCursorItem(menuDef_t menu) {`
`3231`	`3233`	`}`
`3232`	`3234`	`if (Item_SetFocus(menu->items[menu->cursorItem], DC->cursorx, DC->cursory)) {`
`3233`	`3235`	`Menu_HandleMouseMove(menu, menu->items[menu->cursorItem]->window.rect.x + 1, menu->items[menu->cursorItem]->window.rect.y + 1);`
	`3236`	`+ if ( oldCursor > 0 && menu->items[oldCursor] )`
	`3237`	`+ Item_MouseLeave(menu->items[oldCursor]); // Stop highlighting the last element.`
`3234`	`3238`	`return menu->items[menu->cursorItem];`
`3235`	`3239`	`}`
`3236`	`3240`