View Single Post
07-21-10, 04:54 PM   #46
Dridzt
A Pyroguard Emberseer
 
Dridzt's Avatar
AddOn Author - Click to view addons
Join Date: Nov 2005
Posts: 1,360
Don't be too fast to thank me

Running with it there are some "side-effects"

The way the setting works now is like a global switch, but some screenshots are not served well from hiding the UI.
For example trade/achievement shots as the achievement/trade frame is also hidden.

I guess it should be made a little "smarter" so it only hides the UI for killshots.

Since you're passing the originating event to the screenshot function a way to make it more intuitive
would probably be to check that it's coming from PLAYER_REGEN_ENABLED and not another event.
eg.
Code:
if MultishotConfig.uihide and (string.find(strDebug, "PLAYER_REGEN_ENABLED") or string.find(strDebug, "UNIT_DIED") or string.find(strDebug, "PARTY_KILL")) then
It's good as proof of concept anyway, since it does simulate Alt+z
without running against protected status or causing taint.

Last edited by Dridzt : 07-21-10 at 07:15 PM.
  Reply With Quote