Rune Sets

Rune Sets

7.4k Downloads

UI is jumbled after Phase 4 patch

Closed this issue ยท 6 comments

commented

After the update for Phase 4 today the UI for RuneSets has become a little jumbled, see the screenshot below. I wonder if Blizz updated a portion of the API to the retail version, thus messing up previous layouts. But, this is purely speculation. Thank you for the great addon! It's been a great QoL improvement.

image
commented
35x RuneSets/SetsUI.lua:1141: attempt to index field 'db' (a nil value)
[string "@RuneSets/SetsUI.lua"]:1141: in function `UpdateSetList'
[string "@RuneSets/SetsUI.lua"]:1279: in function <RuneSets/SetsUI.lua:1277>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: in function `Show'
[string "@Blizzard_UIParentPanelManager/Classic/UIParentPanelManager_Shared.lua"]:606: in function `SetUIPanel'
[string "@Blizzard_UIParentPanelManager/Classic/UIParentPanelManager_Shared.lua"]:451: in function `ShowUIPanel'
[string "@Blizzard_UIParentPanelManager/Classic/UIParentPanelManager_Shared.lua"]:319: in function <...PanelManager/Classic/UIParentPanelManager_Shared.lua:315>
[string "=[C]"]: in function `SetAttribute'
[string "@Blizzard_UIParentPanelManager/Classic/UIParentPanelManager_Shared.lua"]:1095: in function `ShowUIPanel'
[string "@Blizzard_CharacterFrame/Vanilla/CharacterFrame.lua"]:26: in function <...s/Blizzard_CharacterFrame/Vanilla/CharacterFrame.lua:4>
[string "=[C]"]: in function `ToggleCharacter'
[string "TOGGLECHARACTER0"]:1: in function <[string "TOGGLECHARACTER0"]:1>
commented
8x RuneSets/SetsUI.lua:125: bad argument #1 to 'SetJustifyV' (Usage: self:SetJustifyV(justifyV))
[string "=[C]"]: in function `SetJustifyV'
[string "@RuneSets/SetsUI.lua"]:125: in function <RuneSets/SetsUI.lua:118>
[string "@RuneSets/SetsUI.lua"]:185: in function <RuneSets/SetsUI.lua:156>
[string "@RuneSets/SetsUI.lua"]:719: in function `?'
[string "@RuneSets/SetsUI.lua"]:107: in function <RuneSets/SetsUI.lua:106>

Locals:
(*temporary) = EngravingFrameScrollFrameButton1Name {
 0 = <userdata>
}
(*temporary) = "CENTER"
commented

I'll try get on and have a look.
Unfortunately I stopped playing SoD when I realized my "todo" for the day when logging in, is larger than the phonebook ..
If I wanted retail gameplay I'd be playing shadowlands or whatever main line is now
๐Ÿ˜Š

commented

Much appreciated, let me know if I can provide any more info

commented

Much appreciated, let me know if I can provide any more info

You can try https://github.com/Road-block/RuneSets/releases/latest and tell me if it works ๐Ÿ˜„

Should show up on curse soon as well (in moderation queue atm)

Edit: For this #2 (comment) error I would look at #1
Some other addons are trying to cache runelist the wrong way; I'm not too keen on fixing their bugs when there's ways to do what they want without breaking other addons ๐Ÿคท๐Ÿปโ€โ™‚๏ธ

commented

Seems to be working perfectly again! Thank you for the speedy fixes and thank you again for the addon.