MovePlayerFrame

It works, a lot of work for me, but it do the job. I had not test the vehicles, but: Thank you.

You're a genious. I've tried other addons to get them moving but this is just great. Simplicity at its best. Thank you :)

Just a reminder to everyone, make sure to write down your current coordinates for your frames before you update. I forgot and had to take a few minutes to reconfigure.

Extra thanks to Rom for using my party frame edit.

Love this thing. I've been looking for an add-on like this for some time. I don't like the ugly bars from Pitbull/Xperl/etc and this works great for me. Even with manual coordinates, setup was pretty fast and easy (and I'm very picky).

My only question is: Is/will there be an update to move party member frames?
If I could move those down a few notches, I could put all my raid frames at the top and make my UI much cleaner.

Thanks again for this great add-on.

Response to my previous question:

I looked up the party frames codes and made a custom edit. (Periods used to preserve formatting.)

..if (PartyMemberFrameZ:IsShown()) then
....PartyMemberFrameZ:ClearAllPoints();
....PartyMemberFrameZ:SetPoint("CENTER", UIParent, "CENTER", X, Y);
..end

Just copy that 4 times and replace Z with the party numbers (1,2,3,4), and then make your coordinate adjustments accordingly.

If you like the edit, please feel free to add it to your current code. Freaking awesome add-on just got a bit better, if I do say so myself.

Few things I'd like suggest now that I've actually opened up the file.

You can also eliminate the .xml entirely if you wanted to in order to simplify things more. At the start of the .lua file, you can:

local f = CreateFrame("FRAME")
f:SetScript("OnUpdate", MovePlayerFrame)

No sense in making a file bigger than it needs to be. Just that much more for the compiler to read during loading. Yes, I realize not much loading is needed for this, but every little bit adds up in the end.

On that note, if you wanted to boil it down further, you can hook into the script as I mentioned before. Here's a link to when I asked about it and the response: http://www.wowinterface.com/forums/showthread.php?p=146188#post146188

For yours, you could simply:

PlayerFrame:HookScript("OnUpdate", MovePlayerFrame)
TargetFrame:HookScript("OnUpdate", MovePlayerFrame)
FocusFrame:HookScript("OnUpdate", MovePlayerFrame)

This would also eliminate the need for running your own OnUpdate process, reducing performance impact even more.

As I said, these are just suggestions. Use em if you want. Kudos. =D

Event: UNIT_EXITED_VEHICLE

Requires a unit check for arg1.

if arg1 == "player" then
do stuff
end

Note on the OnUpdate hooking: The point is that if you hooked the script, you would have no need for the OnUpdate at all. You wouldn't even need the visibility check as "OnUpdate" scripts only run while a frame is visible (which would be then handled by the frames themselves).

I've played with this issue for a while now until someone pointed me to the event firing. I just recently added the fix to CFM and it works fine now; just haven't released yet as I'm fixing some other stuff first. I started with XML at first, but it became a pain. I also banged my head for a while with OnUpdate stuff as well. Finally figured it out, here's an outline for you explaining each step.

http://pastey.net/117152

The problem you'd run into with this, however, is that your code would override the existing OnUpdate script for the frame, which would break it (no hp/mana/energy updates, etc). Which is why hooking is a much safer if you were to use this method.

Don't feel bad if you don't get it at first. It's a lot of "advanced" stuff to take in all at once, especially if you've only been working with it for 2 days. Remember: everyone started new at some point. =D

Alternatively, if you want don't want to run OnUpdate constantly, you can directly hook into the frames existing scripts as I did with CFM. You can simply hook into the OnShow function (or whatever you want) and apply the frame changes.

Nice job finding a solution for the vehicle issue. I did a CPU profile of this though, and CPU time is a little ugly because of the OnUpdate nature of this mod. Not a problem if you have a good computer, but I think it's worth noting for those with underpowered computers.

Incidentally, an easy optimization you could do to reduce that CPU time considerably is add a check for the Pet, Target and Focus frames to see if they're shown ( i.e. if (TargetFrame:IsShown()) then ) before moving them.

Anyways, thanks. I've been looking for a solution for the vehicle thing and I never would have thought of this.

Tested this addon, and love it totally not hard to edit the settings at all. A+ just wish my buffs would move as well anytips? there still at top right