From e8ec86b20186c7599d9041a52f9174e37226a370 Mon Sep 17 00:00:00 2001
From: urnati <urnati@gmail.com>
Date: Mon, 8 Jul 2019 15:07:33 -0400
Subject: [PATCH] #1241 - Fix Movable to not remove the frame from the
 Blizzard managed table and not set isMovable on frames
 movable by Titan

---
 Titan/TitanMovable.lua |   40 ++++++++++++++++++++++++++++++++--------
 1 file changed, 32 insertions(+), 8 deletions(-)

diff --git a/Titan/TitanMovable.lua b/Titan/TitanMovable.lua
index cb59c6c..481d954 100755
--- a/Titan/TitanMovable.lua
+++ b/Titan/TitanMovable.lua
@@ -231,7 +231,7 @@ DESC: Adjust a given frame with the passed in values.
 VAR: frame - Text string of the frame name
 VAR: ... - list of frame position info
 NOTE: 
-Swiped from Vrul on wowinterface forum
+Swiped & modified from Vrul on wowinterface forum (https://www.wowinterface.com/forums/showthread.php?t=56519)
 
 The table UIPARENT_MANAGED_FRAME_POSITIONS does not hold all Blizzard frames.
 It is cleared for each frame in case the frame is in or might be in the table in the future.
@@ -240,26 +240,50 @@ Titan does not control the frames as other addons so we honor a user placed fram
 :NOTE
 --]]
 local function SetPosition(frame, ...)
+	local function UserMovable(frame)
+		local fname = frame:GetName()
+		local res = false
+		-- user may move these frames via the UI
+		if (fname == 'PlayerFrame' or fname == 'TargetFrame')
+		then
+			res = true 
+		else
+			res = false
+		end
+	end
     if type(frame) == 'string' then
         UIPARENT_MANAGED_FRAME_POSITIONS[frame] = nil
         frame = _G[frame]
     end
     if type(frame) == 'table' and type(frame.IsObjectType) == 'function' and frame:IsObjectType('Frame') then
-        local name = frame:GetName()
-        if name then
-            UIPARENT_MANAGED_FRAME_POSITIONS[name] = nil
+		if UserMovable(frame) then
+			-- Back off if the user has moved them via the UI
+		else
+			local name = frame:GetName()
+			-- Titan does not set in case the user needs to 
+			if name then
+				UIPARENT_MANAGED_FRAME_POSITIONS[name] = nil
+			end
         end
+
         frame:SetMovable(true)          -- allow frame to move
 -- Titan honors a user placed frame so we don't need this
 --        frame:SetUserPlaced(true)       -- tell Blizzard to back off
-        frame:SetDontSavePosition(true) 
-        frame:SetAttribute('ignoreFramePositionManager', true)
-        frame.ignoreFramePositionManager = true
+-- Since Titan adjusts rather than controls some frames, 
+-- Titan does not need to set these
+--        frame:SetDontSavePosition(true) 
+--        frame:SetAttribute('ignoreFramePositionManager', true)
+--        frame.ignoreFramePositionManager = true
         if ... then
             frame:ClearAllPoints()
             frame:SetPoint(...)
         end
-        frame:SetMovable(false)         -- lock frame from moving
+		-- Need to add in case user (not an addon) wants to move this frames
+		if UserMovable(frame) then
+			-- do nothing
+		else
+			frame:SetMovable(false)         -- lock frame from moving
+		end
     end
 end
 
-- 
1.7.9.5