From 55aaca0e0da7d3f3b1637398bd2f866a170cc485 Mon Sep 17 00:00:00 2001 From: BruebachL <44814898+BruebachL@users.noreply.github.com> Date: Sat, 8 Nov 2025 22:22:02 +0100 Subject: [PATCH] Introduce additional checks for playerMenu shortcut activation for judges (#6275) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Introduce additional checks for playerMenu shortcut activation when they are accessed by a judge, only activating them if the player is a local player. Took 2 hours 7 minutes Took 17 minutes * Undo example change. Took 5 minutes --------- Co-authored-by: Lukas Brübach --- cockatrice/src/game/player/menu/player_menu.cpp | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/cockatrice/src/game/player/menu/player_menu.cpp b/cockatrice/src/game/player/menu/player_menu.cpp index f1c461435..82747d482 100644 --- a/cockatrice/src/game/player/menu/player_menu.cpp +++ b/cockatrice/src/game/player/menu/player_menu.cpp @@ -139,7 +139,14 @@ void PlayerMenu::retranslateUi() void PlayerMenu::refreshShortcuts() { if (shortcutsActive) { - setShortcutsActive(); + // Judges get access to every player's menus but only want shortcuts to be set for their own. + if (player->getPlayerInfo()->getLocalOrJudge() && !player->getPlayerInfo()->getLocal()) { + setShortcutsInactive(); + } else { + setShortcutsActive(); + } + } else { + setShortcutsInactive(); } }