Merge 2b5d8a0978 into fc640d5f6c

Closes #6456.

The i3 window manager ignores modifiers when a button is being released.
Mimic this behaviour in Sway.

sway/input/keyboard.c

@@ -162,7 +162,7 @@ static void get_active_binding(const struct sway_shortcut_state *state,
 		bool binding_inhibited = (binding->flags & BINDING_INHIBITED) != 0;
 		bool binding_release = binding->flags & BINDING_RELEASE;
 
-		if (modifiers ^ binding->modifiers ||
+		if ((modifiers ^ binding->modifiers && !release) ||
 				release != binding_release ||
 				locked > binding_locked ||
 				inhibited > binding_inhibited ||

sway/input/seatop_default.c

@@ -131,7 +131,7 @@ static struct sway_binding* get_active_mouse_binding(
 	struct sway_binding *current = NULL;
 	for (int i = 0; i < bindings->length; ++i) {
 		struct sway_binding *binding = bindings->items[i];
-		if (modifiers ^ binding->modifiers ||
+		if ((modifiers ^ binding->modifiers && !release) ||
 				e->pressed_button_count != (size_t)binding->keys->length ||
 				release != (binding->flags & BINDING_RELEASE) ||
 				!(click_region & binding->flags) ||