WowAce

When choosing the 'All' drop down, next to the 'Sets' button, it only shows addons that all characters have set (with a grey check for ones set for *some* characters.) When you 'load' a new set of addons, it doesn't actually set it for all characters, it still only sets it for your current character.

I would like if this option would actually set the addons for all characters. I have 60+ characters that I login to for a cycle of activities, and I typically do 5-10 characters in a row for each activity before switching. It would be much easier to just activate a set for all characters when I switch activities, instead of doing it individually for just the characters of each activity, which would also need to be updated every time I add/remove addons.

I was able to fix this very simply by modifying the 'addon:LoadSet' function to check the current state of 'AddonList.Dropdown.text'. If it is set to "All", then set character to "nil", otherwise set character to "UnitGUID("player")". No other code needs to be changed, since the character state only matters when loading I believe. Since BetterAddonList appears to support multiple locales, you would probably have to do additional checks based on language, if this were added to the addon.

function addon:LoadSet(name)
	if AddonList and AddonList.Dropdown and AddonList.Dropdown.text then
		if AddonList.Dropdown.text == "All" then
			character = nil
		else
			character = UnitGUID("player")
		end
	end
	
	C_AddOns.DisableAllAddOns(character)
	self:EnableSet(name)
end