From 7e100f3ed4f1279c67d596654b665e3f347ca455 Mon Sep 17 00:00:00 2001 From: Victor Kareh Date: Fri, 29 Aug 2025 12:18:09 -0400 Subject: keybindings: Release keyboard grab on screenshot commands This prevents race conditions with applications like mate-screenshot that need to grab input (e.g. for area selection). Fixes mate-desktop/mate-utils#37 --- src/core/keybindings.c | 9 +++++++++ src/core/prefs.c | 4 ---- src/include/prefs.h | 5 +++++ 3 files changed, 14 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/core/keybindings.c b/src/core/keybindings.c index c74577f9..b1b1780a 100644 --- a/src/core/keybindings.c +++ b/src/core/keybindings.c @@ -2510,6 +2510,15 @@ handle_run_command (MetaDisplay *display, return; } + /* Release keyboard grabs immediately for screenshot commands that may need + * input grabbing. This prevents race conditions with applications like + * mate-screenshot that need to grab input for area selection. + */ + if (which >= SCREENSHOT_COMMAND_IDX && which <= WIN_SCREENSHOT_COMMAND_IDX) + { + ungrab_keyboard (display, event->xkey.time); + } + err = NULL; if (!meta_spawn_command_line_async_on_screen (command, screen, &err)) { diff --git a/src/core/prefs.c b/src/core/prefs.c index 23ed5d45..5299cf81 100644 --- a/src/core/prefs.c +++ b/src/core/prefs.c @@ -36,10 +36,6 @@ #define MAX_REASONABLE_WORKSPACES 36 -#define MAX_COMMANDS (32 + NUM_EXTRA_COMMANDS) -#define NUM_EXTRA_COMMANDS 2 -#define SCREENSHOT_COMMAND_IDX (MAX_COMMANDS - 2) -#define WIN_SCREENSHOT_COMMAND_IDX (MAX_COMMANDS - 1) /* If you add a key, it needs updating in init() and in the GSettings * notify listener and of course in the .gschema file. diff --git a/src/include/prefs.h b/src/include/prefs.h index 1bd7e258..ebfd2c55 100644 --- a/src/include/prefs.h +++ b/src/include/prefs.h @@ -29,6 +29,11 @@ #include "common.h" #include +#define MAX_COMMANDS (32 + NUM_EXTRA_COMMANDS) +#define NUM_EXTRA_COMMANDS 2 +#define SCREENSHOT_COMMAND_IDX (MAX_COMMANDS - 2) +#define WIN_SCREENSHOT_COMMAND_IDX (MAX_COMMANDS - 1) + typedef enum { META_PREF_MOUSE_BUTTON_MODS, -- cgit v1.2.1