Clickthrough Addon Help

[Exinium]

I'm a total addon noob. With that in mind, I think I know what's going on. Well, I should anyway. The post I obtained the addon from is from 2008, and as such, I have no clue what the changes to addons since then were.

Basically, I need to click through a single action bar. I was able to use /framestack to obtain the frame names of each individual button. I'd like to be able to toggle it on and off using the /clickthrough command. Here's what I've got.

local _G = getfenv(0)
local framelist = {
  "ElvUI_Bar6Button1"
  "ElvUI_Bar6Button2"
  "ElvUI_Bar6Button3"
  "ElvUI_Bar6Button4"
  "ElvUI_Bar6Button5"
  }

local clickable = false

SlashCmdList["CLICKTHROUGH"] = function()
  for _, v in ipairs(framelist) do
   _G[v]:EnableMouse(clickable)
  end
  if clickable then
   clickable = false
  else
   clickable = true
  end
end
   
SLASH_CLICKTHROUGH1 = '/clickthrough'

I don't think the issue lies in the .toc file, so I won't post that.

The problem is with the /clickthrough I think.

Edit:

/run ElvUI_Bar6Button1:EnableMouse(0)
/run ElvUI_Bar6Button2:EnableMouse(0)
/run ElvUI_Bar6Button3:EnableMouse(0)
/run ElvUI_Bar6Button4:EnableMouse(0)
/run ElvUI_Bar6Button5:EnableMouse(0)

Making a macro like that works. (I know that can be made into an array/for loop, but I didn't want the opportunity to **** that up.)

[Phanx]

Whenever you're working with addon code, you should always have some kind of error display enabled, such as BugSack. The game's default error display (Interface Options > Help > Display Lua errors) is not good enough, as it cannot show you errors that occur during the initial loading process -- like the ones in the code you posted. Your table syntax was incorrect; values must be separated by commas.

local frames = {
	"ElvUI_Bar6Button1",
	"ElvUI_Bar6Button2",
	"ElvUI_Bar6Button3",
	"ElvUI_Bar6Button4",
	"ElvUI_Bar6Button5"
}

local clickable = true

SLASH_CLICKTHROUGH1 = "/clickthrough"
SlashCmdList["CLICKTHROUGH"] = function()
	clickable = not clickable
	for i = 1, #frames do
		_G[frames[i]]:EnableMouse(clickable)
	end
	print("Frames are now", clickable and "clickable." or "not clickable.")
end

I also simplified some other things for you (you don't need a whole if/else check to toggle a boolean, and you don't need to use getfenv to access _G, and a simple i = 1, #tbl loop is much faster than ipairs(tbl) which really doesn't need to exist at all) and added a notification message.

[semlar]

No point in even having a table if those are the frame names, you could just do this..

local clickable = true
SLASH_CLICKTHROUGH1 = '/clickthrough'
SlashCmdList.CLICKTHROUGH = function()
    clickable = not clickable
    for i = 1, 5 do
        _G['ElvUI_Bar6Button' .. i]:EnableMouse(clickable)
    end
    print('Frames are now', clickable and 'clickable.' or 'not clickable.')
end

[Duugu]

Originally Posted by semlar No point in even having a table if those are the frame names, you could just do this..

chch

Touché ;D

[Phanx]

True, and I considered that, but I figured it was better to leave the table so other frames could easily be added in the future.