Skip to content

ActionBars: throttle the arg1==0 slot-changed full walk too #2021

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
EllesmereGaming merged 1 commit into from
Sep 8, 2026
+6 −3
Merged

ActionBars: throttle the arg1==0 slot-changed full walk too #2021

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions EllesmereUIActionBars/EllesmereUIActionBars.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5057,9 +5057,12 @@ do
-- bug whenever One Button Assist sits on a bar). Leading edge passes
-- immediately (drag-drop/spec-swap bursts hit distinct slots, each passing
-- instantly); repeats for the SAME slot inside the window defer to ONE
-- trailing re-dispatch, so the slot's final content always paints. arg1
-- == 0 ("all slots") is rare and always passes.
if event == "ACTIONBAR_SLOT_CHANGED" and arg1 and arg1 ~= 0 then
-- trailing re-dispatch, so the slot's final content always paints. arg1 == 0
-- ("all slots") shares this throttle under its own key (0 is not a real
-- slot number): a loadout swap changing talents/gear/bars at once can fire
-- it repeatedly, and unthrottled that was a full ~140-button walk per
-- firing with no coalescing (freeze reported via BTWLoadouts, Embrace 9.0.7).
if event == "ACTIONBAR_SLOT_CHANGED" and arg1 then
local now = GetTime()
local nextAt = ns._slotNext
if not nextAt then nextAt = {}; ns._slotNext = nextAt end
Expand Down