sporel/sporel-lib-core.selection
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(selection): sync prev_click_down before early returns

Browse Source 
Previously, set_enabled(false) caused prev_click_down to drift, so a
click that spanned a disable/enable window would be silently dropped.
Move release-edge sync above early returns; handle click_action==nil
guard inline so is_action_down isn't called with nil.
This commit is contained in:
Axel Meyer
2026-05-14 15:06:46 +02:00
parent c50714de51
commit bd43f2659d
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
init.lua
View File

@@ -308,14 +308,15 @@ end
-- Per-Frame -- Per-Frame
function M.update(dt) function M.update(dt)
if not enabled_flag then return end -- Release-edge detection must sync EVERY frame (even when disabled or unbound)
if click_action == nil then return end -- so prev_click_down doesn't drift across enable/disable boundaries.
local cur_down = (click_action ~= nil) and input.is_action_down(click_action) or false
-- Release-edge detection (lib-core.input has no was_action_released).
local cur_down = input.is_action_down(click_action)
local released = (prev_click_down and not cur_down) local released = (prev_click_down and not cur_down)
prev_click_down = cur_down prev_click_down = cur_down
if not enabled_flag then return end
if click_action == nil then return end
local mx, my = engine.input.get_mouse_pos() local mx, my = engine.input.get_mouse_pos()
if drag_state == "idle" then if drag_state == "idle" then