From f530acbf9e5270cd216f0354594fa7ed2927c697 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Tue, 24 Oct 2023 13:15:42 +0200 Subject: [PATCH] screenshot-window-sizer: Use built-in support for reversed bindings Mutter has had explicit support for reversed bindings for a while now. Use that, as it's more reliably than examining the modifiers manually. Close https://gitlab.gnome.org/GNOME/gnome-shell-extensions/-/issues/467 Part-of: --- extensions/screenshot-window-sizer/extension.js | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/extensions/screenshot-window-sizer/extension.js b/extensions/screenshot-window-sizer/extension.js index 497a861c..dfcf4e3f 100644 --- a/extensions/screenshot-window-sizer/extension.js +++ b/extensions/screenshot-window-sizer/extension.js @@ -65,10 +65,7 @@ export default class ScreenshotWindowSizerExtension extends Extension { * @param {Meta.KeyBinding} binding - the key binding */ _cycleScreenshotSizes(display, window, binding) { - // Probably this isn't useful with 5 sizes, but you can decrease instead - // of increase by holding down shift. - let modifiers = binding.get_modifiers(); - let backwards = (modifiers & Meta.VirtualModifier.SHIFT_MASK) !== 0; + const backwards = binding.is_reversed(); // Unmaximize first if (window.get_maximized() !== 0)