WindowClone: allow close window action on mice only (#2210)

Author: lenemter

data/gala.metainfo.xml.in

@@ -55,6 +55,7 @@
         <issue url="https://github.com/elementary/gala/issues/2170">Don’t show app icons in app window spread</issue>
         <issue url="https://github.com/elementary/gala/issues/2171">PiP dragging doesn't start until after mouse is released</issue>
         <issue url="https://github.com/elementary/gala/issues/2175">Gala crashes when receiving a notification while switching workspaces</issue>
+        <issue url="https://github.com/elementary/gala/issues/2199">Swiping in the multitasking view can close applications</issue>
       </issues>
     </release>
 

lib/DragDropAction.vala

@@ -74,7 +74,7 @@ namespace Gala {
          *
          * @param button The button which was pressed
          */
-        public signal void actor_clicked (uint32 button);
+        public signal void actor_clicked (uint32 button, Clutter.InputDeviceType device_type);
 
         /**
          * The type of the action
@@ -290,7 +290,10 @@ namespace Gala {
 
                     // release has happened within bounds of actor
                     if (clicked && x < ex && x + actor.width > ex && y < ey && y + actor.height > ey) {
-                        actor_clicked (event.get_type () == BUTTON_RELEASE ? event.get_button () : Clutter.Button.PRIMARY);
+                        actor_clicked (
+                            event.get_type () == BUTTON_RELEASE ? event.get_button () : Clutter.Button.PRIMARY,
+                            event.get_source_device ().get_device_type ()
+                        );
                     }
 
                     if (clicked) {

src/Widgets/MultitaskingView/WindowClone.vala

@@ -413,14 +413,11 @@ public class Gala.WindowClone : ActorTarget {
         destroy ();
     }
 
-    private void actor_clicked (uint32 button) {
-        switch (button) {
-            case Clutter.Button.PRIMARY:
-                selected ();
-                break;
-            case Clutter.Button.MIDDLE:
-                close_window (display.get_current_time ());
-                break;
+    private void actor_clicked (uint32 button, Clutter.InputDeviceType device_type = POINTER_DEVICE) {
+        if (button == Clutter.Button.PRIMARY) {
+            selected ();
+        } else if (button == Clutter.Button.MIDDLE && device_type == POINTER_DEVICE) {
+            close_window (display.get_current_time ());
         }
     }