Sensi's Portal : Bug Quickslot UI Bug

5,133,421

2,570

My AddOns

View Bug Report

Quickslot UI Bug
Bug #:	4149
File:	AUI - Advanced UI
Date:	06/19/22 07:01 AM
By:	Hobskins
Status:	Unconfirmed
Hey! So earlier today i noticed there was a box on the top right corner of my screen, i confirmed it was this addon and then further narrowed it down to being related to the quick slot bars, i currently have 4 slots of the quick slot bars on display next to my abilitys. I did notces that if a click and drag from the box it pulls out a "Gold Coast swift survivor elixir, which i have in one of my quick slots that isn't one of the 4 slots. It goes away if i raise the number of slots. I'll insert some pictures for some referance. Thanks.

Notes

Sort Options

By: CodeStripper - 07/06/22 08:12 PM

Heyo, I'd like to include some more information on this and a possible solution:

1. The bug will only occur once the "HandleSlotChanged" function is called after the initial call. I noticed that this function calls the base esoui function "SetupItemSlot" when an item is populated in one of the quickslots. This function in turn calls "SetupActionSlotWithBg" then "SetupActionSlot" which enables the button.

2. If the quick slot is empty, it runs the "SetupEmptyActionSlot" function which does nothing, so anything that contains a quick slotted item past the variable defined in the addon settings (default 4) will have it's button UI re-enabled after a HandleSlotChanged function call occurs for that item slot.

The Fix:

In -> "QuickButtons.lua"
Functions ->
"AUI.Actionbar.QuickButtons.UpdateCooldowns"
"AUI.Actionbar.QuickButtons.OnFullUpdate"
"AUI.Actionbar.QuickButtons.OnSlotUpdate"

Add in a check to make sure the quick slot is active before calling "HandleSlotChanged" like the following:

Code:

for slotId, quickButton in pairs(gQuickButtonArray) do	
	if IsQuickSlotActive(slotId) then
		quickButton:HandleSlotChanged(HOTBAR_CATEGORY_QUICKSLOT_WHEEL)
	end
end

I've made this change on my end and everything seems to work great. Hopefully this helps in pushing out an official fix to this issue as it is fairly annoying lol