<type>(scope): <description>

modify:
* `./src/libs/feats.lua`: refactor multiselect to ensure multiselecting
  only happens after the user has moved the mouse
* `./src/libs/timers.lua`: set/unset multiselect start position on
  keypress
* `./src/main.lua`: add multiselect start position to mod state

Signed-off-by: uptu <uptu@uptu.dev>

Author: uptudev

src/libs/feats.lua

@@ -1,36 +1,31 @@
 --- Update function for multiselect
 -- Only runs when `STATE.multiselecting` is not nil.
 function multiselect_hold()
-    if STATE.multiselecting == true then
-        STATE.multiselecting = false;
-        STATE.prev_prev_target = G.CONTROLLER.hovering.prev_target;
-        G.CONTROLLER.hovering.prev_target = G.CONTROLLER.hovering.target;
+    if
+        G.CONTROLLER.hovering.target ~= nil and
+        G.CONTROLLER.hovering.target:is(Card) and
+        G.CONTROLLER.hovering.target.area == G.hand
+    then
         if
-            G.CONTROLLER.hovering.target ~= nil and
-            G.CONTROLLER.hovering.target:is(Card) and not
-            G.CONTROLLER.hovering.target.area == G.hand
+            STATE.multiselecting == true and
+            Vector_Dist(STATE.multiselect_down_pos, G.CONTROLLER.cursor_position) > 0.1 * G.MIN_CLICK_DIST
         then
+            STATE.multiselecting = false;
+            STATE.prev_prev_target = G.CONTROLLER.hovering.prev_target;
+            G.CONTROLLER.hovering.prev_target = G.CONTROLLER.hovering.target;
             G.CONTROLLER.hovering.target:click();
-        end
-    elseif G.CONTROLLER.hovering.prev_target ~= G.CONTROLLER.hovering.target then
-        if
-            G.CONTROLLER.hovering.target ~= nil and
-            G.CONTROLLER.hovering.target:is(Card) and not
-            G.CONTROLLER.hovering.target.area == G.hand and
-            STATE.prev_prev_target == G.CONTROLLER.hovering.target and
-            G.CONTROLLER.hovering.prev_target:is(Card) and
-            G.CONTROLLER.hovering.prev_target.area == G.hand
-        then
-            G.CONTROLLER.hovering.prev_target:click();
-        end
-        if
-            G.CONTROLLER.hovering.target ~= nil and
-            G.CONTROLLER.hovering.target:is(Card) and
-            G.CONTROLLER.hovering.target.area == G.hand
+        elseif
+            G.CONTROLLER.hovering.prev_target ~= G.CONTROLLER.hovering.target and
+            STATE.multiselecting == false
         then
+            if
+                STATE.prev_prev_target == G.CONTROLLER.hovering.target
+            then
+                G.CONTROLLER.hovering.prev_target:click();
+            end
             G.CONTROLLER.hovering.target:click();
+            STATE.prev_prev_target = G.CONTROLLER.hovering.prev_target;
         end
-        STATE.prev_prev_target = G.CONTROLLER.hovering.prev_target;
     end
 end
 

src/libs/timers.lua

@@ -31,6 +31,10 @@ function TimerTable:start(button)
         STATE.bind_map.binding_to_button['multiselect'].button == button
     then
         STATE.multiselecting = true;
+        STATE.multiselect_down_pos = {
+            x = G.CONTROLLER.cursor_position.x,
+            y = G.CONTROLLER.cursor_position.y,
+        };
     end
     self[button] = 0;
     return self;
@@ -47,6 +51,7 @@ function TimerTable:stop(button)
         STATE.bind_map.binding_to_button['multiselect'].button == button
     then
         STATE.multiselecting = nil;
+        STATE.multiselect_down_pos = nil;
     end
     -- binds to config if listening here
     if STATE.listening then

src/main.lua

@@ -39,4 +39,5 @@ STATE = {
     cfg_gui_parent = nil,
     multiselecting = nil,
     prev_prev_target = nil,
+    multiselect_down_pos = nil,
 };