Incompatibility with Bandits User Interface - property "requiresReload" of sliders
david-tub opened this issue · 3 comments
Im faced with a bug when using the "requiresReload" property in sliders when using LibAddonMenu together with Bandits User Interface (BUI)!
The warning symbol + tooltip appears but when I change the value, the reload/apply button (corner bottom right) dos not appear and there is also no warning dialog for reloading / discard when leaving the settings menu.
It works like expected when BUI is disabled. Because I think it is a library related issue, I posted here.
Reproduction:
The bug occurs in my Addon BeamMeUp. Therefore following addons are active when I reproduce the issue:
BeamMeUp - Teleporter v1.5.4 Beta -> https://www.dropbox.com/s/0w8857ugl0jxq5f/BeamMeUp-Teleporter_1.5.4_Beta.zip?dl=0
LibAddonMenu-2.0
LibDialog
LibMapPing
LibCustomMenu
LibSlashCommander
LibZone
Bandits User Interface
The issue occurs in the last option in the settings menu of BeamMeUp ("UI Scaling").
I used the following option configuration:
{
type = "slider",
name = "UI Scale",
tooltip = "",
min = 0.5,
max = 1.5,
step = 0.05,
decimals = 2,
requiresReload = true,
getFunc = function() return mTeleSavedVars.Scale end,
setFunc = function(value) mTeleSavedVars.Scale = value end,
default = teleporterDefs["Scale"],
},
(The code can be found in "TeleAppUI.lua" from line 325)
I see that Bandits User Interface is embedding LAM in a very strange way. Will have to take a closer look at what it does.
Is this issue and #104 fixed, now that BUI removed LAM?
Tested with the latest version of BUI and I can confirm that both issues are fixed now.