Make this script work in combat?

[Dynamicz]

/run local f={PlayerFrame} for i=1,#f do f[i]:EnableMouse(0)end

This makes it so I can click through my player frame but I want to use this in combat. How do I do this?

[Phanx]

Not possible. You cannot modify the mouse-enabled state of secure frames while in combat. The best you could do would be to enable the mouse when you enter combat, and disable it again when you leave.

Also, that is a horribly written macro. I'm guessing you found it, or based it on something you found, on ArenaJunkies. You can just do this instead:

/run PlayerFrame:EnableMouse(false)

If you want it to be a toggle:

/run PlayerFrame:EnableMouse(not PlayerFrame:IsMouseEnabled())

If you want to automatically enable the mouse during combat, and disable it out of combat:

local f = CreateFrame("Frame")
f:RegisterEvent("PLAYER_REGEN_ENABLED")
f:RegisterEvent("PLAYER_REGEN_DISABLED")
f:SetScript("OnEvent", function()
    if InCombatLockdown() then return end
    PlayerFrame:EnableMouse(UnitAffectingCombat("player"))
end)

If you need help turning the above code into an addon, copy and paste it into this page:
http://addon.ziuo.net/

[Sharparam]

Originally Posted by Phanx If you want it to be a toggle: /run PlayerFrame:EnableMouse(PlayerFrame:IsMouseEnabled())

Shouldn't that be "not PlayerFrame:IsMouseEnabled()" in order to toggle it? :)

[ravagernl]

Originally Posted by F16Gaming Shouldn't that be "not PlayerFrame:IsMouseEnabled()" in order to toggle it?

Quoted for truth.

A more fitting solution would be to enable the mouse based on modifier state (ctrl/shift/alt keys):

local f = CreateFrame("Frame")
 
function f:Event = function()
    if InCombatLockdown() then return end
    PlayerFrame:EnableMouse(IsModifierKeyDown())
end
 
function f:Load()
     PlayerFrame:EnableMouse(false)
     self:UnregisterEvent("PLAYER_REGEN_ENABLED")
     self:RegisterEvent("MODIFIER_STATE_CHANGED")
     self:SetScript("OnEvent", f.Event)
     self.Load = nil
end
 
if InCombatLockDown() then
     f:RegisterEvent("PLAYER_REGEN_ENABLED")
     f:SetScript("OnEvent", f.Load)
else
     f:Load()
end

Untested, and probably has errors, but I think something like this would work

[p3lim]

It's possible to change the attributes on the frames, so you could for example only click the frame if you're holding down shift.

PlayerFrame:SetAttribute('*type1', nil)
PlayerFrame:SetAttribute('shift-type1', 'target')

[Dynamicz]

Originally Posted by p3lim It's possible to change the attributes on the frames, so you could for example only click the frame if you're holding down shift. PlayerFrame:SetAttribute('*type1', nil) PlayerFrame:SetAttribute('shift-type1', 'target')

That actually sounds a lot better than what I'm trying to do. How do I put this in the game? To be honest I'm not too familiar with all this.

[Phanx]

Fixed the missing "not" in my original post.

Originally Posted by p3lim It's possible to change the attributes on the frames, so you could for example only click the frame if you're holding down shift. PlayerFrame:SetAttribute('*type1', nil) PlayerFrame:SetAttribute('shift-type1', 'target')

That doesn't affect the frame's mouse-interactability, though. You still can't click through the frame. All the click attributes do is change what happens when you click on the frame. Using the above code, clicking on the frame without holding Shift would simply create a "dead zone" where clicks did nothing at all.

A better overall solution might be to simply move the frames to some part of the screen you aren't accidentally clicking all the time.